Bug 87064 - [Chromium] Re-enable some layout tests that now pass reliably
Summary: [Chromium] Re-enable some layout tests that now pass reliably
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-21 18:22 PDT by Pete Williamson
Modified: 2012-05-24 19:18 PDT (History)
3 users (show)

See Also:

Attachments
Patch (2.66 KB, patch)
2012-05-24 16:47 PDT, Pete Williamson 		no flags Details | Formatted Diff | Diff
Patch (2.52 KB, patch)
2012-05-24 17:21 PDT, Pete Williamson 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) 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 Pete Williamson 2012-05-21 18:22:34 PDT 
I noticed that the following tests which are commented out appear to succeed now.  We should take them out of the exceptions file (test_expectatsions.txt) so they can protect us.
    a. fast/files/revoke-blob-url.html
    b. fast/files/worker/worker-apply-blob-url-to=xhr.html
    c. fast/workers/stress-js-execution.html
I also checked the flaky tests list and noticed that revoke-blob-url.html is not listed (the comments in the exception file note that it was once flaky)

Related Chromium bugs:  79540, 80058, 64129
Comment 1 Pete Williamson 2012-05-24 16:47:54 PDT 
Created attachment 143926 [details]
Patch
Comment 2 Pete Williamson 2012-05-24 17:21:18 PDT 
Created attachment 143931 [details]
Patch
Comment 3 Dmitry Titov 2012-05-24 17:28:21 PDT 
Comment on attachment 143931 [details]
Patch

r+ and cq+ (previous patch had merge conflict in test_expectations.txt)
Comment 4 WebKit Review Bot 2012-05-24 19:18:35 PDT 
Comment on attachment 143931 [details]
Patch

Clearing flags on attachment: 143931

Committed r118463: <http://trac.webkit.org/changeset/118463>
Comment 5 WebKit Review Bot 2012-05-24 19:18:39 PDT 
All reviewed patches have been landed.  Closing bug.