Bug 85952

Summary:

[Chromium] Layout Test http/tests/media/media-source/webm/video-media-source-*.html is flaky

Product:

WebKit

Reporter:

Kent Tamura <tkent>

Component:

Tools / Tests

Assignee:

Ami Fischman <fischman>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

acolwell, dominicc, eric.carlson, feature-media-reviews, fischman, pkasting, schenney, webkit.review.bot

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none
Patch	none

Description Kent Tamura 2012-05-08 20:36:00 PDT

The following layout test is flaky on mac,win

http/tests/media/media-source/webm/video-media-source-play.html

Probable cause:

Unknown.

Comment 1 Kent Tamura 2012-05-09 00:13:04 PDT

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=http%2Ftests%2Fmedia%2Fmedia-source%2Fwebm%2Fvideo-media-source-play.html

Comment 2 Kent Tamura 2012-05-09 02:55:20 PDT

I found they were not slow, they sometimes timed out.
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=http%2Ftests%2Fmedia%2Fmedia-source%2Fwebm%2Fvideo-media-

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:

<http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=http%2Ftests%2Fmedia%2Fmedia-source%2Fvideo-media-source-seek.html>

Comment 7 Ami Fischman 2012-07-31 16:29:16 PDT

Created attachment 155671 [details]
Patch

Comment 8 WebKit Review Bot 2012-07-31 19:44:11 PDT

Comment on attachment 155671 [details]
Patch

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]
Patch

Comment 12 WebKit Review Bot 2012-08-03 20:13:54 PDT

Comment on attachment 156467 [details]
Patch

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.