Change PowerStatus::GetBatteryImage to not use ExtractImageRep and scale

ash/common/system/chromeos/power/power_status_view.cc

Index: ash/common/system/chromeos/power/power_status_view.cc
diff --git a/ash/common/system/chromeos/power/power_status_view.cc b/ash/common/system/chromeos/power/power_status_view.cc
index 3e918b98496f4f9c1f728cc96a27d14a1732124a..bef2e0680c3c1f6c157c201b67c59a62bebec71c 100644
--- a/ash/common/system/chromeos/power/power_status_view.cc
+++ b/ash/common/system/chromeos/power/power_status_view.cc
@@ -51,20 +51,6 @@ PowerStatusView::~PowerStatusView() {
 
 void PowerStatusView::OnPowerStatusChanged() {
   UpdateText();
-
-  // We do not show a battery icon in the material design system menu.
-  // TODO(tdanderson): Remove the non-MD code and the IconSet enum once
-  // material design is enabled by default. See crbug.com/614453.
-  if (MaterialDesignController::UseMaterialDesignSystemIcons())
-    return;
-
-  const PowerStatus::BatteryImageInfo info =
-      PowerStatus::Get()->GetBatteryImageInfo(PowerStatus::ICON_DARK);
-  if (info != previous_battery_image_info_) {

[Peter Kasting, 2017/03/31 22:07:20]

Looks like this member variable is now dead and should be removed.

[danakj, 2017/04/03 14:43:14]

Done.

-    icon_->SetImage(PowerStatus::Get()->GetBatteryImage(info));

[Peter Kasting, 2017/03/31 22:07:20]

Looks like we no longer need to track this pointer as a member variable, since
it's never read; we can just AddChildView() directly in the places that
currently set it.

[danakj, 2017/04/03 14:43:14]

Done.

-    icon_->SetVisible(true);
-    previous_battery_image_info_ = info;
-  }
 }
 
 void PowerStatusView::LayoutView() {