Refactoring button field and its type

content/browser/renderer_host/input/web_input_event_builders_win.cc

Index: content/browser/renderer_host/input/web_input_event_builders_win.cc
diff --git a/content/browser/renderer_host/input/web_input_event_builders_win.cc b/content/browser/renderer_host/input/web_input_event_builders_win.cc
index 51f64e6ed9b8d6ddf6fd3bc0b0bb23973baaf188..81939ac2754f974543176d3c4783bdf49b4df64d 100644
--- a/content/browser/renderer_host/input/web_input_event_builders_win.cc
+++ b/content/browser/renderer_host/input/web_input_event_builders_win.cc
@@ -101,20 +101,20 @@ WebMouseEvent WebMouseEventBuilder::Build(
   case WM_MOUSEMOVE:
     result.type = WebInputEvent::MouseMove;
     if (wparam & MK_LBUTTON)
-      result.button = WebMouseEvent::ButtonLeft;
+      result.button = WebMouseEvent::Button::Left;
     else if (wparam & MK_MBUTTON)
-      result.button = WebMouseEvent::ButtonMiddle;
+      result.button = WebMouseEvent::Button::Middle;
     else if (wparam & MK_RBUTTON)
-      result.button = WebMouseEvent::ButtonRight;
+      result.button = WebMouseEvent::Button::Right;
     else
-      result.button = WebMouseEvent::ButtonNone;
+      result.button = WebMouseEvent::Button::NoButton;
     break;
   case WM_MOUSELEAVE:
   case WM_NCMOUSELEAVE:
     // TODO(rbyers): This should be MouseLeave but is disabled temporarily.
     // See http://crbug.com/450631
     result.type = WebInputEvent::MouseMove;
-    result.button = WebMouseEvent::ButtonNone;
+    result.button = WebMouseEvent::Button::NoButton;
     // set the current mouse position (relative to the client area of the
     // current window) since none is specified for this event
     lparam = GetRelativeCursorPos(hwnd);
@@ -122,29 +122,29 @@ WebMouseEvent WebMouseEventBuilder::Build(
   case WM_LBUTTONDOWN:
   case WM_LBUTTONDBLCLK:
     result.type = WebInputEvent::MouseDown;
-    result.button = WebMouseEvent::ButtonLeft;
+    result.button = WebMouseEvent::Button::Left;
     break;
   case WM_MBUTTONDOWN:
   case WM_MBUTTONDBLCLK:
     result.type = WebInputEvent::MouseDown;
-    result.button = WebMouseEvent::ButtonMiddle;
+    result.button = WebMouseEvent::Button::Middle;
     break;
   case WM_RBUTTONDOWN:
   case WM_RBUTTONDBLCLK:
     result.type = WebInputEvent::MouseDown;
-    result.button = WebMouseEvent::ButtonRight;
+    result.button = WebMouseEvent::Button::Right;
     break;
   case WM_LBUTTONUP:
     result.type = WebInputEvent::MouseUp;
-    result.button = WebMouseEvent::ButtonLeft;
+    result.button = WebMouseEvent::Button::Left;
     break;
   case WM_MBUTTONUP:
     result.type = WebInputEvent::MouseUp;
-    result.button = WebMouseEvent::ButtonMiddle;
+    result.button = WebMouseEvent::Button::Middle;
     break;
   case WM_RBUTTONUP:
     result.type = WebInputEvent::MouseUp;
-    result.button = WebMouseEvent::ButtonRight;
+    result.button = WebMouseEvent::Button::Right;
     break;
   default:
     NOTREACHED();
@@ -176,7 +176,7 @@ WebMouseEvent WebMouseEventBuilder::Build(
   // where their original code looks buggy.
   static int last_click_position_x;
   static int last_click_position_y;
-  static WebMouseEvent::Button last_click_button = WebMouseEvent::ButtonLeft;
+  static WebMouseEvent::Button last_click_button = WebMouseEvent::Button::Left;
 
   double current_time = result.timeStampSeconds;
   bool cancel_previous_click =
@@ -239,7 +239,7 @@ WebMouseWheelEvent WebMouseWheelEventBuilder::Build(
 
   result.timeStampSeconds = time_stamp;
 
-  result.button = WebMouseEvent::ButtonNone;
+  result.button = WebMouseEvent::Button::NoButton;
 
   result.pointerType = pointer_type;