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

Unified Diff: chrome/browser/ui/toolbar/app_menu_icon_controller.cc

Issue 2374803002: Remove a bunch of pre-MD toolbar code and assets. (Closed)
Patch Set: fix mac? Created 4 years, 3 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/ui/toolbar/app_menu_icon_controller.cc
diff --git a/chrome/browser/ui/toolbar/app_menu_icon_controller.cc b/chrome/browser/ui/toolbar/app_menu_icon_controller.cc
index 6ebcd7f0c9dfd3e1a537095fcd59c201bb2e65a7..94409738ee8860f9b759e78c599e8d1d9cf8329e 100644
--- a/chrome/browser/ui/toolbar/app_menu_icon_controller.cc
+++ b/chrome/browser/ui/toolbar/app_menu_icon_controller.cc
@@ -19,24 +19,22 @@
namespace {
// Maps an upgrade level to a severity level.
-AppMenuIconPainter::Severity SeverityFromUpgradeLevel(
+AppMenuIconController::Severity SeverityFromUpgradeLevel(
UpgradeDetector::UpgradeNotificationAnnoyanceLevel level) {
switch (level) {
case UpgradeDetector::UPGRADE_ANNOYANCE_NONE:
- return AppMenuIconPainter::SEVERITY_NONE;
+ return AppMenuIconController::Severity::NONE;
case UpgradeDetector::UPGRADE_ANNOYANCE_LOW:
- return AppMenuIconPainter::SEVERITY_LOW;
+ return AppMenuIconController::Severity::LOW;
case UpgradeDetector::UPGRADE_ANNOYANCE_ELEVATED:
- return AppMenuIconPainter::SEVERITY_MEDIUM;
+ return AppMenuIconController::Severity::MEDIUM;
case UpgradeDetector::UPGRADE_ANNOYANCE_HIGH:
- return AppMenuIconPainter::SEVERITY_HIGH;
case UpgradeDetector::UPGRADE_ANNOYANCE_SEVERE:
- return AppMenuIconPainter::SEVERITY_HIGH;
case UpgradeDetector::UPGRADE_ANNOYANCE_CRITICAL:
- return AppMenuIconPainter::SEVERITY_HIGH;
+ return AppMenuIconController::Severity::HIGH;
}
NOTREACHED();
- return AppMenuIconPainter::SEVERITY_NONE;
+ return AppMenuIconController::Severity::NONE;
}
// Checks if the app menu icon should be animated for the given upgrade level.
@@ -110,7 +108,7 @@ void AppMenuIconController::UpdateDelegate() {
if (ShouldShowIncompatibilityWarning()) {
delegate_->UpdateSeverity(IconType::INCOMPATIBILITY_WARNING,
- AppMenuIconPainter::SEVERITY_MEDIUM, true);
+ Severity::MEDIUM, true);
return;
}
@@ -119,12 +117,12 @@ void AppMenuIconController::UpdateDelegate() {
// If you change the severity here, make sure to also change the menu icon
// and the bubble icon.
delegate_->UpdateSeverity(IconType::GLOBAL_ERROR,
- AppMenuIconPainter::SEVERITY_MEDIUM, true);
+ Severity::MEDIUM, true);
return;
}
delegate_->UpdateSeverity(IconType::NONE,
- AppMenuIconPainter::SEVERITY_NONE, true);
+ Severity::NONE, true);
}
#if defined(OS_WIN)

Powered by Google App Engine
This is Rietveld 408576698