Materialized font style for TrayItemMore type system tray rows.

ash/common/system/tray/tray_item_more.h

 // 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.
 
 #ifndef ASH_COMMON_SYSTEM_TRAY_TRAY_ITEM_MORE_H_
 #define ASH_COMMON_SYSTEM_TRAY_TRAY_ITEM_MORE_H_
 
+#include <memory>

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

is this needed?

[bruthig, 2016/09/15 18:56:29]

Yup, for unique_ptr<>.

+
 #include "ash/common/system/tray/actionable_view.h"
+#include "ash/common/system/tray/tray_popup_item_style.h"
+#include "ash/common/system/tray/tray_popup_item_style_observer.h"
 #include "base/macros.h"
 #include "ui/views/view.h"
 
 namespace views {
 class ImageView;
 class Label;
 class View;
 }
 
 namespace ash {
 class SystemTrayItem;
 
-// A view with a chevron ('>') on the right edge. Clicking on the view brings up
-// the detailed view of the tray-item that owns it.
-class TrayItemMore : public ActionableView {
+// A view with a 'more' arrow icon on the right edge. Clicking on the view
+// brings up the detailed view of the tray-item that owns it.
+class TrayItemMore : public ActionableView, public TrayPopupItemStyleObserver {
  public:
   TrayItemMore(SystemTrayItem* owner, bool show_more);
   ~TrayItemMore() override;
 
   SystemTrayItem* owner() const { return owner_; }
 
   void SetLabel(const base::string16& label);
   void SetImage(const gfx::ImageSkia& image_skia);
   void SetAccessibleName(const base::string16& name);
 
+  // Set the current visual style of this item. OnTrayPopupItemStyleUpdated()
+  // will be called.
+  void SetStyle(std::unique_ptr<TrayPopupItemStyle> style);
+
  protected:
-  // Replaces the default icon (on the left of the label), and allows a custom
-  // view to be placed there. Once the default icon is replaced, |SetImage|
-  // should never be called.
-  void ReplaceIcon(views::View* view);
+  // TrayPopupItemStyleObserver:
+  void OnTrayPopupItemStyleUpdated() override;
+
+  TrayPopupItemStyle* style() const { return style_.get(); }
+
+  // Update the content of this (e.g. Label text, font, color, etc).
+  virtual void UpdateContent();
 
  private:
   // Overridden from ActionableView.

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

nit: just "// ActionableView:" while you're here. Ditto for line 55.

[bruthig, 2016/09/15 18:56:29]

Done.

   bool PerformAction(const ui::Event& event) override;
 
   // Overridden from views::View.
   void Layout() override;
   void GetAccessibleState(ui::AXViewState* state) override;
+  void OnThemeChanged() override;
 
   SystemTrayItem* owner_;
   // True if |more_| should be shown.
   bool show_more_;
   views::ImageView* icon_;
   views::Label* label_;
   views::ImageView* more_;
   base::string16 accessible_name_;
 
+  // Current visual style of this item. This only applies when system menu is
+  // using the material design look and feel.
+  std::unique_ptr<TrayPopupItemStyle> style_;
+
   DISALLOW_COPY_AND_ASSIGN(TrayItemMore);
 };
 
 }  // namespace ash
 
 #endif  // ASH_COMMON_SYSTEM_TRAY_TRAY_ITEM_MORE_H_