Bug 87657 - [chromium] http/tests/cache/cancel-in-progress-load.html is flaky
Summary: [chromium] http/tests/cache/cancel-in-progress-load.html is flaky
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Marcus Bulach
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-28 06:38 PDT by Marcus Bulach
Modified: 2012-09-04 15:23 PDT (History)
1 user (show)

See Also:

Attachments
Patch (1.36 KB, patch)
2012-05-28 06:42 PDT, Marcus Bulach 		no flags Details | Formatted Diff | Diff
Patch (1.43 KB, patch)
2012-05-28 06:48 PDT, Marcus Bulach 		no flags Details | Formatted Diff | Diff
Patch (1.40 KB, patch)
2012-05-28 08:07 PDT, Marcus Bulach 		no flags Details | Formatted Diff | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Marcus Bulach 2012-05-28 06:38:11 PDT 
Added by r118618

This test has been flaky. At some point, it dumps the render tree, even if it's just supposed to dumpAsText().
Nate, can you please take a look?

I'll mark the test temporarily as failing.
Comment 1 Marcus Bulach 2012-05-28 06:42:33 PDT 
Created attachment 144348 [details]
Patch
Comment 2 Marcus Bulach 2012-05-28 06:48:21 PDT 
Created attachment 144349 [details]
Patch
Comment 3 Marcus Bulach 2012-05-28 06:49:50 PDT 
Committed r118680: <http://trac.webkit.org/changeset/118680>
Comment 4 Marcus Bulach 2012-05-28 08:07:15 PDT 
Created attachment 144362 [details]
Patch
Comment 5 Marcus Bulach 2012-05-28 08:08:08 PDT 
Committed r118683: <http://trac.webkit.org/changeset/118683>
Comment 6 Marcus Bulach 2012-05-28 08:10:14 PDT 
Sorry, I should clarify. :)

The test sometimes pass, but sometimes it looks like "dumpAsText()" is not being called, so that drt also expects an image and fails with a "MISSING".
I added "PASS TEXT IMAGE" on the latest patch.
Comment 7 Marcus Bulach 2012-05-28 09:41:41 PDT 
Committed r118691: <http://trac.webkit.org/changeset/118691>
Comment 8 Nate Chapin 2012-09-04 15:23:05 PDT 
The offending test was rolled out a while ago: http://trac.webkit.org/changeset/119918