Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1082)

Unified Diff: content/browser/renderer_host/web_input_event_aura.cc

Issue 929053004: [KeyboardEvent] Add embedder APIs to translate between Dom |key| enum and strings (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Code review comments Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | content/child/blink_platform_impl.h » ('j') | content/child/blink_platform_impl.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/renderer_host/web_input_event_aura.cc
diff --git a/content/browser/renderer_host/web_input_event_aura.cc b/content/browser/renderer_host/web_input_event_aura.cc
index ce015d6691a9922ab8ec8383f13069922d600fb3..65d45215cf647f49265bf658b866136511176d86 100644
--- a/content/browser/renderer_host/web_input_event_aura.cc
+++ b/content/browser/renderer_host/web_input_event_aura.cc
@@ -72,6 +72,7 @@ blink::WebKeyboardEvent MakeWebKeyboardEventFromAuraEvent(
webkit_event.nativeKeyCode =
ui::KeycodeConverter::DomCodeToNativeKeycode(event.code());
webkit_event.domCode = static_cast<int>(event.code());
+ webkit_event.domKey = static_cast<int>(event.GetDomKey());
Wez 2015/05/07 00:23:22 You also need to propagate the ui::KeyEvent::chara
Habib Virji 2015/05/19 16:16:28 It is already covered in the below unmodifiedText
kpschoedel 2015/05/19 20:00:26 You'll also need to set .domKey in the OS_WIN case
Habib Virji 2015/05/21 16:36:09 Done.
Wez 2015/06/03 00:35:08 unmodifiedText is an old field in WebKit keyboard
Habib Virji 2015/06/05 20:05:11 In GetUnmodifiedText it does not check the |keypre
Wez 2015/06/11 00:09:05 No, but the GetCharacter() function that it delega
Habib Virji 2015/06/24 14:32:12 Acknowledged.
webkit_event.unmodifiedText[0] = event.GetUnmodifiedText();
webkit_event.text[0] = event.GetText();
« no previous file with comments | « no previous file | content/child/blink_platform_impl.h » ('j') | content/child/blink_platform_impl.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698