Currently, the Mute interface says that the CM will tell the streaming implemention to mute, but that is not currently in the spec.
I propose adding Pending_Mute and Muted to the Stream_Flow_State enum in Stream.I.Media.
Created attachment 54476 [details] [review]
Add Muted and Pending_Mute states
Created attachment 54477 [details] [review]
Patch that also removes the paused state
I have no idea what the paused state is for, so I just removed it.. My guess is that is was to implement Mute, but it's quite unclear to me.
+ The streaming implementation has successfully muted sending,
+ and the local user's privacy is protected.
+ The streaming implementation has been told to mute sending,
+ but it has not yet indicated that it has done so.
Not sure why your indentation is funky here.
Otherwise it looks fine. Merge away.
(In reply to comment #3)
> Not sure why your indentation is funky here.
(As in, please fix this before merging.)
Author: Olivier Crête <email@example.com>
Date: Thu Dec 15 19:25:44 2011 -0500
Stream.I.Media: Add Muted and Pending_Mute stream flow states.
This way the CM can tell the streaming implementation to mute.
Also, remove the useless Paused state