Fix is_browser_shortcut; add a test.

third_party/WebKit/Source/core/input/KeyboardEventManager.cpp

 // Copyright 2016 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 "KeyboardEventManager.h"
 
 #include <memory>
 
 #include "core/dom/Element.h"
 #include "core/dom/UserGestureIndicator.h"
@@ skipping 148 matching lines @@
   Element* elem =
       frame_->GetDocument()->GetElementByAccessKey(key.DeprecatedLower());
   if (!elem)
     return false;
   elem->AccessKeyAction(false);
   return true;
 }
 
 WebInputEventResult KeyboardEventManager::KeyEvent(
     const WebKeyboardEvent& initial_key_event) {
+  LOG(ERROR) << "KeyboardEventManager::KeyEvent";
   frame_->GetChromeClient().ClearToolTip(*frame_);
 
   if (initial_key_event.windows_key_code == VK_CAPITAL)
     CapsLockStateMayHaveChanged();
 
   if (scroll_manager_->MiddleClickAutoscrollInProgress()) {
     DCHECK(RuntimeEnabledFeatures::MiddleClickAutoscrollEnabled());
     // If a key is pressed while the middleClickAutoscroll is in progress then
     // we want to stop.
     if (initial_key_event.GetType() == WebInputEvent::kKeyDown ||
@@ skipping 10 matching lines @@
   if (!node)
     return WebInputEventResult::kNotHandled;
 
   // To be meaningful enough to indicate user intention, a keyboard event needs
   // - not to be a modifier event
   // - not to be a browser shortcut
   // https://crbug.com/709765
   bool is_modifier =
       Platform::Current()->IsDomKeyForModifier(initial_key_event.dom_key);
   bool is_browser_shortcut = initial_key_event.is_browser_shortcut;
+  LOG(ERROR) << "initial_key_event.GetType() " << initial_key_event.GetType();
+  LOG(ERROR) << "initial_key_event.is_browser_shortcut "
+             << initial_key_event.is_browser_shortcut;
+  LOG(ERROR) << "initial_key_event.windows_key_code "
+             << initial_key_event.windows_key_code;
+  LOG(ERROR) << "initial_key_event.dom_code " << initial_key_event.dom_code;
+  LOG(ERROR) << "initial_key_event.dom_key " << initial_key_event.dom_key;
+  LOG(ERROR) << "initial_key_event.text[0] " << initial_key_event.text[0] << " "
+             << (initial_key_event.text[0] ? ((char)initial_key_event.text[0])
+                                           : ' ');
+  LOG(ERROR) << "is_modifier " << is_modifier;
 
   std::unique_ptr<UserGestureIndicator> gesture_indicator;
   if (!is_modifier && !is_browser_shortcut) {
     gesture_indicator.reset(new UserGestureIndicator(
         UserGestureToken::Create(frame_->GetDocument())));
   }
 
   // In IE, access keys are special, they are handled after default keydown
   // processing, but cannot be canceled - this is hard to match.  On Mac OS X,
   // we process them before dispatching keydown, as the default keydown handler
@@ skipping 255 matching lines @@
   if (current_modifiers & ::cmdKey)
     modifiers |= WebInputEvent::kMetaKey;
 #else
   // TODO(crbug.com/538289): Implement on other platforms.
   return static_cast<WebInputEvent::Modifiers>(0);
 #endif
   return static_cast<WebInputEvent::Modifiers>(modifiers);
 }
 
 }  // namespace blink