WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
117535
[Qt] Set windows key-code for multimedia keys
https://bugs.webkit.org/show_bug.cgi?id=117535
Summary
[Qt] Set windows key-code for multimedia keys
Allan Sandfeld Jensen
Reported
2013-06-12 03:27:08 PDT
We do not currently set the windows keycode for generated key events for multimedia keys. Some of them do have well-defined values though.
Attachments
Patch
(2.53 KB, patch)
2013-06-12 03:28 PDT
,
Allan Sandfeld Jensen
jturcotte
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Allan Sandfeld Jensen
Comment 1
2013-06-12 03:28:49 PDT
Created
attachment 204423
[details]
Patch
Jocelyn Turcotte
Comment 2
2013-06-12 03:48:03 PDT
Comment on
attachment 204423
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=204423&action=review
> Source/WebCore/platform/qt/PlatformKeyboardEventQt.cpp:439 > // VK_VOLUME_MUTE (AD) Windows 2000/XP: Volume Mute key > + case Qt::Key_VolumeMute: > + return VK_VOLUME_MUTE;
Part of the file moves the comment after the value and remove the name in the comment. return VK_VOLUME_MUTE; // (AD) Windows 2000/XP: Volume Mute key I think it would be easier to read (slightly). r=me for the rest.
Allan Sandfeld Jensen
Comment 3
2013-06-12 04:44:48 PDT
Committed
r151494
: <
http://trac.webkit.org/changeset/151494
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug