Bug 186596

but fast/mediastream/media-stream-track-source-failure.html is flaky

sometimes crashes and sometimes fails with this diff: 

--- a/gtk-linux-64-release-tests/build/layout-test-results/fast/mediastream/media-stream-track-source-failure-expected.txt
+++ b/gtk-linux-64-release-tests/build/layout-test-results/fast/mediastream/media-stream-track-source-failure-actual.txt
@@ -1,6 +1,10 @@
+CONSOLE MESSAGE: line 2451: Unhandled Promise Rejection: Error: assert_equals: Video track should have no size expected 0 but got 1120
+CONSOLE MESSAGE: line 2451: Unhandled Promise Rejection: Error: assert_equals: Video track should have no size expected 0 but got 1120
 
 
+Harness Error (FAIL), message = assert_equals: Video track should have no size expected 0 but got 1120
+
 PASS Create stream 
-PASS End video track only 
-PASS End audio track only 
+FAIL End video track only promise_test: Unhandled rejection with value: "Device state did not change in 5 second"
+FAIL End audio track only promise_test: Unhandled rejection with value: "Device state did not change in 5 second"

And another one:

 - The test imported/w3c/web-platform-tests/mediacapture-streams/MediaStream-default-feature-policy.https.html is timing out

(In reply to Carlos Alberto Lopez Perez from comment #2)
> And another one:
> 
>  - The test
> imported/w3c/web-platform-tests/mediacapture-streams/MediaStream-default-
> feature-policy.https.html is timing out

I think this one is unrelated to the commit (it was not activated in there).

(In reply to Carlos Alberto Lopez Perez from comment #0)
> The following mediastream tests crash on platform GTK since r232589


Remaining after r232796

+ fast/mediastream/change-tracks-media-stream-being-played.htm
+ fast/mediastream/MediaStream-video-element.html

Those seem racy to me (can't reproduce locally yet), they do not happen with Debug builds also it seems. I am investigating it anyway!

(In reply to Carlos Alberto Lopez Perez from comment #1)
> but fast/mediastream/media-stream-track-source-failure.html is flaky
> 
> sometimes crashes and sometimes fails with this diff: 
> 
> ---
> a/gtk-linux-64-release-tests/build/layout-test-results/fast/mediastream/
> media-stream-track-source-failure-expected.txt
> +++
> b/gtk-linux-64-release-tests/build/layout-test-results/fast/mediastream/
> media-stream-track-source-failure-actual.txt
> @@ -1,6 +1,10 @@
> +CONSOLE MESSAGE: line 2451: Unhandled Promise Rejection: Error:
> assert_equals: Video track should have no size expected 0 but got 1120
> +CONSOLE MESSAGE: line 2451: Unhandled Promise Rejection: Error:
> assert_equals: Video track should have no size expected 0 but got 1120
>  
>  
> +Harness Error (FAIL), message = assert_equals: Video track should have no
> size expected 0 but got 1120
> +
>  PASS Create stream 
> -PASS End video track only 
> -PASS End audio track only 
> +FAIL End video track only promise_test: Unhandled rejection with value:
> "Device state did not change in 5 second"
> +FAIL End audio track only promise_test: Unhandled rejection with value:
> "Device state did not change in 5 second"

Will investigate.

All looks green with latest build:


https://build.webkit.org/results/GTK%20Linux%2064-bit%20Release%20(Tests)/r232842%20(7008)/results.html

I have not been able to reproduce issues that happened in r232796 (namely  fast/mediastream/change-tracks-media-stream-being-played.html) and will follow test results see if it flakes again.

fast/mediastream/apply-constraints-audio.html seems to be passing.
fast/mediastream/change-tracks-media-stream-being-played.html times out flakily.
fast/mediastream/media-stream-track-source-failure.html times out flakily.

The rest seem to pass to me.

Applies to bots and locally.

What I mean is that I see no crash.