Materialized font style for TrayItemMore type system tray rows.

ash/common/system/tray/tray_item_more.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/common/system/tray/tray_item_more.h"
 
 #include "ash/common/material_design/material_design_controller.h"
 #include "ash/common/system/tray/fixed_sized_image_view.h"
 #include "ash/common/system/tray/system_tray_item.h"
 #include "ash/common/system/tray/tray_constants.h"
 #include "grit/ash_resources.h"
 #include "ui/accessibility/ax_view_state.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/image/image.h"
 #include "ui/gfx/paint_vector_icon.h"
 #include "ui/gfx/vector_icons_public.h"
+#include "ui/native_theme/native_theme.h"
 #include "ui/views/controls/image_view.h"
 #include "ui/views/controls/label.h"
 #include "ui/views/layout/box_layout.h"
 
 namespace ash {
 
 TrayItemMore::TrayItemMore(SystemTrayItem* owner, bool show_more)
     : owner_(owner),
       show_more_(show_more),
-      icon_(NULL),
-      label_(NULL),
-      more_(NULL) {
+      icon_(nullptr),
+      label_(nullptr),
+      more_(nullptr),
+      style_(new TrayPopupItemStyle(GetNativeTheme(),
+                                    TrayPopupItemStyle::FontStyle::TITLE)) {
   SetLayoutManager(new views::BoxLayout(views::BoxLayout::kHorizontal,
                                         kTrayPopupPaddingHorizontal, 0,
                                         kTrayPopupPaddingBetweenItems));
 
   icon_ = new FixedSizedImageView(0, GetTrayConstant(TRAY_POPUP_ITEM_HEIGHT));
   AddChildView(icon_);
 
   label_ = new views::Label;
   label_->SetHorizontalAlignment(gfx::ALIGN_LEFT);
   AddChildView(label_);
 
-  if (show_more) {
-    more_ = new views::ImageView;
-    more_->EnableCanvasFlippingForRTLUI(true);
-    if (MaterialDesignController::IsSystemTrayMenuMaterial()) {
-      more_->SetImage(gfx::CreateVectorIcon(
-          gfx::VectorIconId::SYSTEM_MENU_ARROW_RIGHT, kMenuIconColor));
-    } else {
-      more_->SetImage(ui::ResourceBundle::GetSharedInstance()
-                          .GetImageNamed(IDR_AURA_UBER_TRAY_MORE)
-                          .ToImageSkia());
-    }
-    AddChildView(more_);
-  }
+  UpdateChildren();
 }
 
-TrayItemMore::~TrayItemMore() {}
+TrayItemMore::~TrayItemMore() {
+  style_->RemoveObserver(this);
+}
 
 void TrayItemMore::SetLabel(const base::string16& label) {
   label_->SetText(label);
   Layout();
   SchedulePaint();
 }
 
 void TrayItemMore::SetImage(const gfx::ImageSkia& image_skia) {
   icon_->SetImage(image_skia);
   SchedulePaint();
 }
 
 void TrayItemMore::SetAccessibleName(const base::string16& name) {
   accessible_name_ = name;
 }
 
-void TrayItemMore::ReplaceIcon(views::View* view) {
-  delete icon_;
-  icon_ = NULL;
-  AddChildViewAt(view, 0);
+void TrayItemMore::SetStyle(std::unique_ptr<TrayPopupItemStyle> style) {
+  DCHECK(style);
+  style_->RemoveObserver(this);
+  style_ = std::move(style);
+  style_->AddObserver(this);
+  OnTrayPopupItemStyleChanged();
+}
+
+void TrayItemMore::OnTrayPopupItemStyleChanged() {
+  SetEnabled(style_->color_style() != TrayPopupItemStyle::ColorStyle::DISABLED);
+  UpdateChildren();
+}
+
+void TrayItemMore::UpdateChildren() {
+  if (more_) {
+    delete more_;
+    more_ = nullptr;
+  }
+
+  if (MaterialDesignController::IsSystemTrayMenuMaterial())
+    style()->SetupLabel(label_);

[tdanderson, 2016/09/08 16:11:02]

I wonder if it would be better to invoke this whenever |label_| is instantiated,
and if it's possible for Label (or a derived type) could do the setup for us to
avoid a manual call to SetupLabel()?

[bruthig, 2016/09/12 13:45:34]

I'm not sure I follow 100%.  The style will need to be re-applied to a Label
whenever the style changes so doing this during construction only wouldn't be
sufficient.  However, if we made Label extend TrayPopupItemStyleObserver and add
the |label_| as an observer than perhaps we can make it a little more automatic.
 I don't think we should do this until the Style concept is mature enough and
promoted into the ui namespace.

[tdanderson, 2016/09/12 18:51:46]

Ah, disregard my comment then - in my first skim through I thought you only
needed to do this once.
Agreed.

+
+  if (show_more_) {
+    more_ = new views::ImageView;
+    more_->EnableCanvasFlippingForRTLUI(true);
+
+    if (MaterialDesignController::IsSystemTrayMenuMaterial()) {
+      more_->SetImage(
+          gfx::CreateVectorIcon(gfx::VectorIconId::SYSTEM_MENU_ARROW_RIGHT,
+                                style()->GetForegroundColor()));
+    } else {
+      more_->SetImage(ui::ResourceBundle::GetSharedInstance()
+                          .GetImageNamed(IDR_AURA_UBER_TRAY_MORE)
+                          .ToImageSkia());
+    }
+    AddChildView(more_);
+  }
 }
 
 bool TrayItemMore::PerformAction(const ui::Event& event) {
   if (!show_more_)
     return false;
 
   owner()->TransitionDetailedView();
   return true;
 }
 
@@ skipping 19 matching lines @@
     label_->SetBoundsRect(bounds);
   }
 }
 
 void TrayItemMore::GetAccessibleState(ui::AXViewState* state) {
   ActionableView::GetAccessibleState(state);
   if (!accessible_name_.empty())
     state->name = accessible_name_;
 }
 
+void TrayItemMore::OnThemeChanged() {
+  style_->SetTheme(GetNativeTheme());
+}
+
 }  // namespace ash