Add battery status to settings row in uber tray bubble.

ash/system/settings/tray_settings.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/settings/tray_settings.h"
 
 #include "ash/shell.h"
+#include "ash/system/power/power_status_view.h"
 #include "ash/system/tray/system_tray_delegate.h"
 #include "ash/system/tray/tray_constants.h"
 #include "ash/system/tray/tray_views.h"
 #include "base/logging.h"
 #include "base/utf_string_conversions.h"
 #include "grit/ash_strings.h"
 #include "grit/ui_resources_standard.h"
 #include "third_party/skia/include/core/SkColor.h"
 #include "ui/base/accessibility/accessible_view_state.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/image/image.h"
 #include "ui/views/controls/image_view.h"
 #include "ui/views/controls/label.h"
 #include "ui/views/layout/box_layout.h"
 #include "ui/views/layout/fill_layout.h"
 #include "ui/views/view.h"
 
-namespace {
+namespace ash {
+namespace internal {
 
-class SettingsView : public ash::internal::ActionableView {
+namespace tray {
+
+class SettingsDefaultView : public ash::internal::ActionableView {
  public:
-  SettingsView() {
+  explicit SettingsDefaultView(user::LoginStatus status)
+      : power_status_view_(NULL) {
     SetLayoutManager(new views::BoxLayout(views::BoxLayout::kHorizontal,
         ash::kTrayPopupPaddingHorizontal, 0,
         ash::kTrayPopupPaddingBetweenItems));
 
-    ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
-    views::ImageView* icon =
-        new ash::internal::FixedSizedImageView(0, ash::kTrayPopupItemHeight);
-    icon->SetImage(rb.GetImageNamed(IDR_AURA_UBER_TRAY_SETTINGS).ToImageSkia());
-    AddChildView(icon);
+    if (status != user::LOGGED_IN_NONE && status != user::LOGGED_IN_LOCKED) {
+      ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
+      views::ImageView* icon =
+          new ash::internal::FixedSizedImageView(0, ash::kTrayPopupItemHeight);
+      icon->SetImage(
+          rb.GetImageNamed(IDR_AURA_UBER_TRAY_SETTINGS).ToImageSkia());
+      AddChildView(icon);
 
-    string16 text = rb.GetLocalizedString(
-          IDS_ASH_STATUS_TRAY_SETTINGS);
-    label_ = new views::Label(text);
-    AddChildView(label_);
+      string16 text = rb.GetLocalizedString(IDS_ASH_STATUS_TRAY_SETTINGS);
+      label_ = new views::Label(text);
+      AddChildView(label_);
+      SetAccessibleName(text);
+    }
 
-    SetAccessibleName(text);
+    PowerSupplyStatus power_status =
+        ash::Shell::GetInstance()->tray_delegate()->GetPowerSupplyStatus();
+    if (power_status.battery_is_present) {
+      power_status_view_ = new ash::internal::tray::PowerStatusView(
+          ash::internal::tray::PowerStatusView::VIEW_DEFAULT);
+      AddChildView(power_status_view_);
+      power_status_view_->UpdatePowerStatus(power_status);

[stevenjb, 2012/06/07 01:31:18]

nit: Put this into UpdatePowerStatus()? (See comment below)

[jennyz, 2012/06/07 22:06:52]

Done.

+    }
   }
 
-  virtual ~SettingsView() {}
+  virtual ~SettingsDefaultView() {}
 
   // Overridden from ash::internal::ActionableView.
   virtual bool PerformAction(const views::Event& event) OVERRIDE {
     ash::Shell::GetInstance()->tray_delegate()->ShowSettings();
     return true;
   }
 
+  // Overridden from views::View.
+  virtual void Layout() OVERRIDE {
+    // Let the box-layout do the layout first. Then move power_status_view_
+    // to right align if it is created.
+    views::View::Layout();
+
+    if (power_status_view_) {
+      gfx::Size size = power_status_view_->GetPreferredSize();
+      gfx::Rect bounds(size);
+      bounds.set_x(width() - size.width() - ash::kTrayPopupPaddingBetweenItems);
+      bounds.set_y((height() - size.height()) / 2);
+      power_status_view_->SetBoundsRect(bounds);
+    }
+  }
+
+  // Overridden from views::View.
+  virtual void ChildPreferredSizeChanged(views::View* child) OVERRIDE {
+    views::View::ChildPreferredSizeChanged(child);
+    Layout();
+  }
+
+  ash::internal::tray::PowerStatusView* power_status_view() {
+    return power_status_view_;
+  }
+
  private:
   views::Label* label_;
+  ash::internal::tray::PowerStatusView* power_status_view_;
 
-  DISALLOW_COPY_AND_ASSIGN(SettingsView);
-};
+  DISALLOW_COPY_AND_ASSIGN(SettingsDefaultView);
+ };
 
-}  // namespace
+}  // namespace tray
 
-namespace ash {
-namespace internal {
-
-TraySettings::TraySettings() {}
+TraySettings::TraySettings()
+    : default_(NULL) {
+}
 
 TraySettings::~TraySettings() {}
 
 views::View* TraySettings::CreateTrayView(user::LoginStatus status) {
   return NULL;
 }
 
 views::View* TraySettings::CreateDefaultView(user::LoginStatus status) {
-  if (status == user::LOGGED_IN_NONE || status == user::LOGGED_IN_LOCKED)
+  if ((status == user::LOGGED_IN_NONE || status == user::LOGGED_IN_LOCKED) &&
+      (!ash::Shell::GetInstance()->tray_delegate()->
+          GetPowerSupplyStatus().battery_is_present))
     return NULL;
 
-  return new SettingsView;
+  CHECK(default_ == NULL);
+  default_ =  new tray::SettingsDefaultView(status);
+  return default_;
 }
 
 views::View* TraySettings::CreateDetailedView(user::LoginStatus status) {
   NOTIMPLEMENTED();
   return NULL;
 }
 
 void TraySettings::DestroyTrayView() {
 }
 
 void TraySettings::DestroyDefaultView() {
+  default_ = NULL;
 }
 
 void TraySettings::DestroyDetailedView() {
 }
 
 void TraySettings::UpdateAfterLoginStatusChange(user::LoginStatus status) {
 }
 
+// Overridden from PowerStatusObserver.
+void TraySettings::OnPowerStatusChanged(const PowerSupplyStatus& status) {
+  if (default_ && default_->power_status_view())
+    default_->power_status_view()->UpdatePowerStatus(status);

[stevenjb, 2012/06/07 01:31:18]

nit: Make UpdatePowerStatus() a member of PowerStatusView instead of exposing
power_status_view()?

[jennyz, 2012/06/07 22:06:52]

Done, you mean make UpdatePowerStatus() a member of SettingsDefaultView, right?

+}
+
 }  // namespace internal
 }  // namespace ash