<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[210504] trunk</title>
</head>
<body>
<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; }
#msg dl a { font-weight: bold}
#msg dl a:link { color:#fc3; }
#msg dl a:active { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/210504">210504</a></dd>
<dt>Author</dt> <dd>gns@gnome.org</dd>
<dt>Date</dt> <dd>2017-01-09 06:14:57 -0800 (Mon, 09 Jan 2017)</dd>
</dl>
<h3>Log Message</h3>
<pre>[GTK] Should support key and code properties on keyboard events
https://bugs.webkit.org/show_bug.cgi?id=166759
Reviewed by Carlos Garcia Campos.
Source/WebCore:
This change fixes a bunch of tests which are currently failing, such as:
- fast/events/arrow-keys-on-body.html
- fast/events/constructors/keyboard-event-constructor.html
- fast/events/key-events-in-input-button.html
- fast/events/key-events-in-input-text.html
- fast/events/keyboardevent-code.html
- fast/events/keyboardevent-key.html
* platform/PlatformKeyboardEvent.h:
* platform/gtk/PlatformKeyboardEventGtk.cpp:
(WebCore::PlatformKeyboardEvent::keyValueForGdkKeyCode): match web key value
to GDK key symbol.
(WebCore::PlatformKeyboardEvent::keyCodeForHardwareKeyCode): match web key code
to hardware code as reported by GDK.
(WebCore::PlatformKeyboardEvent::PlatformKeyboardEvent): add web key value and
code to the keyboard event upon creation.
Source/WebKit2:
* Shared/WebEvent.h:
* Shared/WebKeyboardEvent.cpp:
(WebKit::WebKeyboardEvent::WebKeyboardEvent): accept web key value and code on construction.
* Shared/gtk/WebEventFactory.cpp:
(WebKit::WebEventFactory::createWebKeyboardEvent): pass web key value and code on construction.
Source/WTF:
* wtf/FeatureDefines.h: enable key and code properties support for GTK.
Tools:
* WebKitTestRunner/gtk/EventSenderProxyGtk.cpp:
(WTR::getGDKKeySymForKeyRef): add missing modifier keys which were being reported as 'l' and 'r' only
on tests.
LayoutTests:
* platform/gtk/fast/events/keyboardevent-code-expected.txt: Added. Hardware
codes are different for Mac and Linux for a couple of keys, so we need
platform-specific results.</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkLayoutTestsChangeLog">trunk/LayoutTests/ChangeLog</a></li>
<li><a href="#trunkSourceWTFChangeLog">trunk/Source/WTF/ChangeLog</a></li>
<li><a href="#trunkSourceWTFwtfFeatureDefinesh">trunk/Source/WTF/wtf/FeatureDefines.h</a></li>
<li><a href="#trunkSourceWebCoreChangeLog">trunk/Source/WebCore/ChangeLog</a></li>
<li><a href="#trunkSourceWebCoreplatformPlatformKeyboardEventh">trunk/Source/WebCore/platform/PlatformKeyboardEvent.h</a></li>
<li><a href="#trunkSourceWebCoreplatformgtkPlatformKeyboardEventGtkcpp">trunk/Source/WebCore/platform/gtk/PlatformKeyboardEventGtk.cpp</a></li>
<li><a href="#trunkSourceWebKit2ChangeLog">trunk/Source/WebKit2/ChangeLog</a></li>
<li><a href="#trunkSourceWebKit2SharedWebEventh">trunk/Source/WebKit2/Shared/WebEvent.h</a></li>
<li><a href="#trunkSourceWebKit2SharedWebKeyboardEventcpp">trunk/Source/WebKit2/Shared/WebKeyboardEvent.cpp</a></li>
<li><a href="#trunkSourceWebKit2SharedgtkWebEventFactorycpp">trunk/Source/WebKit2/Shared/gtk/WebEventFactory.cpp</a></li>
<li><a href="#trunkToolsChangeLog">trunk/Tools/ChangeLog</a></li>
<li><a href="#trunkToolsWebKitTestRunnergtkEventSenderProxyGtkcpp">trunk/Tools/WebKitTestRunner/gtk/EventSenderProxyGtk.cpp</a></li>
</ul>
<h3>Added Paths</h3>
<ul>
<li><a href="#trunkLayoutTestsplatformgtkfasteventskeyboardeventcodeexpectedtxt">trunk/LayoutTests/platform/gtk/fast/events/keyboardevent-code-expected.txt</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkLayoutTestsChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/ChangeLog (210503 => 210504)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/ChangeLog 2017-01-09 12:46:34 UTC (rev 210503)
+++ trunk/LayoutTests/ChangeLog 2017-01-09 14:14:57 UTC (rev 210504)
</span><span class="lines">@@ -1,3 +1,14 @@
</span><ins>+2017-01-06 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>
+
+ [GTK] Should support key and code properties on keyboard events
+ https://bugs.webkit.org/show_bug.cgi?id=166759
+
+ Reviewed by Carlos Garcia Campos.
+
+ * platform/gtk/fast/events/keyboardevent-code-expected.txt: Added. Hardware
+ codes are different for Mac and Linux for a couple of keys, so we need
+ platform-specific results.
+
</ins><span class="cx"> 2017-01-09 Alejandro G. Castro <alex@igalia.com>
</span><span class="cx">
</span><span class="cx"> [OWR] Unskip fast/mediastream/MediaStream-video-element-track-stop.html
</span></span></pre></div>
<a id="trunkLayoutTestsplatformgtkfasteventskeyboardeventcodeexpectedtxt"></a>
<div class="addfile"><h4>Added: trunk/LayoutTests/platform/gtk/fast/events/keyboardevent-code-expected.txt (0 => 210504)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/platform/gtk/fast/events/keyboardevent-code-expected.txt (rev 0)
+++ trunk/LayoutTests/platform/gtk/fast/events/keyboardevent-code-expected.txt 2017-01-09 14:14:57 UTC (rev 210504)
</span><span class="lines">@@ -0,0 +1,190 @@
</span><ins>+Test the code attribute on KeyboardEvent.
+
+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
+
+
+
+* keydown
+PASS keyboardEvent.code is "Space"
+* keyup
+PASS keyboardEvent.code is "Space"
+
+* keydown
+PASS keyboardEvent.code is "KeyA"
+* keyup
+PASS keyboardEvent.code is "KeyA"
+
+* keydown
+PASS keyboardEvent.code is "Backspace"
+* keyup
+PASS keyboardEvent.code is "Backspace"
+
+* keydown
+FAIL keyboardEvent.code should be Minus. Was Escape.
+* keyup
+FAIL keyboardEvent.code should be Minus. Was Escape.
+
+* keydown
+PASS keyboardEvent.code is "ArrowLeft"
+* keyup
+PASS keyboardEvent.code is "ArrowLeft"
+
+* keydown
+PASS keyboardEvent.code is "ArrowRight"
+* keyup
+PASS keyboardEvent.code is "ArrowRight"
+
+* keydown
+PASS keyboardEvent.code is "ArrowUp"
+* keyup
+PASS keyboardEvent.code is "ArrowUp"
+
+* keydown
+PASS keyboardEvent.code is "ArrowDown"
+* keyup
+PASS keyboardEvent.code is "ArrowDown"
+
+* keydown
+PASS keyboardEvent.code is "PageUp"
+* keyup
+PASS keyboardEvent.code is "PageUp"
+
+* keydown
+PASS keyboardEvent.code is "PageDown"
+* keyup
+PASS keyboardEvent.code is "PageDown"
+
+* keydown
+PASS keyboardEvent.code is "Home"
+* keyup
+PASS keyboardEvent.code is "Home"
+
+* keydown
+PASS keyboardEvent.code is "End"
+* keyup
+PASS keyboardEvent.code is "End"
+
+* keydown
+PASS keyboardEvent.code is "Delete"
+* keyup
+PASS keyboardEvent.code is "Delete"
+
+* keydown
+PASS keyboardEvent.code is "ControlLeft"
+* keyup
+PASS keyboardEvent.code is "ControlLeft"
+
+* keydown
+PASS keyboardEvent.code is "ShiftLeft"
+* keyup
+PASS keyboardEvent.code is "ShiftLeft"
+
+* keydown
+PASS keyboardEvent.code is "AltLeft"
+* keyup
+PASS keyboardEvent.code is "AltLeft"
+
+* keydown
+PASS keyboardEvent.code is "ControlRight"
+* keyup
+PASS keyboardEvent.code is "ControlRight"
+
+* keydown
+PASS keyboardEvent.code is "ShiftRight"
+* keyup
+PASS keyboardEvent.code is "ShiftRight"
+
+* keydown
+PASS keyboardEvent.code is "AltRight"
+* keyup
+PASS keyboardEvent.code is "AltRight"
+
+* keydown
+PASS keyboardEvent.code is "F1"
+* keyup
+PASS keyboardEvent.code is "F1"
+
+* keydown
+PASS keyboardEvent.code is "F2"
+* keyup
+PASS keyboardEvent.code is "F2"
+
+* keydown
+PASS keyboardEvent.code is "F3"
+* keyup
+PASS keyboardEvent.code is "F3"
+
+* keydown
+PASS keyboardEvent.code is "F4"
+* keyup
+PASS keyboardEvent.code is "F4"
+
+* keydown
+PASS keyboardEvent.code is "F5"
+* keyup
+PASS keyboardEvent.code is "F5"
+
+* keydown
+PASS keyboardEvent.code is "F6"
+* keyup
+PASS keyboardEvent.code is "F6"
+
+* keydown
+PASS keyboardEvent.code is "F7"
+* keyup
+PASS keyboardEvent.code is "F7"
+
+* keydown
+PASS keyboardEvent.code is "F8"
+* keyup
+PASS keyboardEvent.code is "F8"
+
+* keydown
+PASS keyboardEvent.code is "F9"
+* keyup
+PASS keyboardEvent.code is "F9"
+
+* keydown
+PASS keyboardEvent.code is "F10"
+* keyup
+PASS keyboardEvent.code is "F10"
+
+* keydown
+PASS keyboardEvent.code is "F11"
+* keyup
+PASS keyboardEvent.code is "F11"
+
+* keydown
+PASS keyboardEvent.code is "F12"
+* keyup
+PASS keyboardEvent.code is "F12"
+
+* keydown
+FAIL keyboardEvent.code should be NumpadEnter. Was Enter.
+* keyup
+FAIL keyboardEvent.code should be NumpadEnter. Was Enter.
+
+* keydown
+PASS keyboardEvent.code is "KeyA"
+* keyup
+PASS keyboardEvent.code is "KeyA"
+
+* keydown
+PASS keyboardEvent.code is "KeyA"
+* keyup
+PASS keyboardEvent.code is "KeyA"
+
+* keydown
+PASS keyboardEvent.code is "KeyA"
+* keyup
+PASS keyboardEvent.code is "KeyA"
+
+* keydown
+PASS keyboardEvent.code is "Digit1"
+* keyup
+PASS keyboardEvent.code is "Digit1"
+
+PASS successfullyParsed is true
+
+TEST COMPLETE
+
</ins></span></pre></div>
<a id="trunkSourceWTFChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WTF/ChangeLog (210503 => 210504)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WTF/ChangeLog 2017-01-09 12:46:34 UTC (rev 210503)
+++ trunk/Source/WTF/ChangeLog 2017-01-09 14:14:57 UTC (rev 210504)
</span><span class="lines">@@ -1,3 +1,12 @@
</span><ins>+2017-01-06 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>
+
+ [GTK] Should support key and code properties on keyboard events
+ https://bugs.webkit.org/show_bug.cgi?id=166759
+
+ Reviewed by Carlos Garcia Campos.
+
+ * wtf/FeatureDefines.h: enable key and code properties support for GTK.
+
</ins><span class="cx"> 2017-01-08 Antti Koivisto <antti@apple.com>
</span><span class="cx">
</span><span class="cx"> Move cache coders to WTF
</span></span></pre></div>
<a id="trunkSourceWTFwtfFeatureDefinesh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WTF/wtf/FeatureDefines.h (210503 => 210504)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WTF/wtf/FeatureDefines.h 2017-01-09 12:46:34 UTC (rev 210503)
+++ trunk/Source/WTF/wtf/FeatureDefines.h 2017-01-09 14:14:57 UTC (rev 210504)
</span><span class="lines">@@ -324,6 +324,13 @@
</span><span class="cx">
</span><span class="cx"> /* --------- Gtk port (Unix, Windows, Mac) --------- */
</span><span class="cx"> #if PLATFORM(GTK)
</span><ins>+#if !defined(ENABLE_KEYBOARD_KEY_ATTRIBUTE)
+#define ENABLE_KEYBOARD_KEY_ATTRIBUTE 1
+#endif
+
+#if !defined(ENABLE_KEYBOARD_CODE_ATTRIBUTE)
+#define ENABLE_KEYBOARD_CODE_ATTRIBUTE 1
+#endif
</ins><span class="cx"> #endif /* PLATFORM(GTK) */
</span><span class="cx">
</span><span class="cx"> /* ENABLE macro defaults for WebCore */
</span></span></pre></div>
<a id="trunkSourceWebCoreChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/ChangeLog (210503 => 210504)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/ChangeLog 2017-01-09 12:46:34 UTC (rev 210503)
+++ trunk/Source/WebCore/ChangeLog 2017-01-09 14:14:57 UTC (rev 210504)
</span><span class="lines">@@ -1,3 +1,27 @@
</span><ins>+2017-01-06 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>
+
+ [GTK] Should support key and code properties on keyboard events
+ https://bugs.webkit.org/show_bug.cgi?id=166759
+
+ Reviewed by Carlos Garcia Campos.
+
+ This change fixes a bunch of tests which are currently failing, such as:
+ - fast/events/arrow-keys-on-body.html
+ - fast/events/constructors/keyboard-event-constructor.html
+ - fast/events/key-events-in-input-button.html
+ - fast/events/key-events-in-input-text.html
+ - fast/events/keyboardevent-code.html
+ - fast/events/keyboardevent-key.html
+
+ * platform/PlatformKeyboardEvent.h:
+ * platform/gtk/PlatformKeyboardEventGtk.cpp:
+ (WebCore::PlatformKeyboardEvent::keyValueForGdkKeyCode): match web key value
+ to GDK key symbol.
+ (WebCore::PlatformKeyboardEvent::keyCodeForHardwareKeyCode): match web key code
+ to hardware code as reported by GDK.
+ (WebCore::PlatformKeyboardEvent::PlatformKeyboardEvent): add web key value and
+ code to the keyboard event upon creation.
+
</ins><span class="cx"> 2017-01-09 Carlos Garcia Campos <cgarcia@igalia.com>
</span><span class="cx">
</span><span class="cx"> [GTK] WebProcess from WebKitGtk+ 2.15.2 SIGSEGVs in std::unique_ptr<SoupBuffer, WTF::GPtrDeleter<SoupBuffer> >::get() const () at /usr/include/c++/6/bits/unique_ptr.h:305
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformPlatformKeyboardEventh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/PlatformKeyboardEvent.h (210503 => 210504)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/PlatformKeyboardEvent.h 2017-01-09 12:46:34 UTC (rev 210503)
+++ trunk/Source/WebCore/platform/PlatformKeyboardEvent.h 2017-01-09 14:14:57 UTC (rev 210504)
</span><span class="lines">@@ -161,6 +161,8 @@
</span><span class="cx"> const CompositionResults& compositionResults() const { return m_compositionResults; }
</span><span class="cx">
</span><span class="cx"> // Used by WebKit2
</span><ins>+ static String keyValueForGdkKeyCode(unsigned);
+ static String keyCodeForHardwareKeyCode(unsigned);
</ins><span class="cx"> static String keyIdentifierForGdkKeyCode(unsigned);
</span><span class="cx"> static int windowsKeyCodeForGdkKeyCode(unsigned);
</span><span class="cx"> static String singleCharacterString(unsigned);
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformgtkPlatformKeyboardEventGtkcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/gtk/PlatformKeyboardEventGtk.cpp (210503 => 210504)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/gtk/PlatformKeyboardEventGtk.cpp 2017-01-09 12:46:34 UTC (rev 210503)
+++ trunk/Source/WebCore/platform/gtk/PlatformKeyboardEventGtk.cpp 2017-01-09 14:14:57 UTC (rev 210504)
</span><span class="lines">@@ -34,10 +34,10 @@
</span><span class="cx"> #include "NotImplemented.h"
</span><span class="cx"> #include "TextEncoding.h"
</span><span class="cx"> #include "WindowsKeyboardCodes.h"
</span><del>-
</del><span class="cx"> #include <gdk/gdk.h>
</span><span class="cx"> #include <gdk/gdkkeysyms.h>
</span><span class="cx"> #include <wtf/CurrentTime.h>
</span><ins>+#include <wtf/glib/GUniquePtr.h>
</ins><span class="cx">
</span><span class="cx"> namespace WebCore {
</span><span class="cx">
</span><span class="lines">@@ -44,6 +44,692 @@
</span><span class="cx"> // FIXME: This is incomplete. We should change this to mirror
</span><span class="cx"> // more like what Firefox does, and generate these switch statements
</span><span class="cx"> // at build time.
</span><ins>+// https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent/key/Key_Values
+String PlatformKeyboardEvent::keyValueForGdkKeyCode(unsigned keyCode)
+{
+ switch (keyCode) {
+ // Modifier keys.
+ case GDK_KEY_Alt_L:
+ case GDK_KEY_Alt_R:
+ return ASCIILiteral("Alt");
+ // Firefox uses GDK_KEY_Mode_switch for AltGraph as well.
+ case GDK_KEY_ISO_Level3_Shift:
+ case GDK_KEY_ISO_Level3_Latch:
+ case GDK_KEY_ISO_Level3_Lock:
+ case GDK_KEY_ISO_Level5_Shift:
+ case GDK_KEY_ISO_Level5_Latch:
+ case GDK_KEY_ISO_Level5_Lock:
+ return ASCIILiteral("AltGraph");
+ case GDK_KEY_Caps_Lock:
+ return ASCIILiteral("CapsLock");
+ case GDK_KEY_Control_L:
+ case GDK_KEY_Control_R:
+ return ASCIILiteral("Control");
+ // Fn: This is typically a hardware key that does not generate a separate code.
+ // FnLock.
+ case GDK_KEY_Hyper_L:
+ case GDK_KEY_Hyper_R:
+ return ASCIILiteral("Hyper");
+ case GDK_KEY_Meta_L:
+ case GDK_KEY_Meta_R:
+ return ASCIILiteral("Meta");
+ case GDK_KEY_Num_Lock:
+ return ASCIILiteral("NumLock");
+ case GDK_KEY_Scroll_Lock:
+ return ASCIILiteral("ScrollLock");
+ case GDK_KEY_Shift_L:
+ case GDK_KEY_Shift_R:
+ return ASCIILiteral("Shift");
+ case GDK_KEY_Super_L:
+ case GDK_KEY_Super_R:
+ return ASCIILiteral("Super");
+ // Symbol.
+ // SymbolLock.
+
+ // Whitespace keys.
+ case GDK_KEY_Return:
+ case GDK_KEY_KP_Enter:
+ case GDK_KEY_ISO_Enter:
+ case GDK_KEY_3270_Enter:
+ return ASCIILiteral("Enter");
+ case GDK_KEY_Tab:
+ case GDK_KEY_KP_Tab:
+ return ASCIILiteral("Tab");
+
+ // Navigation keys.
+ case GDK_KEY_Down:
+ case GDK_KEY_KP_Down:
+ return ASCIILiteral("ArrowDown");
+ case GDK_KEY_Left:
+ case GDK_KEY_KP_Left:
+ return ASCIILiteral("ArrowLeft");
+ case GDK_KEY_Right:
+ case GDK_KEY_KP_Right:
+ return ASCIILiteral("ArrowRight");
+ case GDK_KEY_Up:
+ case GDK_KEY_KP_Up:
+ return ASCIILiteral("ArrowUp");
+ case GDK_KEY_End:
+ case GDK_KEY_KP_End:
+ return ASCIILiteral("End");
+ case GDK_KEY_Home:
+ case GDK_KEY_KP_Home:
+ return ASCIILiteral("Home");
+ case GDK_KEY_Page_Down:
+ case GDK_KEY_KP_Page_Down:
+ return ASCIILiteral("PageDown");
+ case GDK_KEY_Page_Up:
+ case GDK_KEY_KP_Page_Up:
+ return ASCIILiteral("PageUp");
+
+ // Editing keys.
+ case GDK_KEY_BackSpace:
+ return ASCIILiteral("Backspace");
+ case GDK_KEY_Clear:
+ return ASCIILiteral("Clear");
+ case GDK_KEY_Copy:
+ return ASCIILiteral("Copy");
+ case GDK_KEY_3270_CursorSelect:
+ return ASCIILiteral("CrSel");
+ case GDK_KEY_Cut:
+ return ASCIILiteral("Cut");
+ case GDK_KEY_Delete:
+ case GDK_KEY_KP_Delete:
+ return ASCIILiteral("Delete");
+ case GDK_KEY_3270_EraseEOF:
+ return ASCIILiteral("EraseEof");
+ case GDK_KEY_3270_ExSelect:
+ return ASCIILiteral("ExSel");
+ case GDK_KEY_Insert:
+ case GDK_KEY_KP_Insert:
+ return ASCIILiteral("Insert");
+ case GDK_KEY_Paste:
+ return ASCIILiteral("Paste");
+ case GDK_KEY_Redo:
+ return ASCIILiteral("Redo");
+ case GDK_KEY_Undo:
+ return ASCIILiteral("Undo");
+
+ // UI keys.
+ // Accept.
+ // Again.
+ case GDK_KEY_3270_Attn:
+ return ASCIILiteral("Attn");
+ case GDK_KEY_Cancel:
+ return ASCIILiteral("Cancel");
+ case GDK_KEY_Menu:
+ return ASCIILiteral("ContextMenu");
+ case GDK_KEY_Escape:
+ return ASCIILiteral("Escape");
+ case GDK_KEY_Execute:
+ return ASCIILiteral("Execute");
+ case GDK_KEY_Find:
+ return ASCIILiteral("Find");
+ case GDK_KEY_Help:
+ return ASCIILiteral("Help");
+ case GDK_KEY_Pause:
+ case GDK_KEY_Break:
+ return ASCIILiteral("Pause");
+ case GDK_KEY_3270_Play:
+ return ASCIILiteral("Play");
+ // Props.
+ case GDK_KEY_Select:
+ return ASCIILiteral("Select");
+ case GDK_KEY_ZoomIn:
+ return ASCIILiteral("ZoomIn");
+ case GDK_KEY_ZoomOut:
+ return ASCIILiteral("ZoomOut");
+
+ // Device keys.
+ case GDK_KEY_MonBrightnessDown:
+ return ASCIILiteral("BrightnessDown");
+ case GDK_KEY_MonBrightnessUp:
+ return ASCIILiteral("BrightnessUp");
+ case GDK_KEY_Eject:
+ return ASCIILiteral("Eject");
+ case GDK_KEY_LogOff:
+ return ASCIILiteral("LogOff");
+ // Power.
+ case GDK_KEY_PowerDown:
+ case GDK_KEY_PowerOff:
+ return ASCIILiteral("PowerOff");
+ case GDK_KEY_3270_PrintScreen:
+ case GDK_KEY_Print:
+ case GDK_KEY_Sys_Req:
+ return ASCIILiteral("PrintScreen");
+ case GDK_KEY_Hibernate:
+ return ASCIILiteral("Hibernate");
+ case GDK_KEY_Standby:
+ case GDK_KEY_Suspend:
+ case GDK_KEY_Sleep:
+ return ASCIILiteral("Standby");
+ case GDK_KEY_WakeUp:
+ return ASCIILiteral("WakeUp");
+
+ // IME keys.
+ case GDK_KEY_MultipleCandidate:
+ return ASCIILiteral("AllCandidates");
+ case GDK_KEY_Eisu_Shift:
+ case GDK_KEY_Eisu_toggle:
+ return ASCIILiteral("Alphanumeric");
+ case GDK_KEY_Codeinput:
+ return ASCIILiteral("CodeInput");
+ case GDK_KEY_Multi_key:
+ return ASCIILiteral("Compose");
+ case GDK_KEY_Henkan:
+ return ASCIILiteral("Convert");
+ case GDK_KEY_dead_grave:
+ case GDK_KEY_dead_acute:
+ case GDK_KEY_dead_circumflex:
+ case GDK_KEY_dead_tilde:
+ case GDK_KEY_dead_macron:
+ case GDK_KEY_dead_breve:
+ case GDK_KEY_dead_abovedot:
+ case GDK_KEY_dead_diaeresis:
+ case GDK_KEY_dead_abovering:
+ case GDK_KEY_dead_doubleacute:
+ case GDK_KEY_dead_caron:
+ case GDK_KEY_dead_cedilla:
+ case GDK_KEY_dead_ogonek:
+ case GDK_KEY_dead_iota:
+ case GDK_KEY_dead_voiced_sound:
+ case GDK_KEY_dead_semivoiced_sound:
+ case GDK_KEY_dead_belowdot:
+ case GDK_KEY_dead_hook:
+ case GDK_KEY_dead_horn:
+ case GDK_KEY_dead_stroke:
+ case GDK_KEY_dead_abovecomma:
+ case GDK_KEY_dead_abovereversedcomma:
+ case GDK_KEY_dead_doublegrave:
+ case GDK_KEY_dead_belowring:
+ case GDK_KEY_dead_belowmacron:
+ case GDK_KEY_dead_belowcircumflex:
+ case GDK_KEY_dead_belowtilde:
+ case GDK_KEY_dead_belowbreve:
+ case GDK_KEY_dead_belowdiaeresis:
+ case GDK_KEY_dead_invertedbreve:
+ case GDK_KEY_dead_belowcomma:
+ case GDK_KEY_dead_currency:
+ case GDK_KEY_dead_a:
+ case GDK_KEY_dead_A:
+ case GDK_KEY_dead_e:
+ case GDK_KEY_dead_E:
+ case GDK_KEY_dead_i:
+ case GDK_KEY_dead_I:
+ case GDK_KEY_dead_o:
+ case GDK_KEY_dead_O:
+ case GDK_KEY_dead_u:
+ case GDK_KEY_dead_U:
+ case GDK_KEY_dead_small_schwa:
+ case GDK_KEY_dead_capital_schwa:
+ return ASCIILiteral("Dead");
+ // FinalMode
+ case GDK_KEY_ISO_First_Group:
+ return ASCIILiteral("GroupFirst");
+ case GDK_KEY_ISO_Last_Group:
+ return ASCIILiteral("GroupLast");
+ case GDK_KEY_ISO_Next_Group:
+ return ASCIILiteral("GroupNext");
+ case GDK_KEY_ISO_Prev_Group:
+ return ASCIILiteral("GroupPrevious");
+ case GDK_KEY_Mode_switch:
+ return ASCIILiteral("ModeChange");
+ // NextCandidate.
+ case GDK_KEY_Muhenkan:
+ return ASCIILiteral("NonConvert");
+ case GDK_KEY_PreviousCandidate:
+ return ASCIILiteral("PreviousCandidate");
+ // Process.
+ case GDK_KEY_SingleCandidate:
+ return ASCIILiteral("SingleCandidate");
+
+ // Korean and Japanese keys.
+ case GDK_KEY_Hangul:
+ return ASCIILiteral("HangulMode");
+ case GDK_KEY_Hangul_Hanja:
+ return ASCIILiteral("HanjaMode");
+ case GDK_KEY_Hangul_Jeonja:
+ return ASCIILiteral("JunjaMode");
+ case GDK_KEY_Hankaku:
+ return ASCIILiteral("Hankaku");
+ case GDK_KEY_Hiragana:
+ return ASCIILiteral("Hiragana");
+ case GDK_KEY_Hiragana_Katakana:
+ return ASCIILiteral("HiraganaKatakana");
+ case GDK_KEY_Kana_Lock:
+ case GDK_KEY_Kana_Shift:
+ return ASCIILiteral("KanaMode");
+ case GDK_KEY_Kanji:
+ return ASCIILiteral("KanjiMode");
+ case GDK_KEY_Katakana:
+ return ASCIILiteral("Katakana");
+ case GDK_KEY_Romaji:
+ return ASCIILiteral("Romaji");
+ case GDK_KEY_Zenkaku:
+ return ASCIILiteral("Zenkaku");
+ case GDK_KEY_Zenkaku_Hankaku:
+ return ASCIILiteral("ZenkakuHanaku");
+
+ // Multimedia keys.
+ // ChannelDown.
+ // ChannelUp.
+ case GDK_KEY_Close:
+ return ASCIILiteral("Close");
+ case GDK_KEY_MailForward:
+ return ASCIILiteral("MailForward");
+ case GDK_KEY_Reply:
+ return ASCIILiteral("MailReply");
+ case GDK_KEY_Send:
+ return ASCIILiteral("MailSend");
+ case GDK_KEY_AudioForward:
+ return ASCIILiteral("MediaFastForward");
+ case GDK_KEY_AudioPause:
+ return ASCIILiteral("MediaPause");
+ case GDK_KEY_AudioPlay:
+ return ASCIILiteral("MediaPlay");
+ // MediaPlayPause
+ case GDK_KEY_AudioRecord:
+ return ASCIILiteral("MediaRecord");
+ case GDK_KEY_AudioRewind:
+ return ASCIILiteral("MediaRewind");
+ case GDK_KEY_AudioStop:
+ return ASCIILiteral("MediaStop");
+ case GDK_KEY_AudioNext:
+ return ASCIILiteral("MediaTrackNext");
+ case GDK_KEY_AudioPrev:
+ return ASCIILiteral("MediaTrackPrevious");
+ case GDK_KEY_New:
+ return ASCIILiteral("New");
+ case GDK_KEY_Open:
+ return ASCIILiteral("Open");
+ // Print.
+ case GDK_KEY_Save:
+ return ASCIILiteral("Save");
+ case GDK_KEY_Spell:
+ return ASCIILiteral("SpellCheck");
+
+ // Function keys.
+ case GDK_KEY_F1:
+ return ASCIILiteral("F1");
+ case GDK_KEY_F2:
+ return ASCIILiteral("F2");
+ case GDK_KEY_F3:
+ return ASCIILiteral("F3");
+ case GDK_KEY_F4:
+ return ASCIILiteral("F4");
+ case GDK_KEY_F5:
+ return ASCIILiteral("F5");
+ case GDK_KEY_F6:
+ return ASCIILiteral("F6");
+ case GDK_KEY_F7:
+ return ASCIILiteral("F7");
+ case GDK_KEY_F8:
+ return ASCIILiteral("F8");
+ case GDK_KEY_F9:
+ return ASCIILiteral("F9");
+ case GDK_KEY_F10:
+ return ASCIILiteral("F10");
+ case GDK_KEY_F11:
+ return ASCIILiteral("F11");
+ case GDK_KEY_F12:
+ return ASCIILiteral("F12");
+ case GDK_KEY_F13:
+ return ASCIILiteral("F13");
+ case GDK_KEY_F14:
+ return ASCIILiteral("F14");
+ case GDK_KEY_F15:
+ return ASCIILiteral("F15");
+ case GDK_KEY_F16:
+ return ASCIILiteral("F16");
+ case GDK_KEY_F17:
+ return ASCIILiteral("F17");
+ case GDK_KEY_F18:
+ return ASCIILiteral("F18");
+ case GDK_KEY_F19:
+ return ASCIILiteral("F19");
+ case GDK_KEY_F20:
+ return ASCIILiteral("F20");
+
+ default: {
+ guint32 unicodeCharacter = gdk_keyval_to_unicode(keyCode);
+ if (unicodeCharacter) {
+ // UTF-8 will use up to 6 bytes.
+ char utf8[7] = { 0 };
+ g_unichar_to_utf8(unicodeCharacter, utf8);
+ return String::fromUTF8(utf8);
+ }
+ return ASCIILiteral("Unidentified");
+ }
+ }
+}
+
+// FIXME: This is incomplete. We should change this to mirror
+// more like what Firefox does, and generate these switch statements
+// at build time.
+// https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent/code
+String PlatformKeyboardEvent::keyCodeForHardwareKeyCode(unsigned keyCode)
+{
+ switch (keyCode) {
+ case 0x0009:
+ return ASCIILiteral("Escape");
+ case 0x000A:
+ return ASCIILiteral("Digit1");
+ case 0x000B:
+ return ASCIILiteral("Digit2");
+ case 0x000C:
+ return ASCIILiteral("Digit3");
+ case 0x000D:
+ return ASCIILiteral("Digit4");
+ case 0x000E:
+ return ASCIILiteral("Digit5");
+ case 0x000F:
+ return ASCIILiteral("Digit6");
+ case 0x0010:
+ return ASCIILiteral("Digit7");
+ case 0x0011:
+ return ASCIILiteral("Digit8");
+ case 0x0012:
+ return ASCIILiteral("Digit9");
+ case 0x0013:
+ return ASCIILiteral("Digit0");
+ case 0x0014:
+ return ASCIILiteral("Minus");
+ case 0x0015:
+ return ASCIILiteral("Equal");
+ case 0x0016:
+ return ASCIILiteral("Backspace");
+ case 0x0017:
+ return ASCIILiteral("Tab");
+ case 0x0018:
+ return ASCIILiteral("KeyQ");
+ case 0x0019:
+ return ASCIILiteral("KeyW");
+ case 0x001A:
+ return ASCIILiteral("KeyE");
+ case 0x001B:
+ return ASCIILiteral("KeyR");
+ case 0x001C:
+ return ASCIILiteral("KeyT");
+ case 0x001D:
+ return ASCIILiteral("KeyY");
+ case 0x001E:
+ return ASCIILiteral("KeyU");
+ case 0x001F:
+ return ASCIILiteral("KeyI");
+ case 0x0020:
+ return ASCIILiteral("KeyO");
+ case 0x0021:
+ return ASCIILiteral("KeyP");
+ case 0x0022:
+ return ASCIILiteral("BracketLeft");
+ case 0x0023:
+ return ASCIILiteral("BracketRight");
+ case 0x0024:
+ return ASCIILiteral("Enter");
+ case 0x0025:
+ return ASCIILiteral("ControlLeft");
+ case 0x0026:
+ return ASCIILiteral("KeyA");
+ case 0x0027:
+ return ASCIILiteral("KeyS");
+ case 0x0028:
+ return ASCIILiteral("KeyD");
+ case 0x0029:
+ return ASCIILiteral("KeyF");
+ case 0x002A:
+ return ASCIILiteral("KeyG");
+ case 0x002B:
+ return ASCIILiteral("KeyH");
+ case 0x002C:
+ return ASCIILiteral("KeyJ");
+ case 0x002D:
+ return ASCIILiteral("KeyK");
+ case 0x002E:
+ return ASCIILiteral("KeyL");
+ case 0x002F:
+ return ASCIILiteral("Semicolon");
+ case 0x0030:
+ return ASCIILiteral("Quote");
+ case 0x0031:
+ return ASCIILiteral("Backquote");
+ case 0x0032:
+ return ASCIILiteral("ShiftLeft");
+ case 0x0033:
+ return ASCIILiteral("Backslash");
+ case 0x0034:
+ return ASCIILiteral("KeyZ");
+ case 0x0035:
+ return ASCIILiteral("KeyX");
+ case 0x0036:
+ return ASCIILiteral("KeyC");
+ case 0x0037:
+ return ASCIILiteral("KeyV");
+ case 0x0038:
+ return ASCIILiteral("KeyB");
+ case 0x0039:
+ return ASCIILiteral("KeyN");
+ case 0x003A:
+ return ASCIILiteral("KeyM");
+ case 0x003B:
+ return ASCIILiteral("Comma");
+ case 0x003C:
+ return ASCIILiteral("Period");
+ case 0x003D:
+ return ASCIILiteral("Slash");
+ case 0x003E:
+ return ASCIILiteral("ShiftRight");
+ case 0x003F:
+ return ASCIILiteral("NumpadMultiply");
+ case 0x0040:
+ return ASCIILiteral("AltLeft");
+ case 0x0041:
+ return ASCIILiteral("Space");
+ case 0x0042:
+ return ASCIILiteral("CapsLock");
+ case 0x0043:
+ return ASCIILiteral("F1");
+ case 0x0044:
+ return ASCIILiteral("F2");
+ case 0x0045:
+ return ASCIILiteral("F3");
+ case 0x0046:
+ return ASCIILiteral("F4");
+ case 0x0047:
+ return ASCIILiteral("F5");
+ case 0x0048:
+ return ASCIILiteral("F6");
+ case 0x0049:
+ return ASCIILiteral("F7");
+ case 0x004A:
+ return ASCIILiteral("F8");
+ case 0x004B:
+ return ASCIILiteral("F9");
+ case 0x004C:
+ return ASCIILiteral("F10");
+ case 0x004D:
+ return ASCIILiteral("NumLock");
+ case 0x004E:
+ return ASCIILiteral("ScrollLock");
+ case 0x004F:
+ return ASCIILiteral("Numpad7");
+ case 0x0050:
+ return ASCIILiteral("Numpad8");
+ case 0x0051:
+ return ASCIILiteral("Numpad9");
+ case 0x0052:
+ return ASCIILiteral("NumpadSubtract");
+ case 0x0053:
+ return ASCIILiteral("Numpad4");
+ case 0x0054:
+ return ASCIILiteral("Numpad5");
+ case 0x0055:
+ return ASCIILiteral("Numpad6");
+ case 0x0056:
+ return ASCIILiteral("NumpadAdd");
+ case 0x0057:
+ return ASCIILiteral("Numpad1");
+ case 0x0058:
+ return ASCIILiteral("Numpad2");
+ case 0x0059:
+ return ASCIILiteral("Numpad3");
+ case 0x005A:
+ return ASCIILiteral("Numpad0");
+ case 0x005B:
+ return ASCIILiteral("NumpadDecimal");
+ case 0x005E:
+ return ASCIILiteral("IntlBackslash");
+ case 0x005F:
+ return ASCIILiteral("F11");
+ case 0x0060:
+ return ASCIILiteral("F12");
+ case 0x0061:
+ return ASCIILiteral("IntlRo");
+ case 0x0064:
+ return ASCIILiteral("Convert");
+ case 0x0065:
+ return ASCIILiteral("KanaMode");
+ case 0x0066:
+ return ASCIILiteral("NonConvert");
+ case 0x0068:
+ return ASCIILiteral("NumpadEnter");
+ case 0x0069:
+ return ASCIILiteral("ControlRight");
+ case 0x006A:
+ return ASCIILiteral("NumpadDivide");
+ case 0x006B:
+ return ASCIILiteral("PrintScreen");
+ case 0x006C:
+ return ASCIILiteral("AltRight");
+ case 0x006E:
+ return ASCIILiteral("Home");
+ case 0x006F:
+ return ASCIILiteral("ArrowUp");
+ case 0x0070:
+ return ASCIILiteral("PageUp");
+ case 0x0071:
+ return ASCIILiteral("ArrowLeft");
+ case 0x0072:
+ return ASCIILiteral("ArrowRight");
+ case 0x0073:
+ return ASCIILiteral("End");
+ case 0x0074:
+ return ASCIILiteral("ArrowDown");
+ case 0x0075:
+ return ASCIILiteral("PageDown");
+ case 0x0076:
+ return ASCIILiteral("Insert");
+ case 0x0077:
+ return ASCIILiteral("Delete");
+ case 0x0079:
+ return ASCIILiteral("AudioVolumeMute");
+ case 0x007A:
+ return ASCIILiteral("AudioVolumeDown");
+ case 0x007B:
+ return ASCIILiteral("AudioVolumeUp");
+ case 0x007D:
+ return ASCIILiteral("NumpadEqual");
+ case 0x007F:
+ return ASCIILiteral("Pause");
+ case 0x0081:
+ return ASCIILiteral("NumpadComma");
+ case 0x0082:
+ return ASCIILiteral("Lang1");
+ case 0x0083:
+ return ASCIILiteral("Lang2");
+ case 0x0084:
+ return ASCIILiteral("IntlYen");
+ case 0x0085:
+ return ASCIILiteral("OSLeft");
+ case 0x0086:
+ return ASCIILiteral("OSRight");
+ case 0x0087:
+ return ASCIILiteral("ContextMenu");
+ case 0x0088:
+ return ASCIILiteral("BrowserStop");
+ case 0x0089:
+ return ASCIILiteral("Again");
+ case 0x008A:
+ return ASCIILiteral("Props");
+ case 0x008B:
+ return ASCIILiteral("Undo");
+ case 0x008C:
+ return ASCIILiteral("Select");
+ case 0x008D:
+ return ASCIILiteral("Copy");
+ case 0x008E:
+ return ASCIILiteral("Open");
+ case 0x008F:
+ return ASCIILiteral("Paste");
+ case 0x0090:
+ return ASCIILiteral("Find");
+ case 0x0091:
+ return ASCIILiteral("Cut");
+ case 0x0092:
+ return ASCIILiteral("Help");
+ case 0x0094:
+ return ASCIILiteral("LaunchApp2");
+ case 0x0097:
+ return ASCIILiteral("WakeUp");
+ case 0x0098:
+ return ASCIILiteral("LaunchApp1");
+ case 0x00A3:
+ return ASCIILiteral("LaunchMail");
+ case 0x00A4:
+ return ASCIILiteral("BrowserFavorites");
+ case 0x00A6:
+ return ASCIILiteral("BrowserBack");
+ case 0x00A7:
+ return ASCIILiteral("BrowserForward");
+ case 0x00A9:
+ return ASCIILiteral("Eject");
+ case 0x00AB:
+ return ASCIILiteral("MediaTrackNext");
+ case 0x00AC:
+ return ASCIILiteral("MediaPlayPause");
+ case 0x00AD:
+ return ASCIILiteral("MediaTrackPrevious");
+ case 0x00AE:
+ return ASCIILiteral("MediaStop");
+ case 0x00B3:
+ return ASCIILiteral("LaunchMediaPlayer");
+ case 0x00B4:
+ return ASCIILiteral("BrowserHome");
+ case 0x00B5:
+ return ASCIILiteral("BrowserRefresh");
+ case 0x00BF:
+ return ASCIILiteral("F13");
+ case 0x00C0:
+ return ASCIILiteral("F14");
+ case 0x00C1:
+ return ASCIILiteral("F15");
+ case 0x00C2:
+ return ASCIILiteral("F16");
+ case 0x00C3:
+ return ASCIILiteral("F17");
+ case 0x00C4:
+ return ASCIILiteral("F18");
+ case 0x00C5:
+ return ASCIILiteral("F19");
+ case 0x00C6:
+ return ASCIILiteral("F20");
+ case 0x00C7:
+ return ASCIILiteral("F21");
+ case 0x00C8:
+ return ASCIILiteral("F22");
+ case 0x00C9:
+ return ASCIILiteral("F23");
+ case 0x00CA:
+ return ASCIILiteral("F24");
+ case 0x00E1:
+ return ASCIILiteral("BrowserSearch");
+ default:
+ return ASCIILiteral("Unidentified");
+ }
+}
+
+// FIXME: This is incomplete. We should change this to mirror
+// more like what Firefox does, and generate these switch statements
+// at build time.
</ins><span class="cx"> String PlatformKeyboardEvent::keyIdentifierForGdkKeyCode(unsigned keyCode)
</span><span class="cx"> {
</span><span class="cx"> switch (keyCode) {
</span><span class="lines">@@ -574,6 +1260,8 @@
</span><span class="cx"> : PlatformEvent(eventTypeForGdkKeyEvent(event), modifiersForGdkKeyEvent(event), currentTime())
</span><span class="cx"> , m_text(compositionResults.simpleString.length() ? compositionResults.simpleString : singleCharacterString(event->keyval))
</span><span class="cx"> , m_unmodifiedText(m_text)
</span><ins>+ , m_key(keyValueForGdkKeyCode(event->keyval))
+ , m_code(keyCodeForHardwareKeyCode(event->hardware_keycode))
</ins><span class="cx"> , m_keyIdentifier(keyIdentifierForGdkKeyCode(event->keyval))
</span><span class="cx"> , m_windowsVirtualKeyCode(windowsKeyCodeForGdkKeyCode(event->keyval))
</span><span class="cx"> , m_autoRepeat(false)
</span></span></pre></div>
<a id="trunkSourceWebKit2ChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/ChangeLog (210503 => 210504)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/ChangeLog 2017-01-09 12:46:34 UTC (rev 210503)
+++ trunk/Source/WebKit2/ChangeLog 2017-01-09 14:14:57 UTC (rev 210504)
</span><span class="lines">@@ -1,3 +1,16 @@
</span><ins>+2017-01-06 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>
+
+ [GTK] Should support key and code properties on keyboard events
+ https://bugs.webkit.org/show_bug.cgi?id=166759
+
+ Reviewed by Carlos Garcia Campos.
+
+ * Shared/WebEvent.h:
+ * Shared/WebKeyboardEvent.cpp:
+ (WebKit::WebKeyboardEvent::WebKeyboardEvent): accept web key value and code on construction.
+ * Shared/gtk/WebEventFactory.cpp:
+ (WebKit::WebEventFactory::createWebKeyboardEvent): pass web key value and code on construction.
+
</ins><span class="cx"> 2017-01-08 Antti Koivisto <antti@apple.com>
</span><span class="cx">
</span><span class="cx"> Move cache coders to WTF
</span></span></pre></div>
<a id="trunkSourceWebKit2SharedWebEventh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/Shared/WebEvent.h (210503 => 210504)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/Shared/WebEvent.h 2017-01-09 12:46:34 UTC (rev 210503)
+++ trunk/Source/WebKit2/Shared/WebEvent.h 2017-01-09 14:14:57 UTC (rev 210504)
</span><span class="lines">@@ -246,7 +246,7 @@
</span><span class="cx"> #if USE(APPKIT)
</span><span class="cx"> WebKeyboardEvent(Type, const String& text, const String& unmodifiedText, const String& key, const String& code, const String& keyIdentifier, int windowsVirtualKeyCode, int nativeVirtualKeyCode, int macCharCode, bool handledByInputMethod, const Vector<WebCore::KeypressCommand>&, bool isAutoRepeat, bool isKeypad, bool isSystemKey, Modifiers, double timestamp);
</span><span class="cx"> #elif PLATFORM(GTK)
</span><del>- WebKeyboardEvent(Type, const String& text, const String& keyIdentifier, int windowsVirtualKeyCode, int nativeVirtualKeyCode, bool handledByInputMethod, Vector<String>&& commands, bool isKeypad, Modifiers, double timestamp);
</del><ins>+ WebKeyboardEvent(Type, const String& text, const String& key, const String& code, const String& keyIdentifier, int windowsVirtualKeyCode, int nativeVirtualKeyCode, bool handledByInputMethod, Vector<String>&& commands, bool isKeypad, Modifiers, double timestamp);
</ins><span class="cx"> #else
</span><span class="cx"> WebKeyboardEvent(Type, const String& text, const String& unmodifiedText, const String& keyIdentifier, int windowsVirtualKeyCode, int nativeVirtualKeyCode, int macCharCode, bool isAutoRepeat, bool isKeypad, bool isSystemKey, Modifiers, double timestamp);
</span><span class="cx"> #endif
</span></span></pre></div>
<a id="trunkSourceWebKit2SharedWebKeyboardEventcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/Shared/WebKeyboardEvent.cpp (210503 => 210504)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/Shared/WebKeyboardEvent.cpp 2017-01-09 12:46:34 UTC (rev 210503)
+++ trunk/Source/WebKit2/Shared/WebKeyboardEvent.cpp 2017-01-09 14:14:57 UTC (rev 210504)
</span><span class="lines">@@ -58,10 +58,12 @@
</span><span class="cx">
</span><span class="cx"> #elif PLATFORM(GTK)
</span><span class="cx">
</span><del>-WebKeyboardEvent::WebKeyboardEvent(Type type, const String& text, const String& keyIdentifier, int windowsVirtualKeyCode, int nativeVirtualKeyCode, bool handledByInputMethod, Vector<String>&& commands, bool isKeypad, Modifiers modifiers, double timestamp)
</del><ins>+WebKeyboardEvent::WebKeyboardEvent(Type type, const String& text, const String& key, const String& code, const String& keyIdentifier, int windowsVirtualKeyCode, int nativeVirtualKeyCode, bool handledByInputMethod, Vector<String>&& commands, bool isKeypad, Modifiers modifiers, double timestamp)
</ins><span class="cx"> : WebEvent(type, modifiers, timestamp)
</span><span class="cx"> , m_text(text)
</span><span class="cx"> , m_unmodifiedText(text)
</span><ins>+ , m_key(key)
+ , m_code(code)
</ins><span class="cx"> , m_keyIdentifier(keyIdentifier)
</span><span class="cx"> , m_windowsVirtualKeyCode(windowsVirtualKeyCode)
</span><span class="cx"> , m_nativeVirtualKeyCode(nativeVirtualKeyCode)
</span></span></pre></div>
<a id="trunkSourceWebKit2SharedgtkWebEventFactorycpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/Shared/gtk/WebEventFactory.cpp (210503 => 210504)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/Shared/gtk/WebEventFactory.cpp 2017-01-09 12:46:34 UTC (rev 210503)
+++ trunk/Source/WebKit2/Shared/gtk/WebEventFactory.cpp 2017-01-09 14:14:57 UTC (rev 210504)
</span><span class="lines">@@ -193,6 +193,8 @@
</span><span class="cx"> return WebKeyboardEvent(
</span><span class="cx"> event->type == GDK_KEY_RELEASE ? WebEvent::KeyUp : WebEvent::KeyDown,
</span><span class="cx"> compositionResults.simpleString.length() ? compositionResults.simpleString : PlatformKeyboardEvent::singleCharacterString(event->key.keyval),
</span><ins>+ PlatformKeyboardEvent::keyValueForGdkKeyCode(event->key.keyval),
+ PlatformKeyboardEvent::keyCodeForHardwareKeyCode(event->key.hardware_keycode),
</ins><span class="cx"> PlatformKeyboardEvent::keyIdentifierForGdkKeyCode(event->key.keyval),
</span><span class="cx"> PlatformKeyboardEvent::windowsKeyCodeForGdkKeyCode(event->key.keyval),
</span><span class="cx"> static_cast<int>(event->key.keyval),
</span></span></pre></div>
<a id="trunkToolsChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Tools/ChangeLog (210503 => 210504)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Tools/ChangeLog 2017-01-09 12:46:34 UTC (rev 210503)
+++ trunk/Tools/ChangeLog 2017-01-09 14:14:57 UTC (rev 210504)
</span><span class="lines">@@ -1,3 +1,14 @@
</span><ins>+2017-01-06 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>
+
+ [GTK] Should support key and code properties on keyboard events
+ https://bugs.webkit.org/show_bug.cgi?id=166759
+
+ Reviewed by Carlos Garcia Campos.
+
+ * WebKitTestRunner/gtk/EventSenderProxyGtk.cpp:
+ (WTR::getGDKKeySymForKeyRef): add missing modifier keys which were being reported as 'l' and 'r' only
+ on tests.
+
</ins><span class="cx"> 2017-01-06 Per Arne Vollan <pvollan@apple.com>
</span><span class="cx">
</span><span class="cx"> [Win] Some tests are flaky because certain DLLs are writing to stdout.
</span></span></pre></div>
<a id="trunkToolsWebKitTestRunnergtkEventSenderProxyGtkcpp"></a>
<div class="modfile"><h4>Modified: trunk/Tools/WebKitTestRunner/gtk/EventSenderProxyGtk.cpp (210503 => 210504)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Tools/WebKitTestRunner/gtk/EventSenderProxyGtk.cpp 2017-01-09 12:46:34 UTC (rev 210503)
+++ trunk/Tools/WebKitTestRunner/gtk/EventSenderProxyGtk.cpp 2017-01-09 14:14:57 UTC (rev 210504)
</span><span class="lines">@@ -226,6 +226,18 @@
</span><span class="cx"> return GDK_KEY_VoidSymbol;
</span><span class="cx"> }
</span><span class="cx">
</span><ins>+ if (WKStringIsEqualToUTF8CString(keyRef, "leftControl"))
+ return GDK_KEY_Control_L;
+ if (WKStringIsEqualToUTF8CString(keyRef, "rightControl"))
+ return GDK_KEY_Control_R;
+ if (WKStringIsEqualToUTF8CString(keyRef, "leftShift"))
+ return GDK_KEY_Shift_L;
+ if (WKStringIsEqualToUTF8CString(keyRef, "rightShift"))
+ return GDK_KEY_Shift_R;
+ if (WKStringIsEqualToUTF8CString(keyRef, "leftAlt"))
+ return GDK_KEY_Alt_L;
+ if (WKStringIsEqualToUTF8CString(keyRef, "rightAlt"))
+ return GDK_KEY_Alt_R;
</ins><span class="cx"> if (WKStringIsEqualToUTF8CString(keyRef, "leftArrow"))
</span><span class="cx"> return GDK_KEY_Left;
</span><span class="cx"> if (WKStringIsEqualToUTF8CString(keyRef, "rightArrow"))
</span></span></pre>
</div>
</div>
</body>
</html>