Bug 29595

Summary:	Call: Add an explicit instruction to start sending on Stream.I.Media
Product:	Telepathy	Reporter:	Olivier Crête <olivier.crete>
Component:	tp-spec	Assignee:	Telepathy bugs list <telepathy-bugs>
Status:	RESOLVED FIXED	QA Contact:	Telepathy bugs list <telepathy-bugs>
Severity:	normal
Priority:	medium
Version:	unspecified
Hardware:	Other
OS:	All
Whiteboard:	Call
i915 platform:		i915 features:
Bug Depends on:
Bug Blocks:	29590

Description Olivier Crête 2010-08-16 11:03:48 UTC

Forcing tp-fs to track the self handle, group or otherwise in the senders group is too complex. I don't to have to track handles and stuff in tp-fs. And I don't want to differentiate between Group and non-Group channels.

There should be a simple "Sending" property on the Stream.I.Media interface

We should also add a "Receiving" property, this would help with bug #28707.

Comment 1 Jonny Lamb 2010-10-06 01:18:54 UTC

(In reply to comment #0)
> Forcing tp-fs to track the self handle, group or otherwise in the senders group
> is too complex. I don't to have to track handles and stuff in tp-fs. And I
> don't want to differentiate between Group and non-Group channels.
> 
> There should be a simple "Sending" property on the Stream.I.Media interface

Wouldn't something like a (Start)Sending signal be better? If you have
the property then you'll already have to have change notification for
you to know when to start sending, so we could just leave it with the
signal?

> We should also add a "Receiving" property, this would help with bug #28707.

Same here.

Comment 2 Olivier Crête 2010-10-06 06:46:42 UTC

I want a property with change notification, there is a PropertiesChanged signal on the standard dbus Properties interface now. The reason to have a property, not only a signal, is that otherwise the CM has to know when you start listening. Also, you want a 3rd party to be able to look in and see what the current status is.

Comment 3 Jonny Lamb 2010-10-14 00:21:11 UTC

This is what I'm adding:

http://people.freedesktop.org/~jonny/telepathy-spec-call/spec/Call_Stream.html#org.freedesktop.Telepathy.Call.Stream.DRAFT.LocalSendingState

pending review from Sjoerd.

Comment 4 Olivier Crête 2010-10-14 07:12:21 UTC

SetSending() should probably be in the Media Interface, not the UI-facing one.

Comment 5 Jonny Lamb 2010-10-18 08:58:33 UTC

Fixed in git.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.