Summary: | fast/events/suspend-timers.html is failing in Mac WebKit2, even though LayoutTestController.overridePreference is implemented in WebKitTestRunner | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | mitz | ||||||
Component: | Tools / Tests | Assignee: | zalan <zalan> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | jer.noble, sam, slewis, webkit-bug-importer, webkit.review.bot, zalan | ||||||
Priority: | P2 | Keywords: | InRadar, LayoutTestFailure | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
mitz
2012-03-22 12:01:40 PDT
Added the current incorrect results as Mac WebKit2 expected results in <http://trac.webkit.org/r111741>. The failing results are inconsistent, so I am going to skip the test in Mac WebKit2. Added to the Mac WebKit2 skip list in <http://trac.webkit.org/r111761>. Created attachment 137208 [details]
Patch
Alternatively, CacheModel can be set to PrimaryWebBrowser, only when the PageCache's preference key is also set to enabled. Though, I believe it's better to run WebKitTestRunner with PrimaryWebBrowser setting instead of that DocumentViewer. *** Bug 81637 has been marked as a duplicate of this bug. *** Comment on attachment 137208 [details]
Patch
It looks like DumpRenderTree (on Mac, at least) uses WebCacheModelDocumentBrowser. Shouldn’t WebKitTestRunner use the same by default?
(In reply to comment #8) > (From update of attachment 137208 [details]) > It looks like DumpRenderTree (on Mac, at least) uses WebCacheModelDocumentBrowser. Shouldn’t WebKitTestRunner use the same by default? The only reason I picked PrimaryWebBrowser was because it guarantees capacity > 0. However for compatibility reasons, i guess DocumentBrowser is a better pick. Created attachment 137213 [details]
Patch
Comment on attachment 137213 [details] Patch Clearing flags on attachment: 137213 Committed r114206: <http://trac.webkit.org/changeset/114206> All reviewed patches have been landed. Closing bug. This commit broke the LayoutTests/fast/harness/use-page-cache.html test on Lion and Snow Leopard. See: http://build.webkit.org/results/Lion%20Debug%20(WebKit2%20Tests)/r114298%20(6058)/results.html Filed <https://bugs.webkit.org/show_bug.cgi?id=84206> to track the test failure. |