Add a new constructor to KeyEvent.

views/events/event_win.cc

Index: views/events/event_win.cc
===================================================================
--- views/events/event_win.cc	(revision 74437)
+++ views/events/event_win.cc	(working copy)
@@ -1,4 +1,4 @@
-// Copyright (c) 2009 The Chromium Authors. All rights reserved.
+// Copyright (c) 2011 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.
 
@@ -6,18 +6,75 @@
 
 #include <windows.h>
 
+#include "base/logging.h"
+#include "ui/base/keycodes/keyboard_code_conversion_win.h"
+
 namespace views {
 
-KeyEvent::KeyEvent(ui::EventType type, ui::KeyboardCode key_code,
-                   int event_flags, int repeat_count, int message_flags,
-                   UINT message)
-    : Event(type, event_flags),
-      key_code_(key_code),
-      repeat_count_(repeat_count),
-      message_flags_(message_flags),
-      message_(message) {
+namespace {
+
+// Returns a mask corresponding to the set of modifier keys that are currently
+// pressed. Windows key messages don't come with control key state as parameters
+// as with mouse messages, so we need to explicitly ask for these states.
+int GetKeyStateFlags() {
+  int flags = 0;
+  if (GetKeyState(VK_MENU) & 0x80)
+    flags |= ui::EF_ALT_DOWN;
+  if (GetKeyState(VK_SHIFT) & 0x80)
+    flags |= ui::EF_SHIFT_DOWN;
+  if (GetKeyState(VK_CONTROL) & 0x80)
+    flags |= ui::EF_CONTROL_DOWN;
+  return flags;
 }
 
+// Convert windows message identifiers to Event types.
+ui::EventType EventTypeFromNative(NativeEvent native_event) {
+  switch (native_event.message) {
+    case WM_KEYDOWN:
+    case WM_SYSKEYDOWN:
+      return ui::ET_KEY_PRESSED;
+    case WM_KEYUP:
+    case WM_SYSKEYUP:
+      return ui::ET_KEY_RELEASED;
+    case WM_LBUTTONDOWN:
+    case WM_MBUTTONDOWN:
+    case WM_NCLBUTTONDOWN:
+    case WM_NCMBUTTONDOWN:
+    case WM_NCRBUTTONDOWN:
+    case WM_RBUTTONDOWN:
+      return ui::ET_MOUSE_PRESSED;
+    case WM_LBUTTONDBLCLK:
+    case WM_LBUTTONUP:
+    case WM_MBUTTONDBLCLK:
+    case WM_MBUTTONUP:
+    case WM_NCLBUTTONDBLCLK:
+    case WM_NCLBUTTONUP:
+    case WM_NCMBUTTONDBLCLK:
+    case WM_NCMBUTTONUP:
+    case WM_NCRBUTTONDBLCLK:
+    case WM_NCRBUTTONUP:
+    case WM_RBUTTONDBLCLK:
+    case WM_RBUTTONUP:
+      return ui::ET_MOUSE_RELEASED;
+    case WM_MOUSEMOVE:
+    case WM_NCMOUSEMOVE:
+      return ui::ET_MOUSE_MOVED;
+    case WM_MOUSEWHEEL:
+      return ui::ET_MOUSEWHEEL;
+    case WM_MOUSELEAVE:
+    case WM_NCMOUSELEAVE:
+      return ui::ET_MOUSE_EXITED;
+    default:
+      NOTREACHED();
+  }
+  return ui::ET_UNKNOWN;
+}
+
+}  // namespace
+
+////////////////////////////////////////////////////////////////////////////////
+// Event, public:
+
 int Event::GetWindowsFlags() const {
   // TODO: need support for x1/x2.
   int result = 0;
@@ -47,23 +104,21 @@
   return r;
 }
 
-// static
-int KeyEvent::GetKeyStateFlags() {
-  // Windows Keyboard messages don't come with control key state as parameters
-  // like mouse messages do, so we need to explicitly probe for these key
-  // states.
-  int flags = 0;
-  if (GetKeyState(VK_MENU) & 0x80)
-    flags |= ui::EF_ALT_DOWN;
-  if (GetKeyState(VK_SHIFT) & 0x80)
-    flags |= ui::EF_SHIFT_DOWN;
-  if (GetKeyState(VK_CONTROL) & 0x80)
-    flags |= ui::EF_CONTROL_DOWN;
-  return flags;
+////////////////////////////////////////////////////////////////////////////////
+// KeyEvent, public:
+
+KeyEvent::KeyEvent(NativeEvent native_event)
+    : Event(native_event,
+            EventTypeFromNative(native_event),
+            GetKeyStateFlags()),
+      key_code_(ui::KeyboardCodeForWindowsKeyCode(native_event.wParam)) {
 }
 
-bool KeyEvent::IsExtendedKey() const {
-  return (message_flags_ & KF_EXTENDED) == KF_EXTENDED;
+KeyEvent::KeyEvent(NativeEvent2 native_event_2)
+    : Event(native_event_2, ui::ET_UNKNOWN, 0) {
+  // No one should ever call this on Windows.
+  // TODO(beng): remove once we rid views of Gtk/Gdk.
+  NOTREACHED();
 }
 
 }  // namespace views