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

Kent Tamura

Reported 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.

Attachments
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
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Kent Tamura

Comment 1 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

Kent Tamura

Comment 2 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-

Kent Tamura

Comment 3 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

Stephen Chenney

Comment 4 2012-05-30 06:34:29 PDT

Committed r118917: <http://trac.webkit.org/changeset/118917>

Stephen Chenney

Comment 5 2012-05-30 06:35:10 PDT

I broadened the expectation to include timeout on all platforms.

Dominic Cooney

Comment 6 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>

Ami Fischman

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

Created attachment 155671 [details] Patch

WebKit Review Bot

Comment 8 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>

WebKit Review Bot

Comment 9 2012-07-31 19:44:16 PDT

All reviewed patches have been landed. Closing bug.

Peter Kasting

Comment 10 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.

Ami Fischman

Comment 11 2012-08-03 15:08:06 PDT

Created attachment 156467 [details] Patch

WebKit Review Bot

Comment 12 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>

WebKit Review Bot

Comment 13 2012-08-03 20:13:58 PDT

All reviewed patches have been landed. Closing bug.

Note You need to log in before you can comment on or make changes to this bug.