25250 2014-04-03 17:18:08 +0000 Ended track should not be added to MediaStream, when addTrack() is invoked for and ended track. 2014-04-10 08:09:16 +0000 1 1 1 Unclassified WebRTC Working Group Media Capture and Streams unspecified All All RESOLVED WONTFIX P2 normal --- 1 kiran.guduru adam.bergkvist kiran.guduru public-media-capture stefan.lk.hakansson oldest_to_newest 103363 0 kiran.guduru 2014-04-03 17:18:08 +0000 When section 4.2.3 [1] addTrack() is invoked, if that track is already ended, it should not be added to a MediaStream. The steps for addTrack() should be modified as below to make it sync with 4.2 (Adding tracks when MediaStream constructor is invoked "If track has ended, then abort these steps and continue with the next track (if any).") 1. Let track be the MediaStreamTrack argument and stream this MediaStream object. 2. If track has ended, then abort these steps. 3. If track is already in stream's track set, then abort these steps. 4. Add track to stream's track set. (step 2 is the newly added step). [1] http://dev.w3.org/2011/webrtc/editor/getusermedia.html#dom-mediastreamtrack-enabled 103391 1 kiran.guduru 2014-04-04 04:52:18 +0000 As per Martin comments [1], if the resolution is to remove the dependency of track state while adding a track to MediaStream, then the text in 4.2 of [2] should be changed by removing step-2 "2. If track has ended, then abort these steps and continue with the next track (if any)." [1] http://lists.w3.org/Archives/Public/public-media-capture/2014Apr/0075.html [2] http://dev.w3.org/2011/webrtc/editor/getusermedia.html#mediastream 103395 2 stefan.lk.hakansson 2014-04-04 07:21:57 +0000 Assigning to Adam (who has promised to do some bug fixing!). 103396 3 stefan.lk.hakansson 2014-04-04 07:32:49 +0000 Already fixed in the current Ed's draft: http://dev.w3.org/2011/webrtc/editor/archives/20140321/getusermedia.html#methods 103399 4 kiran.guduru 2014-04-04 07:59:10 +0000 addTrack() is upto mark but Step 2 in MediaStream constructor(4.2 MediaStream)[1], which is specifying not to add ended track, is still present as shown below. Run the following sub steps for each MediaStreamTrack in the array: 1. Add track: Let track be the MediaStreamTrack about to be processed. 2. If track has ended, then abort these steps and continue with the next track (if any). 3. Add track to stream's track set. Which is required to be removed (As per comment 2). The new steps should be as follows (removing step 2). 1. Add track: Let track be the MediaStreamTrack about to be processed. 2. Add track to stream's track set. [1] http://dev.w3.org/2011/webrtc/editor/getusermedia.html#mediastream 103654 5 adam.bergkvist 2014-04-10 07:57:24 +0000 I agree that we should stick with one approach here. 103655 6 adam.bergkvist 2014-04-10 08:09:16 +0000 Updated MediaStream() constructor as Kiran proposed. https://github.com/fluffy/webrtc-w3c/commit/1b6070e125689b281ece573d7086ad153c8d28c6 Closing this bug as WONTFIX (since we're not fixing what the bug title says).