[Mac] Add text-editing accelerators to the User Manager.

chrome/browser/ui/cocoa/profiles/user_manager_mac.mm

 // 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.
 
 #include "chrome/browser/ui/cocoa/profiles/user_manager_mac.h"
 
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/browser_dialogs.h"
@@ skipping 37 matching lines @@
     : window_(window) {}
 
   // WebContentsDelegate implementation. Forwards all unhandled keyboard events
   // to the current window.
   virtual void HandleKeyboardEvent(
       content::WebContents* source,
       const content::NativeWebKeyboardEvent& event) OVERRIDE {
     if (![BrowserWindowUtils shouldHandleKeyboardEvent:event])
       return;
 
+    // -getCommandId returns -1 if the event isn't a chrome keyboard shortcut.
     int commandId = [BrowserWindowUtils getCommandId:event];
 
-    // Since the User Manager is a "top level" window, only handle close events.
-    if (commandId == IDC_CLOSE_WINDOW || commandId == IDC_EXIT) {
+    // From the chrome accelerators, only handle close window accelerators, and
+    // allow all other accelerators to be handled by the web contents.
+    if (commandId == -1 || commandId == IDC_CLOSE_WINDOW ||

[groby-ooo-7-16, 2014/09/10 16:38:21]

No support for IDC_CLOSE_TAB? (which is cmd-w. CLOSE_WINDOW is cmd-shift-w,
IIRC)

[noms (inactive), 2014/09/10 16:55:09]

According to
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/ui/...
It looks like Cmd+W. (Also, Cmd+W works, Cmd+Shit+W doesn't work in the User
Manager)

On 2014/09/10 16:38:21, groby wrote:

[groby-ooo-7-16, 2014/09/10 20:06:40]

Sigh. It turns out that the "Close window" entry in the OSX menu doesn't have a
command attached -
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/ui/...

Cmd-W is definitely "Close Tab", but you're right, it's mapped to
IDC_CLOSE_WINDOW. Why that works is beyond me...

+        commandId == IDC_EXIT) {
       // Not invoking +[BrowserWindowUtils handleKeyboardEvent here], since the
       // window in question is a ConstrainedWindowCustomWindow, not a
       // BrowserWindow.
       [window_ redispatchKeyEvent:event.os_event];
     }
   }
 
  private:
   ChromeEventProcessingWindow* window_;  // Used to redispatch key events.
 };
@@ skipping 134 matching lines @@
 void UserManagerMac::OnGuestProfileCreated(Profile* guest_profile,
                                            const std::string& url) {
   instance_ = new UserManagerMac(guest_profile);
   [instance_->window_controller_ showURL:GURL(url)];
 }
 
 void UserManagerMac::WindowWasClosed() {
   instance_ = NULL;
   delete this;
 }