[DomKey] Expose Korean special keys on Korean keyboard layout only

ui/events/keycodes/platform_key_map_win_unittest.cc

 // Copyright (c) 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/events/keycodes/platform_key_map_win.h"
 
 #include "base/macros.h"
 #include "base/strings/string16.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/events/event_constants.h"
 #include "ui/events/keycodes/dom/dom_code.h"
 #include "ui/events/keycodes/dom/dom_key.h"
 #include "ui/events/keycodes/dom/keycode_converter.h"
 #include "ui/events/keycodes/dom_us_layout_data.h"
 
 namespace ui {
 
 namespace {
 
 const wchar_t* LAYOUT_US = L"00000409";
 const wchar_t* LAYOUT_FR = L"0000040c";
+const wchar_t* LAYOUT_KO = L"00000412";
 
 struct TestKey {
   // Have to use KeyboardCode instead of DomCode because we don't know the
   // physical keyboard layout for try bots.
   KeyboardCode key_code;
   const char* normal;
   const char* shift;
   const char* capslock;
   const char* altgr;
   const char* shift_capslock;
@@ skipping 248 matching lines @@
         test_case.dom_key,
         DomKeyFromNativeImpl(keymap, DomCode::NONE, key_code, EF_CONTROL_DOWN))
         << key_code << ", " << scan_code;
     EXPECT_EQ(test_case.dom_key,
               DomKeyFromNativeImpl(keymap, DomCode::NONE, key_code,
                                    EF_ALTGR_DOWN | EF_CONTROL_DOWN))
         << key_code << ", " << scan_code;
   }
 }
 
+TEST_F(PlatformKeyMapTest, KoreanSpecialDomKey) {

[Wez, 2016/06/10 22:55:09]

nit: Suggest just KoreanDomKey or KoreanSpecificKeys

[chongz, 2016/06/14 20:41:47]

Done.

+  const struct TestCase {
+    KeyboardCode key_code;
+    DomKey key;
+  } kKoreanTestCases[] = {
+      {VKEY_HANGUL, DomKey::HANGUL_MODE}, {VKEY_HANJA, DomKey::HANJA_MODE},
+  };
+
+  // US layout does not have these keys.

[Wez, 2016/06/10 22:55:09]

nit: "US English should not return values for these keys."

[chongz, 2016/06/14 20:41:47]

Done.

+  HKL us_layout = ::LoadKeyboardLayout(LAYOUT_US, 0);
+  PlatformKeyMap us_keymap(us_layout);
+  for (const auto& test_case : kKoreanTestCases) {
+    EXPECT_EQ(DomKey::NONE, DomKeyFromNativeImpl(us_keymap, DomCode::NONE,
+                                                 test_case.key_code, EF_NONE))
+        << test_case.key_code;
+  }
+
+  // Korean layout should return specific DomKey.
+  HKL ko_layout = ::LoadKeyboardLayout(LAYOUT_KO, 0);
+  PlatformKeyMap ko_keymap(ko_layout);
+  for (const auto& test_case : kKoreanTestCases) {
+    EXPECT_EQ(test_case.key, DomKeyFromNativeImpl(ko_keymap, DomCode::NONE,
+                                                  test_case.key_code, EF_NONE))
+        << test_case.key_code;
+  }
+}
+
 }  // namespace ui