Remove Yoshi charger recall pop up warning UI code.

ash/system/chromeos/power/tray_power.cc

Index: ash/system/chromeos/power/tray_power.cc
diff --git a/ash/system/chromeos/power/tray_power.cc b/ash/system/chromeos/power/tray_power.cc
index d5d410c8d660a49c62c6442ae585d49604b64290..3692cc2e3ef22cdb08f04cf9af20df0e0b9d782e 100644
--- a/ash/system/chromeos/power/tray_power.cc
+++ b/ash/system/chromeos/power/tray_power.cc
@@ -39,19 +39,11 @@ using message_center::Notification;
 
 namespace ash {
 namespace tray {
-namespace {
-
-const int kMaxSpringChargerAccessibilityNotifyCount = 3;
-const int kSpringChargerAccessibilityTimerFirstTimeNotifyInSeconds = 30;
-const int kSpringChargerAccessibilityTimerRepeatInMinutes = 5;
-
-}
 
 // This view is used only for the tray.
 class PowerTrayView : public views::ImageView {
  public:
-  PowerTrayView()
-      : spring_charger_spoken_notification_count_(0) {
+  PowerTrayView() {
     UpdateImage();
   }
 
@@ -74,58 +66,13 @@ class PowerTrayView : public views::ImageView {
     }
   }
 
-  void SetupNotifyBadCharger() {
-    // Poll with a shorter duration timer to notify the charger issue
-    // for the first time after the charger dialog is displayed.
-    spring_charger_accessibility_timer_.Start(
-        FROM_HERE, base::TimeDelta::FromSeconds(
-            kSpringChargerAccessibilityTimerFirstTimeNotifyInSeconds),
-        this, &PowerTrayView::NotifyChargerIssue);
-  }
-
  private:
   void UpdateImage() {
     SetImage(PowerStatus::Get()->GetBatteryImage(PowerStatus::ICON_LIGHT));
   }
 
-  void NotifyChargerIssue() {
-    if (!Shell::GetInstance()->accessibility_delegate()->
-            IsSpokenFeedbackEnabled())
-      return;
-
-    if (!Shell::GetInstance()->system_tray_delegate()->
-            IsSpringChargerReplacementDialogVisible()) {
-      spring_charger_accessibility_timer_.Stop();
-      return;
-    }
-
-    accessible_name_ = ui::ResourceBundle::GetSharedInstance().
-        GetLocalizedString(IDS_CHARGER_REPLACEMENT_ACCESSIBILITY_NOTIFICATION);
-    NotifyAccessibilityEvent(ui::AX_EVENT_ALERT, true);
-    ++spring_charger_spoken_notification_count_;
-
-    if (spring_charger_spoken_notification_count_ == 1) {
-      // After notify the charger issue for the first time, repeat the
-      // notification with a longer duration timer.
-      spring_charger_accessibility_timer_.Stop();
-      spring_charger_accessibility_timer_.Start(
-          FROM_HERE, base::TimeDelta::FromMinutes(
-              kSpringChargerAccessibilityTimerRepeatInMinutes),
-          this, &PowerTrayView::NotifyChargerIssue);
-    } else if (spring_charger_spoken_notification_count_ >=
-        kMaxSpringChargerAccessibilityNotifyCount) {
-      spring_charger_accessibility_timer_.Stop();
-    }
-  }
-
   base::string16 accessible_name_;
 
-  // Tracks how many times the original spring charger accessibility
-  // notification has been spoken.
-  int spring_charger_spoken_notification_count_;
-
-  base::RepeatingTimer<PowerTrayView> spring_charger_accessibility_timer_;
-
   DISALLOW_COPY_AND_ASSIGN(PowerTrayView);
 };
 
@@ -220,15 +167,6 @@ void TrayPower::UpdateAfterShelfAlignmentChange(ShelfAlignment alignment) {
 }
 
 void TrayPower::OnPowerStatusChanged() {
-  RecordChargerType();
-
-  if (PowerStatus::Get()->IsOriginalSpringChargerConnected()) {
-    if (ash::Shell::GetInstance()->system_tray_delegate()->
-            ShowSpringChargerReplacementDialog()) {
-      power_tray_->SetupNotifyBadCharger();
-    }
-  }
-
   bool battery_alert = UpdateNotificationState();
   if (power_tray_)
     power_tray_->UpdateStatus(battery_alert);
@@ -288,8 +226,7 @@ bool TrayPower::UpdateNotificationState() {
   const PowerStatus& status = *PowerStatus::Get();
   if (!status.IsBatteryPresent() ||
       status.IsBatteryTimeBeingCalculated() ||
-      status.IsMainsChargerConnected() ||
-      status.IsOriginalSpringChargerConnected()) {

[Daniel Erat, 2014/11/14 22:48:26]

i think we still want to include this here, at least until powerd is updated to
report the original spring charger as a regular mains charger. (do you want me
to check in a change to do that first?)

[jennyz, 2014/11/15 00:04:04]

Yes, please check in the change for updating powerd before this cl if it does
not take too much trouble.

+      status.IsMainsChargerConnected()) {
     notification_state_ = NOTIFICATION_NONE;
     return false;
   }
@@ -371,28 +308,4 @@ bool TrayPower::UpdateNotificationStateForRemainingPercentage() {
   return false;
 }
 
-void TrayPower::RecordChargerType() {

[Daniel Erat, 2014/11/14 22:48:26]

it might be worthwhile to keep this metric around, but if you want to get rid of
it you should also mark it as deprecated in
tools/metrics/histograms/histograms.xml

[jennyz, 2014/11/15 00:04:04]

I deprecated it in the histograms.xml.

-  if (!PowerStatus::Get()->IsLinePowerConnected() ||
-      line_power_was_connected_)
-    return;
-
-  ChargerType current_charger = UNKNOWN_CHARGER;
-  if (PowerStatus::Get()->IsMainsChargerConnected()) {
-    current_charger = MAINS_CHARGER;
-  } else if (PowerStatus::Get()->IsUsbChargerConnected()) {
-    current_charger = USB_CHARGER;
-  } else if (PowerStatus::Get()->IsOriginalSpringChargerConnected()) {
-    current_charger =
-        ash::Shell::GetInstance()->system_tray_delegate()->
-            HasUserConfirmedSafeSpringCharger() ?
-        SAFE_SPRING_CHARGER : UNCONFIRMED_SPRING_CHARGER;
-  }
-
-  if (current_charger != UNKNOWN_CHARGER) {
-    UMA_HISTOGRAM_ENUMERATION("Power.ChargerType",
-                              current_charger,
-                              CHARGER_TYPE_COUNT);
-  }
-}
-
 }  // namespace ash