Summary: | [WK2] New fast/events/tab-focus-link-in-canvas fails from r126908 | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Szilard Ledan <szledan> | ||||||
Component: | Tools / Tests | Assignee: | Chris Dumez <cdumez> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | cfleizach, dmazzoni, gyuyoung.kim, kadam, kenneth, kkristof, mrobinson, ossy, szledan, thorton, webkit.review.bot | ||||||
Priority: | P2 | Keywords: | Qt, QtTriaged | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | 77884 | ||||||||
Bug Blocks: | 87008, 94967 | ||||||||
Attachments: |
|
Description
Szilard Ledan
2012-08-29 05:15:19 PDT
It isn't timeout, it is a simple fail. And it isn't Qt, but WK2 specific bug. I skipped this test. http://trac.webkit.org/changeset/126995 The underlying issue here is that WebKitTabToLinksPreferenceKey isn't supported in WK2. InjectedBundle::overrideBoolPreferenceForTestRunner has this comment: // FIXME: Need an explicit way to set "WebKitTabToLinksPreferenceKey" directly in WebPage. Also, this bug seems to be essentially the same issue: https://bugs.webkit.org/show_bug.cgi?id=77884 It should be easy to enable this test once that's solved. Created attachment 162951 [details]
Patch
Add support for overriding "WebKitTabToLinksPreferenceKey" preference in WebKitTestRunner. This is needed for fast/events/tab-focus-link-in-canvas to pass since the setting is disabled by default in WK2.
Created attachment 162952 [details]
Patch
Minor changelog fix.
Comment on attachment 162952 [details] Patch Clearing flags on attachment: 162952 Committed r127966: <http://trac.webkit.org/changeset/127966> All reviewed patches have been landed. Closing bug. Oddly, there was already a "tabsToLinks" WebKit2 pref, and you added "tabToLinks", and now we have both... but I'm not 100% sure how to untangle. |