MacViews: Fix GlobalKeyboardShortcutsTest.SwitchTabsMac.

chrome/browser/global_keyboard_shortcuts_mac_browsertest.mm

 // Copyright 2015 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.
 
 #import "chrome/browser/global_keyboard_shortcuts_mac.h"
 
 #import <Cocoa/Cocoa.h>
 
 #include "base/run_loop.h"
 #include "chrome/browser/extensions/extension_browsertest.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_commands.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
+#include "ui/base/accelerators/accelerator.h"

[tapted, 2016/11/30 23:53:03]

(oh, and a nit: is this still needed?)

[themblsha, 2016/12/01 11:24:17]

Whoops. I wanted to use ui::Accelerator, but converting between different types
of modifiers turned out to be too cumbersome.

 #import "ui/events/test/cocoa_test_event_utils.h"
 
 using cocoa_test_event_utils::SynthesizeKeyEvent;
 
 using GlobalKeyboardShortcutsTest = ExtensionBrowserTest;
 
+namespace {
+
+void ActivateAccelerator(NSWindow* window, NSEvent* ns_event) {
+  if ([window performKeyEquivalent:ns_event])
+    return;
+
+  // crbug.com/665823: In MacViews not all accelerators are processed at
+  // -performKeyEquivalent: stage, we also need to invoke -sendEvent:.

[tapted, 2016/11/30 23:52:03]

Thanks! this is much clearer. I would add a bit more like,

  // This is consistent with the way AppKit dispatches events when
  // -performKeyEquivalent: returns NO. See "The Path of Key Events" in the
  // Cocoa Event Architecture documentation.



(I would not include the URL -

https://developer.apple.com/library/content/documentation/Cocoa/Conceptual/Ev...

- it's too long and Apple keep changing links on their developer site).

[themblsha, 2016/12/01 11:24:17]

Done.

+  [window sendEvent:ns_event];
+}
+
+}  // namespace
+
 // Test that global keyboard shortcuts are handled by the native window.
 IN_PROC_BROWSER_TEST_F(GlobalKeyboardShortcutsTest, SwitchTabsMac) {
   NSWindow* ns_window = browser()->window()->GetNativeWindow();
   TabStripModel* tab_strip = browser()->tab_strip_model();
 
   // Set up window with 2 tabs.
   chrome::NewTab(browser());
   EXPECT_EQ(2, tab_strip->count());
   EXPECT_TRUE(tab_strip->IsTabSelected(1));
 
   // Ctrl+Tab goes to the next tab, which loops back to the first tab.
-  [ns_window performKeyEquivalent:SynthesizeKeyEvent(
-      ns_window, true, ui::VKEY_TAB, NSControlKeyMask)];
+  ActivateAccelerator(
+      ns_window,
+      SynthesizeKeyEvent(ns_window, true, ui::VKEY_TAB, NSControlKeyMask));
   EXPECT_TRUE(tab_strip->IsTabSelected(0));
 
   // Cmd+2 goes to the second tab.
-  [ns_window performKeyEquivalent:SynthesizeKeyEvent(
-      ns_window, true, ui::VKEY_2, NSCommandKeyMask)];
+  ActivateAccelerator(ns_window, SynthesizeKeyEvent(ns_window, true, ui::VKEY_2,
+                                                    NSCommandKeyMask));
   EXPECT_TRUE(tab_strip->IsTabSelected(1));
 
   // Cmd+{ goes to the previous tab.
-  [ns_window performKeyEquivalent:SynthesizeKeyEvent(
-      ns_window, true, ui::VKEY_OEM_4, NSShiftKeyMask | NSCommandKeyMask)];
+  ActivateAccelerator(ns_window,
+                      SynthesizeKeyEvent(ns_window, true, ui::VKEY_OEM_4,
+                                         NSShiftKeyMask | NSCommandKeyMask));
   EXPECT_TRUE(tab_strip->IsTabSelected(0));
 }