Bug 33250 – [Qt] windowsKeyCodeForKeyEvent() returns 0 for arrow keys on numeric-pad

• Status: RESOLVED
• Resolution: FIXED
• Component: Platform

[Kinuko Yasuda]

Currently QT returns 0 as windowsVirtualKeyCode for any key events other than 0-9 and +-*/ keys on numeric-pad.
This indirectly makes a layout test fast/tests/keydown-numpad-keys.html failing.

[Kinuko Yasuda]

Created attachment 45957 [details]
patch 1

[Eric Seidel (no email)]

Comment on attachment 45957 [details]
patch 1

Looks OK.

[WebKit Commit Bot]

Comment on attachment 45957 [details]
patch 1

Rejecting patch 45957 from commit-queue.

Failed to run "['/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', '--reviewer', 'Eric Seidel', '--force']" exit_code: 1
Last 500 characters of output:
ile WebCore/platform/qt/PlatformKeyboardEventQt.cpp
patching file WebKitTools/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file WebKitTools/DumpRenderTree/qt/EventSenderQt.cpp
Hunk #1 FAILED at 38.
Hunk #2 succeeded at 135 (offset 19 lines).
Hunk #3 succeeded at 150 (offset 19 lines).
1 out of 3 hunks FAILED -- saving rejects to file WebKitTools/DumpRenderTree/qt/EventSenderQt.cpp.rej
patching file WebKitTools/DumpRenderTree/qt/EventSenderQt.h
Hunk #1 succeeded at 56 (offset 2 lines).

Full output: http://webkit-commit-queue.appspot.com/results/165156

[Eric Seidel (no email)]

Sorry.  The commit-queue was blocked all day by a very red tree.  Looks like someone else checked in in the meanwhile and made your patch not apply.  Normally it doesn't take more than 15 minutes between when we set cq+ and when your patch is landed.

You'll need to either get someone to land this manually, or post an updated patch so that we can r+/cq+ it.

Thanks!

[Kent Tamura]

Landed manually: http://trac.webkit.org/changeset/52902