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

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

Issue 1991083002: Remove ExtensionFunction::SetResult(T*) overload. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: IWYU Created 4 years, 7 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
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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_BROWSER_EXTENSION_WINDOW_CONTROLLER_H_ 5 #ifndef CHROME_BROWSER_EXTENSIONS_BROWSER_EXTENSION_WINDOW_CONTROLLER_H_
6 #define CHROME_BROWSER_EXTENSIONS_BROWSER_EXTENSION_WINDOW_CONTROLLER_H_ 6 #define CHROME_BROWSER_EXTENSIONS_BROWSER_EXTENSION_WINDOW_CONTROLLER_H_
7 7
8 #include "base/macros.h" 8 #include "base/macros.h"
9 #include "chrome/browser/extensions/window_controller.h" 9 #include "chrome/browser/extensions/window_controller.h"
10 10
11 class Browser; 11 class Browser;
12 12
13 namespace extensions { 13 namespace extensions {
14 class Extension; 14 class Extension;
15 } 15 }
16 16
17 class BrowserExtensionWindowController : public extensions::WindowController { 17 class BrowserExtensionWindowController : public extensions::WindowController {
18 public: 18 public:
19 explicit BrowserExtensionWindowController(Browser* browser); 19 explicit BrowserExtensionWindowController(Browser* browser);
20 ~BrowserExtensionWindowController() override; 20 ~BrowserExtensionWindowController() override;
21 21
22 // extensions::WindowController implementation. 22 // extensions::WindowController implementation.
23 int GetWindowId() const override; 23 int GetWindowId() const override;
24 std::string GetWindowTypeText() const override; 24 std::string GetWindowTypeText() const override;
25 base::DictionaryValue* CreateWindowValue() const override; 25 std::unique_ptr<base::DictionaryValue> CreateWindowValueWithTabs(
26 base::DictionaryValue* CreateWindowValueWithTabs(
27 const extensions::Extension* extension) const override; 26 const extensions::Extension* extension) const override;
28 base::DictionaryValue* CreateTabValue(const extensions::Extension* extension, 27 base::DictionaryValue* CreateTabValue(const extensions::Extension* extension,
29 int tab_index) const override; 28 int tab_index) const override;
30 std::unique_ptr<extensions::api::tabs::Tab> CreateTabObject( 29 std::unique_ptr<extensions::api::tabs::Tab> CreateTabObject(
31 const extensions::Extension* extension, 30 const extensions::Extension* extension,
32 int tab_index) const override; 31 int tab_index) const override;
33 bool CanClose(Reason* reason) const override; 32 bool CanClose(Reason* reason) const override;
34 void SetFullscreenMode(bool is_fullscreen, 33 void SetFullscreenMode(bool is_fullscreen,
35 const GURL& extension_url) const override; 34 const GURL& extension_url) const override;
36 Browser* GetBrowser() const override; 35 Browser* GetBrowser() const override;
37 bool IsVisibleToExtension( 36 bool IsVisibleToExtension(
38 const extensions::Extension* extension) const override; 37 const extensions::Extension* extension) const override;
39 38
40 private: 39 private:
41 Browser* browser_; 40 Browser* browser_;
42 41
43 DISALLOW_COPY_AND_ASSIGN(BrowserExtensionWindowController); 42 DISALLOW_COPY_AND_ASSIGN(BrowserExtensionWindowController);
44 }; 43 };
45 44
46 #endif // CHROME_BROWSER_EXTENSIONS_BROWSER_EXTENSION_WINDOW_CONTROLLER_H_ 45 #endif // CHROME_BROWSER_EXTENSIONS_BROWSER_EXTENSION_WINDOW_CONTROLLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698