Implement the spring charger replacement UI.

ash/system/chromeos/power/tray_power.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ash/system/chromeos/power/tray_power.h"
 
 #include "ash/ash_switches.h"
+#include "ash/shell.h"
 #include "ash/system/chromeos/power/power_status_view.h"
 #include "ash/system/date/date_view.h"
 #include "ash/system/system_notifier.h"
+#include "ash/system/tray/system_tray_delegate.h"
 #include "ash/system/tray/tray_constants.h"
 #include "ash/system/tray/tray_notification_view.h"
 #include "ash/system/tray/tray_utils.h"
 #include "base/command_line.h"
+#include "base/metrics/histogram.h"
 #include "grit/ash_resources.h"
 #include "grit/ash_strings.h"
 #include "third_party/icu/source/i18n/unicode/fieldpos.h"
 #include "third_party/icu/source/i18n/unicode/fmtable.h"
 #include "ui/base/accessibility/accessible_view_state.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/message_center/message_center.h"
 #include "ui/message_center/notification.h"
 #include "ui/views/controls/button/button.h"
 #include "ui/views/controls/image_view.h"
@@ skipping 76 matching lines @@
 const int TrayPower::kCriticalPercentage = 5;
 const int TrayPower::kLowPowerPercentage = 10;
 const int TrayPower::kNoWarningPercentage = 15;
 
 TrayPower::TrayPower(SystemTray* system_tray, MessageCenter* message_center)
     : SystemTrayItem(system_tray),
       message_center_(message_center),
       power_tray_(NULL),
       notification_view_(NULL),
       notification_state_(NOTIFICATION_NONE),
-      usb_charger_was_connected_(false) {
+      usb_charger_was_connected_(false),
+      is_line_power_connected_(false) {
   PowerStatus::Get()->AddObserver(this);
 }
 
 TrayPower::~TrayPower() {
   PowerStatus::Get()->RemoveObserver(this);
 }
 
 views::View* TrayPower::CreateTrayView(user::LoginStatus status) {
   // There may not be enough information when this is created about whether
   // there is a battery or not. So always create this, and adjust visibility as
@@ skipping 33 matching lines @@
 }
 
 void TrayPower::UpdateAfterLoginStatusChange(user::LoginStatus status) {
 }
 
 void TrayPower::UpdateAfterShelfAlignmentChange(ShelfAlignment alignment) {
   SetTrayImageItemBorder(power_tray_, alignment);
 }
 
 void TrayPower::OnPowerStatusChanged() {
+  RecordExternalPowerSupply();
+
+  if (PowerStatus::Get()->IsOriginalSpringChargerConnected()) {

[Daniel Erat, 2013/12/10 00:08:58]

shouldn't this also check !HasUserConfirmedSafeSpringCharger()?

[jennyz, 2013/12/10 19:14:11]

The logic for checking if the dialog should be shown is in
ChargerReplacementDialog::ShouldShowDialog, called by
SystemTrayDelegate::ShowSpringChargerReplacementDialog. We won't show the dialog
unless user confirms they are using safe charger, or they decline to order a
replacement charger, or they ordered the charger but confirm that they would
like keep using the original charger. The flow gets more complicated later than
we originally discussed.

+    ash::Shell::GetInstance()->system_tray_delegate()->
+        ShowSpringChargerReplacementDialog();

[Daniel Erat, 2013/12/10 00:08:58]

should the dialog be hidden when the charger is unplugged?

[jennyz, 2013/12/10 19:14:11]

Once the dialog is shown, user has to complete the flow. We won't hide it when
the charger is unplugged, that's kind of confusing.

+  }
+
   bool battery_alert = UpdateNotificationState();
   if (power_tray_)
     power_tray_->UpdateStatus(battery_alert);
   if (notification_view_)
     notification_view_->UpdateStatus();
 
   // Factory testing may place the battery into unusual states.
   if (CommandLine::ForCurrentProcess()->HasSwitch(
           ash::switches::kAshHideNotificationsForFactory))
     return;
@@ skipping 119 matching lines @@
         return true;
       }
       return false;
     case NOTIFICATION_CRITICAL:
       return false;
   }
   NOTREACHED();
   return false;
 }
 
+void TrayPower::RecordExternalPowerSupply() {
+  if (!PowerStatus::Get()->IsLinePowerConnected()) {
+    is_line_power_connected_ = false;
+    return;
+  }
+
+  if (is_line_power_connected_)
+    return;
+
+  is_line_power_connected_ = true;
+  ExternalPowerSupply current_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 : ORIGINAL_SPRING_CHARGER;
+  } else {
+    current_charger = UNKNOWN_CHARGER;

[Daniel Erat, 2013/12/10 00:08:58]

nit: if you just initialize current_charger to UNKNOWN_CHARGER above, you can
remove this else clause

[jennyz, 2013/12/10 19:14:11]

Done.

+  }
+
+  if (current_charger != UNKNOWN_CHARGER) {
+    UMA_HISTOGRAM_ENUMERATION("PowerSupply.ExternalPowerSupply",

[Daniel Erat, 2013/12/10 00:08:58]

where did this name come from? all of the histograms in powerd have names like
"Power.Whatever"; could you name this something like "Power.ChargerType"
instead?

[jennyz, 2013/12/10 19:14:11]

Done.

+                              current_charger,
+                              NUM_EXTERNAL_POWER_SUPPLY);
+  }
+}
+
 }  // namespace internal
 }  // namespace ash