Bug 85952 - [Chromium] Layout Test http/tests/media/media-source/webm/video-media-source-*.html is flaky
Summary: [Chromium] Layout Test http/tests/media/media-source/webm/video-media-source-...
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ami Fischman
Depends on:
Reported: 2012-05-08 20:36 PDT by Kent Tamura
Modified: 2012-08-03 20:13 PDT (History)
8 users (show)

See Also:

Patch (3.22 KB, patch)
2012-07-31 16:29 PDT, Ami Fischman
no flags Details | Formatted Diff | Diff
Patch (1.50 KB, patch)
2012-08-03 15:08 PDT, Ami Fischman
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kent Tamura 2012-05-08 20:36:00 PDT
The following layout test is flaky on mac,win


Probable cause:

Comment 2 Kent Tamura 2012-05-09 02:55:20 PDT
I found they were not slow, they sometimes timed out.
Comment 3 Kent Tamura 2012-05-09 03:01:20 PDT
I updated test_expectations.txt:

BUGWK85952 MAC WIN : http/tests/media/media-source/webm/video-media-source-play.html = PASS TIMEOUT
BUGWK85952 WIN : http/tests/media/media-source/webm/video-media-source-seek.html = PASS TIMEOUT
BUGWK85952 WIN : http/tests/media/media-source/webm/video-media-source-state-changes.html = PASS TIMEOUT
BUGWK85952 MAC DEBUG : http/tests/media/media-source/webm/video-media-source-state-changes.html = PASS TEXT
Comment 4 Stephen Chenney 2012-05-30 06:34:29 PDT
Committed r118917: <http://trac.webkit.org/changeset/118917>
Comment 5 Stephen Chenney 2012-05-30 06:35:10 PDT
I broadened the expectation to include timeout on all platforms.
Comment 6 Dominic Cooney 2012-06-12 21:48:15 PDT
I updated the expectations for video-media-source-seek.html to include TEXT in r120157 because the  EVENT(webkitsourceended) message races with the EVENT(seeked) event causing text diffs.

This mostly seems to be happening on Linux Debug:

Comment 7 Ami Fischman 2012-07-31 16:29:16 PDT
Created attachment 155671 [details]
Comment 8 WebKit Review Bot 2012-07-31 19:44:11 PDT
Comment on attachment 155671 [details]

Clearing flags on attachment: 155671

Committed r124286: <http://trac.webkit.org/changeset/124286>
Comment 9 WebKit Review Bot 2012-07-31 19:44:16 PDT
All reviewed patches have been landed.  Closing bug.
Comment 10 Peter Kasting 2012-07-31 21:29:13 PDT
Reopening.  http/tests/media/media-source/video-media-source-play.html still times out on most bots.  Per fischman I'm going to revert this line of the test expectations and left the rest of the patch in place.
Comment 11 Ami Fischman 2012-08-03 15:08:06 PDT
Created attachment 156467 [details]
Comment 12 WebKit Review Bot 2012-08-03 20:13:54 PDT
Comment on attachment 156467 [details]

Clearing flags on attachment: 156467

Committed r124687: <http://trac.webkit.org/changeset/124687>
Comment 13 WebKit Review Bot 2012-08-03 20:13:58 PDT
All reviewed patches have been landed.  Closing bug.