Bug 83391

Summary: Layout Test http/tests/media/media-document.html is flaky
Product: WebKit Reporter: James Simonsen <simonjam>
Component: Tools / TestsAssignee: Ami Fischman <fischman>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, fischman, jer.noble, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

James Simonsen
Reported 2012-04-06 14:35:28 PDT
The following layout test is flaky on Chromium debug builds. http/tests/media/media-document.html Probable cause: Seemed to start happening when change 113397 went in. Perhaps there was some bizarre dependency on test order, and now this one is racy. I'll suppress it for now.
Attachments
Patch (1.23 KB, patch)
2012-04-06 17:53 PDT, Ami Fischman 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Ami Fischman
Comment 1 2012-04-06 14:50:00 PDT
(In reply to comment #0) > The following layout test is flaky on Chromium debug builds. > http/tests/media/media-document.html > Probable cause: > Seemed to start happening when change 113397 went in. Perhaps there was some bizarre dependency on test order, and now this one is racy. My guess is that this is another case covered by bug 82505: chromium's DRT used to mask (D)CHECK-failures but doesn't anymore. I fixed the RequestProxy::~RequestProxy() crash and the chromium-deps roll in bug 83389 should pick it up.
Ami Fischman
Comment 2 2012-04-06 17:53:43 PDT
Created attachment 136108 [details] Patch
WebKit Review Bot
Comment 3 2012-04-09 03:02:27 PDT
Comment on attachment 136108 [details] Patch Clearing flags on attachment: 136108 Committed r113567: <http://trac.webkit.org/changeset/113567>
WebKit Review Bot
Comment 4 2012-04-09 03:02:31 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.