Summary: | [Qt] Fix the EventSender::keyDown() implementation | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jakub Wieczorek <jwieczorek> | ||||||||
Component: | Platform | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | commit-queue | ||||||||
Priority: | P2 | Keywords: | Qt | ||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Other | ||||||||||
OS: | OS X 10.5 | ||||||||||
Attachments: |
|
Description
Jakub Wieczorek
2009-10-03 07:16:37 PDT
Created attachment 40571 [details]
patch
Created attachment 40572 [details]
patch
Add the missing ChangeLog.
Created attachment 40574 [details]
patch
Fixed the style.
Comment on attachment 40574 [details]
patch
Please list what layout tests this affects. If it doesn't affect any, please explain why, or add one which it affects. :)
(In reply to comment #4) > (From update of attachment 40574 [details]) > Please list what layout tests this affects. If it doesn't affect any, please > explain why, or add one which it affects. :) Similarly to https://bugs.webkit.org/show_bug.cgi?id=30044, it also addresses the fast/events/keydown-function-keys.html failure. I'm not 100% sure if it fixed any other tests but it doesn't seem so. Later on, I will be going through the tests that use it and that are skipped currently to see if there are any that are fine now. Comment on attachment 40574 [details]
patch
Thanks!
Comment on attachment 40574 [details] patch Clearing flags on attachment: 40574 Committed r49166: <http://trac.webkit.org/changeset/49166> All reviewed patches have been landed. Closing bug. |