Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(134)

Side by Side Diff: chrome/browser/extensions/extension_view_host_mac.h

Issue 363233002: Abstract base 'ExtensionView' to Fix DEPS violation in extension_view_host.h (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Init() must be done after assigning Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_EXTENSIONS_EXTENSION_VIEW_HOST_MAC_H_ 5 #ifndef CHROME_BROWSER_EXTENSIONS_EXTENSION_VIEW_HOST_MAC_H_
6 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_VIEW_HOST_MAC_H_ 6 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_VIEW_HOST_MAC_H_
7 7
8 #include "base/basictypes.h" 8 #include "base/basictypes.h"
9 #include "base/compiler_specific.h" 9 #include "base/compiler_specific.h"
10 #include "chrome/browser/extensions/extension_view_host.h" 10 #include "chrome/browser/extensions/extension_view_host.h"
11 11
12 namespace extensions { 12 namespace extensions {
13 class Extension; 13 class Extension;
14 14
15 // TODO(mpcomplete): I don't know what this does or if it is needed anymore, 15 // TODO(mpcomplete): I don't know what this does or if it is needed anymore,
16 // now that ExtensionHost is restructured to rely on WebContents. 16 // now that ExtensionHost is restructured to rely on WebContents.
17 class ExtensionViewHostMac : public ExtensionViewHost { 17 class ExtensionViewHostMac : public ExtensionViewHost {
18 public: 18 public:
19 ExtensionViewHostMac(const Extension* extension, 19 ExtensionViewHostMac(const Extension* extension,
20 content::SiteInstance* site_instance, 20 content::SiteInstance* site_instance,
21 const GURL& url, 21 const GURL& url,
22 ViewType host_type) : 22 ViewType host_type)
23 ExtensionViewHost(extension, site_instance, url, host_type) {} 23 : ExtensionViewHost(extension, site_instance, url, host_type) {}
24 virtual ~ExtensionViewHostMac(); 24 virtual ~ExtensionViewHostMac();
25 25
26 private: 26 private:
27 // ExtensionViewHost overrides:
28 virtual void UnhandledKeyboardEvent(
29 content::WebContents* source,
30 const content::NativeWebKeyboardEvent& event) OVERRIDE;
31
32 DISALLOW_COPY_AND_ASSIGN(ExtensionViewHostMac); 27 DISALLOW_COPY_AND_ASSIGN(ExtensionViewHostMac);
33 }; 28 };
34 29
35 } // namespace extensions 30 } // namespace extensions
36 31
37 #endif // CHROME_BROWSER_EXTENSIONS_EXTENSION_VIEW_HOST_MAC_H_ 32 #endif // CHROME_BROWSER_EXTENSIONS_EXTENSION_VIEW_HOST_MAC_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698