Use NULL for ExtensionKeyBindingRegistry::Delegate for app windows.

This avoids forcing AppWindow to be a delegate that does nothing.

Review URL: https://codereview.chromium.org/180633012

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@254843 0039d316-1c4b-4281-b951-d872f2087c98

4 files changed, 2 insertions, 13 deletions

diff --git a/apps/DEPS b/apps/DEPS
index 814847a..ad5183e 100644
--- a/apps/DEPS
+++ b/apps/DEPS
@@ -31,7 +31,6 @@ include_rules = [
   "+chrome/browser/extensions/extension_function_dispatcher.h",
   "+chrome/browser/extensions/extension_icon_image.h",
   "+chrome/browser/extensions/extension_host.h",
-  "+chrome/browser/extensions/extension_keybinding_registry.h",
   "+chrome/browser/extensions/extension_web_contents_observer.h",
   "+chrome/browser/extensions/suggest_permission_util.h",
   "+chrome/browser/extensions/unpacked_installer.h",
diff --git a/apps/app_window.cc b/apps/app_window.cc
index 57047b2..8bb2b7e 100644
--- a/apps/app_window.cc
+++ b/apps/app_window.cc
@@ -831,12 +831,6 @@ bool AppWindow::IsWebContentsVisible(content::WebContents* web_contents) {
   return delegate_->IsWebContentsVisible(web_contents);
 }
 
-extensions::ActiveTabPermissionGranter*
-AppWindow::GetActiveTabPermissionGranter() {
-  // App windows don't support the activeTab permission.
-  return NULL;
-}
-
 WebContentsModalDialogHost* AppWindow::GetWebContentsModalDialogHost() {
   return native_app_window_.get();
 }
diff --git a/apps/app_window.h b/apps/app_window.h
index 8403153..2614a40 100644
--- a/apps/app_window.h
+++ b/apps/app_window.h
@@ -8,7 +8,6 @@
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "chrome/browser/extensions/extension_icon_image.h"
-#include "chrome/browser/extensions/extension_keybinding_registry.h"
 #include "chrome/browser/sessions/session_id.h"
 #include "components/web_modal/web_contents_modal_dialog_manager_delegate.h"
 #include "content/public/browser/notification_observer.h"
@@ -82,7 +81,6 @@ class AppWindow : public content::NotificationObserver,
                   public content::WebContentsDelegate,
                   public content::WebContentsObserver,
                   public web_modal::WebContentsModalDialogManagerDelegate,
-                  public extensions::ExtensionKeybindingRegistry::Delegate,
                   public extensions::IconImage::Observer {
  public:
   enum WindowType {
@@ -424,10 +422,6 @@ class AppWindow : public content::NotificationObserver,
   // Update the always-on-top bit in the native app window.
   void UpdateNativeAlwaysOnTop();
 
-  // extensions::ExtensionKeybindingRegistry::Delegate implementation.
-  virtual extensions::ActiveTabPermissionGranter*
-      GetActiveTabPermissionGranter() OVERRIDE;
-
   // web_modal::WebContentsModalDialogManagerDelegate implementation.
   virtual web_modal::WebContentsModalDialogHost* GetWebContentsModalDialogHost()
       OVERRIDE;
diff --git a/apps/app_window_contents.cc b/apps/app_window_contents.cc
index 8fa3c46..d50db9e 100644
--- a/apps/app_window_contents.cc
+++ b/apps/app_window_contents.cc
@@ -10,6 +10,8 @@
 #include "chrome/common/extensions/extension_messages.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_thread.h"
+#include "content/public/browser/notification_details.h"
+#include "content/public/browser/notification_source.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/resource_dispatcher_host.h"