Fix 'repeat' state of a DOM KeyboardEvent on Windows

content/browser/renderer_host/input/web_input_event_builders_win.cc

 // Copyright 2013 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 "content/browser/renderer_host/input/web_input_event_builders_win.h"
 
 #include "base/logging.h"
 #include "content/browser/renderer_host/input/web_input_event_util.h"
 
 using blink::WebInputEvent;
@@ skipping 142 matching lines @@
 
   if (::GetKeyState(VK_SHIFT) & 0x8000)
     result.modifiers |= WebInputEvent::ShiftKey;
   if (::GetKeyState(VK_CONTROL) & 0x8000)
     result.modifiers |= WebInputEvent::ControlKey;
   if (::GetKeyState(VK_MENU) & 0x8000)
     result.modifiers |= WebInputEvent::AltKey;
   // NOTE: There doesn't seem to be a way to query the mouse button state in
   // this case.
 
-  if (LOWORD(lparam) > 1)
+  if (lparam & (1 << 30))

[jamesr, 2014/07/28 19:38:49]

the rest of this file uses & 0xXXXXX for masks, you should probably be
consistent here.

is this what other browsers do?

     result.modifiers |= WebInputEvent::IsAutoRepeat;
 
   result.modifiers |= GetLocationModifier(wparam, lparam);
 
   SetToggleKeyState(&result);
   return result;
 }
 
 // WebMouseEvent --------------------------------------------------------------
 
@@ skipping 278 matching lines @@
     result.wheelTicksX = wheel_delta;
   } else {
     result.deltaY = scroll_delta;
     result.wheelTicksY = wheel_delta;
   }
 
   return result;
 }
 
 }  // namespace content