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

Unified Diff: chrome/browser/extensions/chrome_extension_test_notification_observer.h

Issue 2539623002: [Extensions] Move [Chrome]ExtensionTestNotificationObserver into extensions:: (Closed)
Patch Set: Created 4 years, 1 month 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/chrome_extension_test_notification_observer.h
diff --git a/chrome/browser/extensions/chrome_extension_test_notification_observer.h b/chrome/browser/extensions/chrome_extension_test_notification_observer.h
index 2389ad98be125210292565d9337cdd0b1bff027d..3b7151741d0c7c73d54f052f65062bf32e2c8ee9 100644
--- a/chrome/browser/extensions/chrome_extension_test_notification_observer.h
+++ b/chrome/browser/extensions/chrome_extension_test_notification_observer.h
@@ -17,11 +17,12 @@ namespace content {
class BrowserContext;
}
+namespace extensions {
+
// Test helper class for observing extension-related events.
-// TODO(devlin): This should be in the extensions namespace.
class ChromeExtensionTestNotificationObserver
: public ExtensionTestNotificationObserver,
- public extensions::ExtensionActionAPI::Observer {
+ public ExtensionActionAPI::Observer {
public:
explicit ChromeExtensionTestNotificationObserver(Browser* browser);
explicit ChromeExtensionTestNotificationObserver(
@@ -46,7 +47,7 @@ class ChromeExtensionTestNotificationObserver
private:
content::BrowserContext* GetBrowserContext();
- // extensions::ExtensionActionAPI::Observer:
+ // ExtensionActionAPI::Observer:
void OnPageActionsUpdated(content::WebContents* contents) override;
Browser* browser_;
@@ -54,4 +55,6 @@ class ChromeExtensionTestNotificationObserver
DISALLOW_COPY_AND_ASSIGN(ChromeExtensionTestNotificationObserver);
};
+} // namespace extensions
+
#endif // CHROME_BROWSER_EXTENSIONS_CHROME_EXTENSION_TEST_NOTIFICATION_OBSERVER_H_

Powered by Google App Engine
This is Rietveld 408576698