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

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

Issue 624153002: replace OVERRIDE and FINAL with override and final in chrome/browser/extensions/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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 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_ERROR_CONSOLE_ERROR_CONSOLE_H_ 5 #ifndef CHROME_BROWSER_EXTENSIONS_ERROR_CONSOLE_ERROR_CONSOLE_H_
6 #define CHROME_BROWSER_EXTENSIONS_ERROR_CONSOLE_ERROR_CONSOLE_H_ 6 #define CHROME_BROWSER_EXTENSIONS_ERROR_CONSOLE_ERROR_CONSOLE_H_
7 7
8 #include "base/memory/scoped_ptr.h" 8 #include "base/memory/scoped_ptr.h"
9 #include "base/observer_list.h" 9 #include "base/observer_list.h"
10 #include "base/prefs/pref_change_registrar.h" 10 #include "base/prefs/pref_change_registrar.h"
(...skipping 111 matching lines...) Expand 10 before | Expand all | Expand 10 after
122 // Called when the Developer Mode preference is changed; this is important 122 // Called when the Developer Mode preference is changed; this is important
123 // since we use this as a heuristic to determine if the console is enabled or 123 // since we use this as a heuristic to determine if the console is enabled or
124 // not. 124 // not.
125 void OnPrefChanged(); 125 void OnPrefChanged();
126 126
127 // ExtensionRegistry implementation. If the Apps Developer Tools app is 127 // ExtensionRegistry implementation. If the Apps Developer Tools app is
128 // installed or uninstalled, we may need to turn the ErrorConsole on/off. 128 // installed or uninstalled, we may need to turn the ErrorConsole on/off.
129 virtual void OnExtensionUnloaded( 129 virtual void OnExtensionUnloaded(
130 content::BrowserContext* browser_context, 130 content::BrowserContext* browser_context,
131 const Extension* extension, 131 const Extension* extension,
132 UnloadedExtensionInfo::Reason reason) OVERRIDE; 132 UnloadedExtensionInfo::Reason reason) override;
133 virtual void OnExtensionLoaded(content::BrowserContext* browser_context, 133 virtual void OnExtensionLoaded(content::BrowserContext* browser_context,
134 const Extension* extension) OVERRIDE; 134 const Extension* extension) override;
135 virtual void OnExtensionInstalled(content::BrowserContext* browser_context, 135 virtual void OnExtensionInstalled(content::BrowserContext* browser_context,
136 const Extension* extension, 136 const Extension* extension,
137 bool is_update) OVERRIDE; 137 bool is_update) override;
138 virtual void OnExtensionUninstalled( 138 virtual void OnExtensionUninstalled(
139 content::BrowserContext* browser_context, 139 content::BrowserContext* browser_context,
140 const Extension* extension, 140 const Extension* extension,
141 extensions::UninstallReason reason) OVERRIDE; 141 extensions::UninstallReason reason) override;
142 142
143 // Add manifest errors from an extension's install warnings. 143 // Add manifest errors from an extension's install warnings.
144 void AddManifestErrorsForExtension(const Extension* extension); 144 void AddManifestErrorsForExtension(const Extension* extension);
145 145
146 // content::NotificationObserver implementation. 146 // content::NotificationObserver implementation.
147 virtual void Observe(int type, 147 virtual void Observe(int type,
148 const content::NotificationSource& source, 148 const content::NotificationSource& source,
149 const content::NotificationDetails& details) OVERRIDE; 149 const content::NotificationDetails& details) override;
150 150
151 // Returns the applicable bit mask of reporting preferences for the extension. 151 // Returns the applicable bit mask of reporting preferences for the extension.
152 int GetMaskForExtension(const std::string& extension_id) const; 152 int GetMaskForExtension(const std::string& extension_id) const;
153 153
154 // Whether or not the error console should record errors. This is true if 154 // Whether or not the error console should record errors. This is true if
155 // the user is in developer mode, and at least one of the following is true: 155 // the user is in developer mode, and at least one of the following is true:
156 // - The Chrome Apps Developer Tools are installed. 156 // - The Chrome Apps Developer Tools are installed.
157 // - FeatureSwitch::error_console() is enabled. 157 // - FeatureSwitch::error_console() is enabled.
158 // - This is a Dev Channel release. 158 // - This is a Dev Channel release.
159 bool enabled_; 159 bool enabled_;
(...skipping 25 matching lines...) Expand all
185 185
186 ScopedObserver<ExtensionRegistry, ExtensionRegistryObserver> 186 ScopedObserver<ExtensionRegistry, ExtensionRegistryObserver>
187 registry_observer_; 187 registry_observer_;
188 188
189 DISALLOW_COPY_AND_ASSIGN(ErrorConsole); 189 DISALLOW_COPY_AND_ASSIGN(ErrorConsole);
190 }; 190 };
191 191
192 } // namespace extensions 192 } // namespace extensions
193 193
194 #endif // CHROME_BROWSER_EXTENSIONS_ERROR_CONSOLE_ERROR_CONSOLE_H_ 194 #endif // CHROME_BROWSER_EXTENSIONS_ERROR_CONSOLE_ERROR_CONSOLE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698