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

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

Issue 207613002: Cleanup: Remove NOTIFICATION_EXTENSION_UNLOADED usage in MediaScanManager. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: rebase, add error console Created 6 years, 8 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_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 <deque> 8 #include <deque>
9 #include <map> 9 #include <map>
10 10
(...skipping 107 matching lines...) Expand 10 before | Expand all | Expand 10 after
118 // removing all current errors. 118 // removing all current errors.
119 void Disable(); 119 void Disable();
120 120
121 // Called when the Developer Mode preference is changed; this is important 121 // Called when the Developer Mode preference is changed; this is important
122 // since we use this as a heuristic to determine if the console is enabled or 122 // since we use this as a heuristic to determine if the console is enabled or
123 // not. 123 // not.
124 void OnPrefChanged(); 124 void OnPrefChanged();
125 125
126 // ExtensionRegistry implementation. If the Apps Developer Tools app is 126 // ExtensionRegistry implementation. If the Apps Developer Tools app is
127 // installed or uninstalled, we may need to turn the ErrorConsole on/off. 127 // installed or uninstalled, we may need to turn the ErrorConsole on/off.
128 virtual void OnExtensionUnloaded(const Extension* extension) OVERRIDE; 128 virtual void OnExtensionUnloaded(content::BrowserContext* browser_context,
129 virtual void OnExtensionLoaded(const Extension* extension) OVERRIDE; 129 const Extension* extension) OVERRIDE;
130 virtual void OnExtensionLoaded(content::BrowserContext* browser_context,
131 const Extension* extension) OVERRIDE;
130 132
131 // Add manifest errors from an extension's install warnings. 133 // Add manifest errors from an extension's install warnings.
132 void AddManifestErrorsForExtension(const Extension* extension); 134 void AddManifestErrorsForExtension(const Extension* extension);
133 135
134 // content::NotificationObserver implementation. 136 // content::NotificationObserver implementation.
135 virtual void Observe(int type, 137 virtual void Observe(int type,
136 const content::NotificationSource& source, 138 const content::NotificationSource& source,
137 const content::NotificationDetails& details) OVERRIDE; 139 const content::NotificationDetails& details) OVERRIDE;
138 140
139 // Whether or not the error console should record errors. This is true if 141 // Whether or not the error console should record errors. This is true if
(...skipping 28 matching lines...) Expand all
168 170
169 ScopedObserver<ExtensionRegistry, ExtensionRegistryObserver> 171 ScopedObserver<ExtensionRegistry, ExtensionRegistryObserver>
170 registry_observer_; 172 registry_observer_;
171 173
172 DISALLOW_COPY_AND_ASSIGN(ErrorConsole); 174 DISALLOW_COPY_AND_ASSIGN(ErrorConsole);
173 }; 175 };
174 176
175 } // namespace extensions 177 } // namespace extensions
176 178
177 #endif // CHROME_BROWSER_EXTENSIONS_ERROR_CONSOLE_ERROR_CONSOLE_H_ 179 #endif // CHROME_BROWSER_EXTENSIONS_ERROR_CONSOLE_ERROR_CONSOLE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698