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

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

Issue 11419224: Add missing (and remove superfluous) 'explicit' from constructors. Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase + remove non-straightforward changes Created 7 years, 11 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 (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_SCRIPT_BADGE_CONTROLLER_H_ 5 #ifndef CHROME_BROWSER_EXTENSIONS_SCRIPT_BADGE_CONTROLLER_H_
6 #define CHROME_BROWSER_EXTENSIONS_SCRIPT_BADGE_CONTROLLER_H_ 6 #define CHROME_BROWSER_EXTENSIONS_SCRIPT_BADGE_CONTROLLER_H_
7 7
8 #include <map> 8 #include <map>
9 #include <set> 9 #include <set>
10 #include <string> 10 #include <string>
(...skipping 28 matching lines...) Expand all
39 // 39 //
40 // When extension IDs are recorded a NOTIFICATION_EXTENSION_LOCATION_BAR_UPDATED 40 // When extension IDs are recorded a NOTIFICATION_EXTENSION_LOCATION_BAR_UPDATED
41 // is sent, and those extensions will be returned from GetCurrentActions until 41 // is sent, and those extensions will be returned from GetCurrentActions until
42 // the next page navigation. 42 // the next page navigation.
43 class ScriptBadgeController 43 class ScriptBadgeController
44 : public LocationBarController, 44 : public LocationBarController,
45 public TabHelper::ScriptExecutionObserver, 45 public TabHelper::ScriptExecutionObserver,
46 public content::WebContentsObserver, 46 public content::WebContentsObserver,
47 public content::NotificationObserver { 47 public content::NotificationObserver {
48 public: 48 public:
49 explicit ScriptBadgeController(content::WebContents* web_contents, 49 ScriptBadgeController(content::WebContents* web_contents,
50 TabHelper* tab_helper); 50 TabHelper* tab_helper);
51 virtual ~ScriptBadgeController(); 51 virtual ~ScriptBadgeController();
52 52
53 // LocationBarController implementation. 53 // LocationBarController implementation.
54 virtual std::vector<ExtensionAction*> GetCurrentActions() const OVERRIDE; 54 virtual std::vector<ExtensionAction*> GetCurrentActions() const OVERRIDE;
55 virtual void GetAttentionFor(const std::string& extension_id) OVERRIDE; 55 virtual void GetAttentionFor(const std::string& extension_id) OVERRIDE;
56 virtual Action OnClicked(const std::string& extension_id, 56 virtual Action OnClicked(const std::string& extension_id,
57 int mouse_button) OVERRIDE; 57 int mouse_button) OVERRIDE;
58 virtual void NotifyChange() OVERRIDE; 58 virtual void NotifyChange() OVERRIDE;
59 59
60 // TabHelper::ScriptExecutionObserver implementation. 60 // TabHelper::ScriptExecutionObserver implementation.
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
101 101
102 // Listen to extension unloaded notifications. 102 // Listen to extension unloaded notifications.
103 content::NotificationRegistrar registrar_; 103 content::NotificationRegistrar registrar_;
104 104
105 DISALLOW_COPY_AND_ASSIGN(ScriptBadgeController); 105 DISALLOW_COPY_AND_ASSIGN(ScriptBadgeController);
106 }; 106 };
107 107
108 } // namespace extensions 108 } // namespace extensions
109 109
110 #endif // CHROME_BROWSER_EXTENSIONS_SCRIPT_BADGE_CONTROLLER_H_ 110 #endif // CHROME_BROWSER_EXTENSIONS_SCRIPT_BADGE_CONTROLLER_H_
OLDNEW
« no previous file with comments | « chrome/browser/extensions/extension_function_registry.h ('k') | chrome/browser/geolocation/chrome_access_token_store.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698