Wrench Icon: Only animate low severity status once

chrome/browser/ui/toolbar/wrench_icon_painter.cc

Index: chrome/browser/ui/toolbar/wrench_icon_painter.cc
diff --git a/chrome/browser/ui/toolbar/wrench_icon_painter.cc b/chrome/browser/ui/toolbar/wrench_icon_painter.cc
index b3726637cddd6c556d1aa578b50c7813f81fd11e..f0ba3d8716211f44f2597ce0ab37196961998d61 100644
--- a/chrome/browser/ui/toolbar/wrench_icon_painter.cc
+++ b/chrome/browser/ui/toolbar/wrench_icon_painter.cc
@@ -52,6 +52,19 @@ WrenchIconPainter::Severity WrenchIconPainter::SeverityFromUpgradeLevel(
 }
 
 // static
+bool WrenchIconPainter::ShouldAnimateUpgradeLevel(
+    UpgradeDetector::UpgradeNotificationAnnoyanceLevel level) {
+  bool should_animate = true;
+  if (level == UpgradeDetector::UPGRADE_ANNOYANCE_LOW) {
+    // Only animate low severity upgrades once.
+    static bool should_animate_low_severity = true;
+    should_animate = should_animate_low_severity;
+    should_animate_low_severity = false;
+  }
+  return should_animate;
+}
+
+// static
 WrenchIconPainter::Severity WrenchIconPainter::SeverityFromGlobalErrorSeverity(
     GlobalError::Severity severity) {
   switch (severity) {
@@ -73,14 +86,14 @@ WrenchIconPainter::WrenchIconPainter(Delegate* delegate)
 
 WrenchIconPainter::~WrenchIconPainter() {}
 
-void WrenchIconPainter::SetSeverity(Severity severity) {
+void WrenchIconPainter::SetSeverity(Severity severity, bool animate) {
   if (severity_ == severity)
     return;
 
   severity_ = severity;
   delegate_->ScheduleWrenchIconPaint();
   animation_.reset();
-  if (severity_ == SEVERITY_NONE)
+  if (severity_ == SEVERITY_NONE || !animate)
     return;
 
   ui::MultiAnimation::Parts parts;