Clean up event flags a bit:

ui/content_accelerators/accelerator_util.cc

Index: ui/content_accelerators/accelerator_util.cc
diff --git a/ui/content_accelerators/accelerator_util.cc b/ui/content_accelerators/accelerator_util.cc
index 1a3ee9567f52a9a10db97bfb346b8797e3c5b850..9c5bdf3b50ec8ed65725556bedf6e7c27002deb0 100644
--- a/ui/content_accelerators/accelerator_util.cc
+++ b/ui/content_accelerators/accelerator_util.cc
@@ -39,7 +39,8 @@ ui::Accelerator GetAcceleratorFromNativeWebKeyboardEvent(
   if (event.type == blink::WebInputEvent::KeyUp)
     accelerator.set_type(ui::ET_KEY_RELEASED);
 #if defined(USE_AURA)
-  if (event.os_event && event.os_event->IsRepeat())
+  if (event.os_event && event.os_event->IsKeyEvent() &&

[sadrul, 2016/01/06 17:42:12]

It shouldn't be necessary to check IsKeyEvent() here, since we are getting the
event from a NativeWebKeyboardEvent here. Perhaps DCHECK(!event.os_event ||
event.os_event->IsKeyEvent()) instead?

[Peter Kasting, 2016/01/06 20:25:15]

I'll just remove the IsKeyEvent() check.

+      static_cast<ui::KeyEvent*>(event.os_event)->is_repeat())
     accelerator.set_is_repeat(true);
 #endif
   return accelerator;