Bug 160119 - [GTK] Tests failing after enabling ThreadedCompositor
Summary: [GTK] Tests failing after enabling ThreadedCompositor
Status: REOPENED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: Other
Hardware: PC Linux
: P2 Normal
Assignee: Charlie Turner
URL:
Keywords:
Depends on: 174982 175881
Blocks:
  Show dependency treegraph
 
Reported: 2016-07-23 07:55 PDT by Michael Catanzaro
Modified: 2017-08-23 05:33 PDT (History)
6 users (show)

See Also:


Attachments
Patch (1.47 KB, patch)
2017-07-11 10:11 PDT, Charlie Turner
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Catanzaro 2016-07-23 07:55:02 PDT
Using this bug to track test failures caused by the switch to enable ThreadedCompositor.

No patches in this bug; this bug is just for tracking test expectations.
Comment 1 Michael Catanzaro 2016-07-23 08:06:33 PDT
New flaky crashes (excluding bug #160078):

webkit.org/b/160119 media/media-fullscreen-inline.html [ Crash Pass ]
webkit.org/b/160119 media/video-playbackrate.html [ Crash Pass ]
Comment 2 Michael Catanzaro 2016-07-23 09:15:17 PDT
(In reply to comment #1)
> New flaky crashes (excluding bug #160078):
> 
> webkit.org/b/160119 media/media-fullscreen-inline.html [ Crash Pass ]
> webkit.org/b/160119 media/video-playbackrate.html [ Crash Pass ]

Well, more than this. There's enough that I think it's simplest to use just the TestExpectations file (and separate bugs) to track all of these, rather than mentioning all bugs here. Let's just leave this bug open until the last tests against it are removed from the TestExpectations file.
Comment 3 Michael Catanzaro 2016-07-23 14:42:20 PDT
Some of the tests that crash flakily with X window system errors have crashed only once ever. For example:

imported/blink/transforms/inline-transform-and-clipping-roots.html
media/broken-video.html
media/track/audio/audio-track-mkv-vorbis-enabled.html
media/track/track-css-matching-timestamps.html

Most likely something somewhere is triggering X window system errors and these tests just got unlucky.
Comment 4 Michael Catanzaro 2016-09-07 08:24:49 PDT
I removed most of the tests that were flaky from the expectations for this bug, leaving only the tests that consistently fail or the tests that crash or time out. We now have roughly 65 tests marked against this bug. This is not so bad. Some of these are probably even fixed already, and we just need to check.

One concerning exception are the HTML5 media tests, which account for 23 of the broken tests. This is worth investigating.
Comment 5 Charlie Turner 2017-07-11 10:11:14 PDT
Created attachment 315121 [details]
Patch
Comment 6 WebKit Commit Bot 2017-07-11 12:25:57 PDT
Comment on attachment 315121 [details]
Patch

Clearing flags on attachment: 315121

Committed r219353: <http://trac.webkit.org/changeset/219353>
Comment 7 WebKit Commit Bot 2017-07-11 12:25:58 PDT
All reviewed patches have been landed.  Closing bug.
Comment 8 Charlie Turner 2017-07-11 12:40:36 PDT
Was a gardening commit, reopening.