[ash-md] Updated layout of Brightness and Audio system menu rows to the material design spec.

ash/common/system/audio/volume_view.h

 // Copyright 2014 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_AUDIO_VOLUME_VIEW_H_
 #define ASH_COMMON_SYSTEM_AUDIO_VOLUME_VIEW_H_
 
-#include "ash/common/system/tray/actionable_view.h"
 #include "base/macros.h"
-#include "ui/gfx/font.h"
 #include "ui/views/controls/button/button.h"
 #include "ui/views/controls/slider.h"
+#include "ui/views/view.h"
 
 namespace views {
-class View;
+class CustomButton;
 class ImageView;
 class Separator;
 class Slider;
 }
 
 namespace ash {
 class SystemTrayItem;
+class TriView;
 
 namespace system {
 class TrayAudioDelegate;
 }
 
 namespace tray {
 class VolumeButton;
 
-class VolumeView : public ActionableView,
-                   public views::SliderListener {
+class VolumeView : public views::View,
+                   public views::SliderListener,
+                   public views::ButtonListener {
  public:
   VolumeView(SystemTrayItem* owner,
              system::TrayAudioDelegate* audio_delegate,
              bool is_default_view);
 
   ~VolumeView() override;
 
   void Update();
 
   // Sets volume level on slider_, |percent| is ranged from [0.00] to [1.00].
   void SetVolumeLevel(float percent);
 
  private:
-  // Updates bar_, device_type_ icon, and more_ buttons.
+  // Updates device_type_ icon and more_ button.
   void UpdateDeviceTypeAndMore();
   void HandleVolumeUp(float percent);
   void HandleVolumeDown(float percent);
 
-  // Overridden from views::SliderListener.
+  // SliderListener:
   void SliderValueChanged(views::Slider* sender,
                           float value,
                           float old_value,
                           views::SliderChangeReason reason) override;
 
-  // Overriden from ActionableView.
-  bool PerformAction(const ui::Event& event) override;
+  // views::ButtonListener:
   void ButtonPressed(views::Button* sender, const ui::Event& event) override;
 
   // views::View:
   void OnBoundsChanged(const gfx::Rect& previous_bounds) override;
-  void GetAccessibleNodeData(ui::AXNodeData* node_data) override;
 
+  SystemTrayItem* owner_;
+  // Container view used to layout the children.

[tdanderson, 2016/11/09 01:38:33]

Maybe add to this comment to clarify that |tri_view_| is the only child of
|this|, and all other views elements in this row are parented to the tri view.

[bruthig, 2016/11/10 02:54:54]

Done.

+  TriView* tri_view_;
   system::TrayAudioDelegate* audio_delegate_;
-  views::View* more_region_;
+  views::CustomButton* more_button_;
   VolumeButton* icon_;
   views::Slider* slider_;
   views::Separator* separator_;
   views::ImageView* device_type_;
-  views::ImageView* more_;
   bool is_default_view_;

[tdanderson, 2016/11/09 01:38:33]

nit: can you please add a "// not used in material design" or "// only used in
material design" after these members where applicable?

[bruthig, 2016/11/10 02:54:54]

Done.

 
   DISALLOW_COPY_AND_ASSIGN(VolumeView);
 };
 
 }  // namespace tray
 }  // namespace ash
 
 #endif  // ASH_COMMON_SYSTEM_AUDIO_VOLUME_VIEW_H_