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

Unified Diff: extensions/browser/app_window/app_window_registry.h

Issue 664933004: Standardize usage of virtual/override/final in 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 side-by-side diff with in-line comments
Download patch
Index: extensions/browser/app_window/app_window_registry.h
diff --git a/extensions/browser/app_window/app_window_registry.h b/extensions/browser/app_window/app_window_registry.h
index 2ae6a46b85955822a17f626371f6722dc341846e..7037953c4e93de9ae19c18b263698cf7748709f1 100644
--- a/extensions/browser/app_window/app_window_registry.h
+++ b/extensions/browser/app_window/app_window_registry.h
@@ -55,7 +55,7 @@ class AppWindowRegistry : public KeyedService {
typedef std::set<std::string> InspectedWindowSet;
explicit AppWindowRegistry(content::BrowserContext* context);
- virtual ~AppWindowRegistry();
+ ~AppWindowRegistry() override;
// Returns the instance for the given browser context, or NULL if none. This
// is a convenience wrapper around
@@ -112,14 +112,14 @@ class AppWindowRegistry : public KeyedService {
friend struct DefaultSingletonTraits<Factory>;
Factory();
- virtual ~Factory();
+ ~Factory() override;
// BrowserContextKeyedServiceFactory
- virtual KeyedService* BuildServiceInstanceFor(
+ KeyedService* BuildServiceInstanceFor(
content::BrowserContext* context) const override;
- virtual bool ServiceIsCreatedWithBrowserContext() const override;
- virtual bool ServiceIsNULLWhileTesting() const override;
- virtual content::BrowserContext* GetBrowserContextToUse(
+ bool ServiceIsCreatedWithBrowserContext() const override;
+ bool ServiceIsNULLWhileTesting() const override;
+ content::BrowserContext* GetBrowserContextToUse(
content::BrowserContext* context) const override;
};
« no previous file with comments | « extensions/browser/app_window/app_window_geometry_cache.h ('k') | extensions/browser/app_window/native_app_window.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698