Let Chrome app handle Ash accelerators first if the app is launched as a window

ash/shell_delegate.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.
 
 #ifndef ASH_SHELL_DELEGATE_H_
 #define ASH_SHELL_DELEGATE_H_
 #pragma once
 
 #include <vector>
 
@@ skipping 41 matching lines @@
 
   // Shuts down the environment.
   virtual void Shutdown() = 0;
 
   // Invoked when the user uses Ctrl-Shift-Q to close chrome.
   virtual void Exit() = 0;
 
   // Invoked when the user uses Ctrl-N or Ctrl-Shift-N to open a new window.
   virtual void NewWindow(bool incognito) = 0;
 
+  // Returns true if |window| is one of the browser windows.
+  virtual bool IsBrowserWindow(const aura::Window* window) = 0;

[Ben Goodger (Google), 2012/05/01 16:00:37]

IsWebContentsView()

[Yusuke Sato, 2012/05/02 16:40:11]

I've added code directly to accelerator_filter.cc which checks if an
aura::Window is for a Web contents view. Please let me keep the interface as-is
since I also need a way to know whether the |window| is a non-App browser window
or not.

[Ben Goodger (Google), 2012/05/03 17:29:38]

Why do you need to know if it is a browser window?

[Yusuke Sato, 2012/05/03 17:45:59]

This is because when it is a (non-app) browser window, an Ash accelerator such
as Ctrl+N and F6-F10 has to be processed in Ash before the accelerator is passed
to the browser.

+
   // Invoked when the user presses the Search key.
   virtual void Search() = 0;
 
   // Invoked when the user uses Ctrl-M to open file manager.
   virtual void OpenFileManager() = 0;
 
   // Invoked when the user opens Crosh.
   virtual void OpenCrosh() = 0;
 
   // Invoked when the user needs to set up mobile networking.
@@ skipping 15 matching lines @@
   // Creates a system-tray delegate. Shell takes ownership of the delegate.
   virtual SystemTrayDelegate* CreateSystemTrayDelegate(SystemTray* tray) = 0;
 
   // Creates a user wallpaper delegate. Shell takes ownership of the delegate.
   virtual UserWallpaperDelegate* CreateUserWallpaperDelegate() = 0;
 };
 
 }  // namespace ash
 
 #endif  // ASH_SHELL_DELEGATE_H_