[Windows] Produce correct DomKey for NumPad when combined with Shift/NumLock

ui/events/keycodes/platform_key_map_win.h

Index: ui/events/keycodes/platform_key_map_win.h
diff --git a/ui/events/keycodes/platform_key_map_win.h b/ui/events/keycodes/platform_key_map_win.h
index 7517f2a7ee470d2a78bc7b0a6a88f935dbeffe5d..800024d188fa9ea6fec6f29a9a886f49cbee7f4e 100644
--- a/ui/events/keycodes/platform_key_map_win.h
+++ b/ui/events/keycodes/platform_key_map_win.h
@@ -9,8 +9,9 @@
 
 #include <unordered_map>
 
+#include "base/event_types.h"
 #include "base/hash.h"
-#include "ui/events/events_base_export.h"
+#include "ui/events/events_export.h"
 #include "ui/events/keycodes/dom/dom_key.h"
 #include "ui/events/keycodes/keyboard_codes_win.h"
 
@@ -18,27 +19,30 @@ namespace ui {
 
 enum class DomCode;
 
-class EVENTS_BASE_EXPORT PlatformKeyMap {
+class EVENTS_EXPORT PlatformKeyMap {
  public:
   // Create and load key map table with specified keyboard layout.
   explicit PlatformKeyMap(HKL layout);
   ~PlatformKeyMap();

[Wez, 2016/03/18 22:50:39]

nit: Can the ctor also be private, since it's now only used by the static method
below?

Better still, can we move the entire class into the cc file and just have a
ui::DomKeyFromNativeEvent() function declared here, if nothing needs to see the
PlatformKeyMap implementation details?

[chongz, 2016/03/21 20:07:29]

Tried to move ctor/dtor to private but they are required by unit tests (outside
friend class) and |CleanupKeyMapTls| in .cc, and the same reason for not being
able to move the class...

Instead I added a comment stating that the ctor/dtor are visible for testing.

 
-  // Returns the DomKey 'meaning' of |code| in the context of specified
-  // |ui_event_flags| and stored keyboard layout.
-  DomKey DomCodeAndFlagsToDomKey(DomCode code,
-                                 int ui_event_flags) const;
-
-  // Returns the DomKey 'meaning' of |code| in the context of specified
-  // |ui_event_flags| and the keyboard layout of current thread.
+  // Returns the DOM KeyboardEvent key from a native event and the keyboard
+  // layout of current thread.
   // Updates a per-thread key map cache whenever the layout changes.
-  static DomKey DomCodeAndFlagsToDomKeyStatic(
-      DomCode code,
-      int ui_event_flags);
+  static DomKey DomKeyFromNativeStatic(const base::NativeEvent& native_event);
 
  private:
+  friend class PlatformKeyMapTest;
+
   PlatformKeyMap();
 
+  // TODO(chongz): Expose this function when we need to access separate layout.
+  // Returns the DomKey 'meaning' of |code| in the context of specified
+  // |ui_event_flags| and stored keyboard layout.
+  // |key_code| will only be used for NumPad.
+  DomKey DomCodeAndFlagsToDomKey(DomCode code,

[chongz, 2016/03/17 21:30:32]

Moved to internal since no one is actually using it other than
|DomKeyFromNativeStatic| and unittests.

+                                 KeyboardCode key_code,
+                                 int ui_event_flags) const;
+
   // TODO(chongz): Expose this function in response to WM_INPUTLANGCHANGE.
   void UpdateLayout(HKL layout);