Avoid toggling app list if interrupted by mouse

ui/base/accelerators/accelerator.h

 // Copyright (c) 2012 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.
 
 // This class describe a keyboard accelerator (or keyboard shortcut).
 // Keyboard accelerators are registered with the FocusManager.
 // It has a copy constructor and assignment operator so that it can be copied.
 // It also defines the < operator so that it can be used as a key in a std::map.
 //
 
@@ skipping 71 matching lines @@
 
   void set_platform_accelerator(std::unique_ptr<PlatformAccelerator> p) {
     platform_accelerator_ = std::move(p);
   }
 
   // This class keeps ownership of the returned object.
   const PlatformAccelerator* platform_accelerator() const {
     return platform_accelerator_.get();
   }
 
+  void set_interrupted_by_mouse_event(bool interrupted_by_mouse_event) {
+    interrupted_by_mouse_event_ = interrupted_by_mouse_event;
+  }
+
+  bool is_interrupted_by_mouse_event() const {

[oshima, 2017/07/07 22:28:41]

nit: interrupted_by_mouse_event()

(or you can rename the variable. whichever you prefer)

it's just style: accessor should have the same name without _.

[weidongg, 2017/07/07 22:55:31]

Done.

+    return interrupted_by_mouse_event_;
+  }
+
  private:
   // The keycode (VK_...).
   KeyboardCode key_code_;
 
   KeyState key_state_;
 
   // The state of the Shift/Ctrl/Alt keys. This corresponds to Event::flags().
   int modifiers_;
 
   // Stores platform specific data. May be NULL.
   // TODO: this is only used in Mac code and should be removed from here.
   // http://crbug.com/702823.
   std::unique_ptr<PlatformAccelerator> platform_accelerator_;
+
+  // Whether the accelerator is interrupted by a mouse press/release. This is
+  // optionally used by AcceleratorController. Even this is set to true, the
+  // accelerator may still be handled successfully. (Currently only
+  // TOGGLE_APP_LIST is disabled when mouse press/release occurs between
+  // search key down and up. See crbug.com/665897)
+  bool interrupted_by_mouse_event_;
 };
 
 // An interface that classes that want to register for keyboard accelerators
 // should implement.
 class UI_BASE_EXPORT AcceleratorTarget {
  public:
   // Should return true if the accelerator was processed.
   virtual bool AcceleratorPressed(const Accelerator& accelerator) = 0;
 
   // Should return true if the target can handle the accelerator events. The
@@ skipping 15 matching lines @@
   virtual bool GetAcceleratorForCommandId(int command_id,
                                           Accelerator* accelerator) const = 0;
 
  protected:
   virtual ~AcceleratorProvider() {}
 };
 
 }  // namespace ui
 
 #endif  // UI_BASE_ACCELERATORS_ACCELERATOR_H_