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

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

Issue 287623005: Cleanup: Remove some unneeded Extension* class forward declarations. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: pass presubmit Created 6 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 | Annotate | Revision Log
Property Changes:
Added: svn:eol-style
+ LF
OLDNEW
1 // ::Copyright (c) 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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_SETTINGS_API_BUBBLE_CONTROLLER_H_ 5 #ifndef CHROME_BROWSER_EXTENSIONS_SETTINGS_API_BUBBLE_CONTROLLER_H_
6 #define CHROME_BROWSER_EXTENSIONS_SETTINGS_API_BUBBLE_CONTROLLER_H_ 6 #define CHROME_BROWSER_EXTENSIONS_SETTINGS_API_BUBBLE_CONTROLLER_H_
7 7
8 #include <string> 8 #include <string>
9 #include "chrome/browser/extensions/extension_message_bubble_controller.h" 9 #include "chrome/browser/extensions/extension_message_bubble_controller.h"
10 #include "chrome/common/extensions/manifest_handlers/settings_overrides_handler. h" 10 #include "chrome/common/extensions/manifest_handlers/settings_overrides_handler. h"
11 11
12 class ExtensionService;
13
14 namespace extensions { 12 namespace extensions {
15 13
16 class SettingsApiBubble; 14 class SettingsApiBubble;
17 15
18 class SettingsApiBubbleController : public ExtensionMessageBubbleController { 16 class SettingsApiBubbleController : public ExtensionMessageBubbleController {
19 public: 17 public:
20 SettingsApiBubbleController(Profile* profile, SettingsApiOverrideType type); 18 SettingsApiBubbleController(Profile* profile, SettingsApiOverrideType type);
21 virtual ~SettingsApiBubbleController(); 19 virtual ~SettingsApiBubbleController();
22 20
23 // Whether the controller knows that we should show the bubble for extension 21 // Whether the controller knows that we should show the bubble for extension
24 // with |extension_id|. Returns true if so. 22 // with |extension_id|. Returns true if so.
25 bool ShouldShow(const std::string& extension_id); 23 bool ShouldShow(const std::string& extension_id);
26 24
27 // ExtensionMessageBubbleController: 25 // ExtensionMessageBubbleController:
28 virtual bool CloseOnDeactivate() OVERRIDE; 26 virtual bool CloseOnDeactivate() OVERRIDE;
29 27
30 private: 28 private:
31 // A weak pointer to the profile we are associated with. Not owned by us. 29 // A weak pointer to the profile we are associated with. Not owned by us.
32 Profile* profile_; 30 Profile* profile_;
33 31
34 // The type of settings override this bubble will report on. 32 // The type of settings override this bubble will report on.
35 SettingsApiOverrideType type_; 33 SettingsApiOverrideType type_;
36 34
37 DISALLOW_COPY_AND_ASSIGN(SettingsApiBubbleController); 35 DISALLOW_COPY_AND_ASSIGN(SettingsApiBubbleController);
38 }; 36 };
39 37
40 } // namespace extensions 38 } // namespace extensions
41 39
42 #endif // CHROME_BROWSER_EXTENSIONS_SETTINGS_API_BUBBLE_CONTROLLER_H_ 40 #endif // CHROME_BROWSER_EXTENSIONS_SETTINGS_API_BUBBLE_CONTROLLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698