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

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

Issue 921423003: Rename ExtensionWarningBadgeService to WarningBadgeService (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 years, 10 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: chrome/browser/extensions/warning_badge_service.h
diff --git a/chrome/browser/extensions/extension_warning_badge_service.h b/chrome/browser/extensions/warning_badge_service.h
similarity index 74%
rename from chrome/browser/extensions/extension_warning_badge_service.h
rename to chrome/browser/extensions/warning_badge_service.h
index 2117683c629a4767e95a7a8222c6987cd778c895..ebd075c3f264fca4261679e6fa5b1271caeb328d 100644
--- a/chrome/browser/extensions/extension_warning_badge_service.h
+++ b/chrome/browser/extensions/warning_badge_service.h
@@ -11,22 +11,20 @@
#include "extensions/browser/warning_service.h"
#include "extensions/browser/warning_set.h"
-// TODO(battre): Rename ExtensionWarningBadgeService to WarningBadgeService.
-
class Profile;
namespace extensions {
// A service that is responsible for showing an extension warning badge on the
// wrench menu.
-class ExtensionWarningBadgeService : public KeyedService,
- public WarningService::Observer,
- public base::NonThreadSafe {
+class WarningBadgeService : public KeyedService,
+ public WarningService::Observer,
+ public base::NonThreadSafe {
public:
- explicit ExtensionWarningBadgeService(Profile* profile);
- ~ExtensionWarningBadgeService() override;
+ explicit WarningBadgeService(Profile* profile);
+ ~WarningBadgeService() override;
- static ExtensionWarningBadgeService* Get(content::BrowserContext* context);
+ static WarningBadgeService* Get(content::BrowserContext* context);
// Black lists all currently active extension warnings, so that they do not
// trigger a warning badge again for the life-time of the browsing session.
@@ -51,7 +49,7 @@ class ExtensionWarningBadgeService : public KeyedService,
// Warnings that do not trigger a badge on the wrench menu.
WarningSet suppressed_warnings_;
- DISALLOW_COPY_AND_ASSIGN(ExtensionWarningBadgeService);
+ DISALLOW_COPY_AND_ASSIGN(WarningBadgeService);
};
} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698