Bug 28693

Summary:	Call needs to be able to indicate to the UI that starting to stream has failed for some reason
Product:	Telepathy	Reporter:	Sjoerd Simons <sjoerd>
Component:	tp-spec	Assignee:	Telepathy bugs list <telepathy-bugs>
Status:	RESOLVED DUPLICATE	QA Contact:	Telepathy bugs list <telepathy-bugs>
Severity:	normal
Priority:	medium	CC:	olivier.crete
Version:	unspecified	Keywords:	patch
Hardware:	Other
OS:	All
URL:	http://git.collabora.co.uk/?p=user/jonny/telepathy-spec.git;a=shortlog;h=refs/heads/stream-send-failed
Whiteboard:	Call
i915 platform:		i915 features:

Description Sjoerd Simons 2010-06-23 06:34:36 UTC

From #24936

You need to add a way for the streaming implementation to tell
the CM that starting to send has failed (ie, I can't open the camera device
because its in use). And that should result in the change in Senders to fail
and the UI being informed of that. Not in ending the video stream like the old
StreamedMedia.

Comment 1 Jonny Lamb 2010-10-13 02:42:04 UTC

e.g.

Content.Remove(
  Content_Removal_Reason_Error,
  "ofdT.Error.DeviceBusy",
  "The 'Logitech, Inc. QuickCam Pro 9000' webcam is busy so could not be started")

This is fine, no?

Comment 2 Jonny Lamb 2010-10-18 09:39:28 UTC

(In reply to comment #1)
> This is fine, no?

It's not, I misread.

Here is a branch to fix this:

http://git.collabora.co.uk/?p=user/jonny/telepathy-spec.git;a=shortlog;h=refs/heads/stream-send-failed

Comment 3 Olivier Crête 2010-10-18 10:00:35 UTC

Adding a DeviceBusy error is probably fine too (I guess most CMs will not care why they can't send).

Also, I must admit that using the same method indicate the user's desire to start/stop sending and the streaming implementation's ability to do it seems wrong to me.

Comment 4 Sjoerd Simons 2010-10-29 07:25:38 UTC

Using the SetStreaming both as an indication that the user wants to send and for the streaming layer to fail seems wrong and racy.

Seems more sensible to have a seperate option on the Media interface for the stream layer to indicat e its state

Comment 5 Olivier Crête 2011-01-05 19:50:18 UTC

See further comments in bug #28707

Comment 6 Olivier Crête 2011-01-26 12:19:31 UTC

I agree this should be split from SetSending .. the pending modes for SetSending don'T make so much sense imho..

Comment 7 Olivier Crête 2011-02-11 08:50:33 UTC

all solved on bug #28693

*** This bug has been marked as a duplicate of bug 28707 ***

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.