Bug 131534

Summary: [GTK] Layout Test media/media-ended.html is flaky
Product: WebKit Reporter: Enrique Ocaña <eocanha>
Component: Tools / TestsAssignee: Ms2ger (he/him; ⌚ UTC+1/+2) <Ms2ger>
Status: RESOLVED FIXED    
Severity: Normal CC: bugs-noreply, calvaris, clopez, commit-queue, mcatanzaro, Ms2ger, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Description Enrique Ocaña 2014-04-11 04:10:19 PDT
The following layout test is flaky on GTK Linux 64-bit Release

media/media-ended.html [ Timeout Pass ]

The flakiness ins't linked to any particular commit:

http://webkit-test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=media%2Fmedia-ended.html&showExpectations=true&showLargeExpectations=true
Comment 1 Ms2ger (he/him; ⌚ UTC+1/+2) 2017-08-31 08:00:23 PDT
Created attachment 319461 [details]
Patch
Comment 2 Carlos Alberto Lopez Perez 2017-08-31 12:39:48 PDT
Comment on attachment 319461 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=319461&action=review

> LayoutTests/media/media-ended.html:17
> -                audio.src = findMediaFile("audio", "content/silence")
> +                audio.src = findMediaFile("audio", "content/short")

good catch, but "short" is really short.. only 27 msecs! Not sure if is right to use such short audio file here.

The main reason why this test was timing out on GTK bot, but not on other port bots was because our aggressive time out values.
We raised that on r218270 (and is still lower than on Mac ports).

I have checked with wktesthunter that this test is now passing always since r218270.
So I think we can just remove the flaky expectation and close this bug (leaving the test as it was originally wrote with the silence audio file).
Comment 3 Ms2ger (he/him; ⌚ UTC+1/+2) 2017-09-01 01:26:42 PDT
Created attachment 319585 [details]
Patch
Comment 4 Michael Catanzaro 2017-09-01 04:19:35 PDT
Comment on attachment 319585 [details]
Patch

Good catch Carlos!
Comment 5 WebKit Commit Bot 2017-09-01 04:47:48 PDT
Comment on attachment 319585 [details]
Patch

Clearing flags on attachment: 319585

Committed r221476: <http://trac.webkit.org/changeset/221476>
Comment 6 WebKit Commit Bot 2017-09-01 04:47:50 PDT
All reviewed patches have been landed.  Closing bug.
Comment 7 Radar WebKit Bug Importer 2017-09-27 12:52:32 PDT
<rdar://problem/34694188>