Bug 151344 - [GTK] WebRTC: Update MediaStream "active state" definition when the specification is fixed
Summary: [GTK] WebRTC: Update MediaStream "active state" definition when the specifica...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Philippe Normand
URL:
Keywords:
Depends on:
Blocks: 143211
  Show dependency treegraph
 
Reported: 2015-11-17 02:39 PST by Adam Bergkvist
Modified: 2021-09-01 07:43 PDT (History)
12 users (show)

See Also:


Attachments
Patch (3.17 KB, patch)
2021-08-16 07:39 PDT, Philippe Normand
aperez: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Bergkvist 2015-11-17 02:39:33 PST
As of writing, the Media Capture and Stream specification [1] has some inconsistencies with regards to the MediaStream "active state".

[1] http://w3c.github.io/mediacapture-main/archives/20150925/getusermedia.html
Comment 1 Adam Bergkvist 2015-11-17 03:06:49 PST
The proposed patch for https://webkit.org/b/151346 skips two tests and refers to this bug.
Comment 2 Adam Bergkvist 2015-11-17 14:42:41 PST
https://webkit.org/b/151346 mentioned above was closed as a duplicate.

New bug is: https://webkit.org/b/151360
Comment 3 Adam Bergkvist 2015-12-09 02:23:18 PST
Spec update is under way
https://github.com/w3c/mediacapture-main/pull/291
Comment 4 Carlos Alberto Lopez Perez 2017-01-17 17:52:35 PST
The tests fast/mediastream/MediaStream-add-ended-tracks.html and fast/mediastream/MediaStream-add-remove-tracks.html are marked as passing on the Mac port.

On GTK+ fast/mediastream/MediaStream-add-ended-tracks.html timeouts and fast/mediastream/MediaStream-add-remove-tracks.html fails with this diff:

--- /home/clopez/webkit/webkit/layout-test-results/fast/mediastream/MediaStream-add-remove-tracks-expected.txt
+++ /home/clopez/webkit/webkit/layout-test-results/fast/mediastream/MediaStream-add-remove-tracks-actual.txt
@@ -49,19 +49,10 @@
 PASS stream2.getVideoTracks().length is 0
 
 *** active attribute is still true (until event is fired)
-PASS stream2.active is true
-
-Stream2 is inactive.
-PASS stream2.active is false
-
-*** add non-ended track
-PASS audioTrack.readyState is not "ended"
-
-*** active attribute is still false (until event is fired)
-PASS stream2.active is false
+FAIL stream2.active should be true. Was false.
 
 Stream2 is active.
-PASS stream2.active is true
+FAIL stream2.active should be true. Was false.
 PASS successfullyParsed is true
 
 TEST COMPLETE



Updated the GTK+ expectations file in r210841: <http://trac.webkit.org/changeset/210841>
Comment 5 Philippe Normand 2021-08-16 07:39:37 PDT
Created attachment 435601 [details]
Patch