Summary: | [Chromium] media/video-delay-load-event.html is flaky on all platforms | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Andrey Kosyakov <caseq> | ||||
Component: | UI Events | Assignee: | Ami Fischman <fischman> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | eric.carlson, fischman, imasaki, webkit.review.bot, yutak | ||||
Priority: | P2 | Keywords: | LayoutTestFailure | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Andrey Kosyakov
2011-07-06 09:45:50 PDT
I am looking at the change list around this http://trac.webkit.org/log/?verbose=on&rev=90242&stop_rev=90233 But cannot find any specific change relating to this failure. This test is also flaky on chromium-win and chromium-linux (less flaky than chromium-mac, but still flaky). I'm going to update test expectation as such. Created attachment 135218 [details]
Patch
The attached patch reduces FAIL rate on my linux box from 8% to none, and is how I read the spec to imply this should be done. However I'm not sure what part of the spec describes the document-onload delay-until-media-metadata is available (the point of this test) so I'd like to get confirmation that this change doesn't trample on (part of) the point of the test. Comment on attachment 135218 [details]
Patch
Nice fix Ami, thanks!
Comment on attachment 135218 [details] Patch Clearing flags on attachment: 135218 Committed r113057: <http://trac.webkit.org/changeset/113057> All reviewed patches have been landed. Closing bug. |