MacViews: Fix accelerator handling while Omnibox is in focus.

ui/events/test/event_generator.h

 // Copyright 2014 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.
 
 #ifndef UI_EVENTS_TEST_EVENT_GENERATOR_H_
 #define UI_EVENTS_TEST_EVENT_GENERATOR_H_
 
 #include <list>
 #include <memory>
 #include <vector>
@@ skipping 120 matching lines @@
   // various methods that take a location but can be manipulated directly,
   // typically for touch.
   void set_current_location(const gfx::Point& location) {
     current_location_ = location;
   }
   const gfx::Point& current_location() const { return current_location_; }
 
   void set_async(bool async) { async_ = async; }
   bool async() const { return async_; }
 
-  // Dispatch events through the application instead of directly to the
-  // target window. Currently only supported on Mac.
-  void set_targeting_application(bool targeting_application) {
-    targeting_application_ = targeting_application;
+  // Events could be dispatched using different methods.

[tapted, 2016/11/18 07:28:04]

I'd add a sentence like

"The choice is a tradeoff between test robustness and coverage of OS internals
that affect event dispatch."

[themblsha, 2016/11/18 13:31:50]

Done.

+  // Currently only supported on Mac.
+  enum class Target {
+    // Dispatch through the application.

[tapted, 2016/11/18 07:28:04]

"Least robust."

[themblsha, 2016/11/18 13:31:50]

Done.

+    APPLICATION,
+    // Dispatch directly to target window.
+    WINDOW,
+    // Default. Emulates default Window dispatch.

[tapted, 2016/11/18 07:28:04]

"Most robust."

[themblsha, 2016/11/18 13:31:50]

Done.

+    WIDGET,
+  };
+
+  // Selects dispatch method. Currently only supported on Mac.
+  void set_target(Target target) {
+    target_ = target;
   }
-  bool targeting_application() const { return targeting_application_; }
+  Target target() const { return target_; }
 
   // Resets the event flags bitmask.
   void set_flags(int flags) { flags_ = flags; }
   int flags() const { return flags_; }
 
   // Generates a left button press event.
   void PressLeftButton();
 
   // Generates a left button release event.
   void ReleaseLeftButton();
@@ skipping 241 matching lines @@
 
   std::unique_ptr<EventGeneratorDelegate> delegate_;
   gfx::Point current_location_;
   EventTarget* current_target_;
   int flags_;
   bool grab_;
   bool pen_pointer_mode_ = false;
   std::list<std::unique_ptr<Event>> pending_events_;
   // Set to true to cause events to be posted asynchronously.
   bool async_;
-  bool targeting_application_;
+  Target target_;
   std::unique_ptr<base::TickClock> tick_clock_;
 
   DISALLOW_COPY_AND_ASSIGN(EventGenerator);
 };
 
 }  // namespace test
 }  // namespace ui
 
 #endif  // UI_EVENTS_TEST_EVENT_GENERATOR_H_