Bug 28693 - Call needs to be able to indicate to the UI that starting to stream has failed for some reason
Summary: Call needs to be able to indicate to the UI that starting to stream has faile...
Status: RESOLVED DUPLICATE of bug 28707
Alias: None
Product: Telepathy
Classification: Unclassified
Component: tp-spec (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Telepathy bugs list
QA Contact: Telepathy bugs list
URL: http://git.collabora.co.uk/?p=user/jo...
Whiteboard: Call
Keywords: patch
Depends on:
Blocks:
 
Reported: 2010-06-23 06:34 UTC by Sjoerd Simons
Modified: 2011-02-11 08:50 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments

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.