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
"The 'Logitech, Inc. QuickCam Pro 9000' webcam is busy so could not be started")
This is fine, no?
(In reply to comment #1)
> This is fine, no?
It's not, I misread.
Here is a branch to fix this:
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.
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
See further comments in bug #28707
I agree this should be split from SetSending .. the pending modes for SetSending don'T make so much sense imho..
all solved on bug #28693
*** This bug has been marked as a duplicate of bug 28707 ***