Bug 187591 - [GTK] imported/w3c/web-platform-tests/media-source/mediasource-getvideoplaybackquality.html is flaky
Summary: [GTK] imported/w3c/web-platform-tests/media-source/mediasource-getvideoplayba...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ms2ger (he/him; ⌚ UTC+1/+2)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-07-12 06:38 PDT by Miguel Gomez
Modified: 2018-08-22 07:13 PDT (History)
4 users (show)

See Also:


Attachments
Patch (1.65 KB, patch)
2018-08-22 06:39 PDT, Ms2ger (he/him; ⌚ UTC+1/+2)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Miguel Gomez 2018-07-12 06:38:48 PDT
imported/w3c/web-platform-tests/media-source/mediasource-getvideoplaybackquality.html [ Failure Crash ]

The test moves between crashing and failing with text constantly. When it fails with text, this is the diff:

--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/media-source/mediasource-getvideoplaybackquality-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/media-source/mediasource-getvideoplaybackquality-actual.txt
@@ -1,4 +1,4 @@
 
-PASS Test HTMLVideoElement.getVideoPlaybackQuality() with MediaSource API 
-PASS Test the totalFrameDelay attribute of HTMLVideoElement.getVideoPlaybackQuality() with MediaSource API 
+FAIL Test HTMLVideoElement.getVideoPlaybackQuality() with MediaSource API assert_unreached: Unexpected event 'error' Reached unreachable code
+FAIL Test the totalFrameDelay attribute of HTMLVideoElement.getVideoPlaybackQuality() with MediaSource API assert_unreached: Unexpected event 'error' Reached unreachable code
 

The first crash happened in r233638. Before that it was mostly passing with text failure every now and then. I cannot be sure that r233638 is the cause of the crash, but adding the author just in case. The crash log can be seen here


https://build.webkit.org/results/GTK%20Linux%2064-bit%20Release%20(Tests)/r233765%20(7376)/imported/w3c/web-platform-tests/media-source/mediasource-getvideoplaybackquality-crash-log.txt
Comment 1 Thibault Saunier 2018-07-12 07:23:20 PDT
Seem related to the implementation of MediaStream, looking into it.
Comment 2 Thibault Saunier 2018-07-12 09:13:34 PDT
(In reply to Thibault Saunier from comment #1)
> Seem related to the implementation of MediaStream, looking into it.

Sorry I misread it is about MediaSource (a.k.a MSE) and is not related   at all with MediaStream. Also I do not see any way it is related to r233638 as that one is about flatpak which is not use on the build bots yet.
Comment 3 Ms2ger (he/him; ⌚ UTC+1/+2) 2018-08-22 06:39:32 PDT
Created attachment 347786 [details]
Patch
Comment 4 WebKit Commit Bot 2018-08-22 07:13:35 PDT
Comment on attachment 347786 [details]
Patch

Clearing flags on attachment: 347786

Committed r235168: <https://trac.webkit.org/changeset/235168>
Comment 5 WebKit Commit Bot 2018-08-22 07:13:37 PDT
All reviewed patches have been landed.  Closing bug.