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

ash/common/system/tray/actionable_view.cc

 // Copyright 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/actionable_view.h"
 
 #include "ash/common/ash_constants.h"
 #include "ash/common/system/tray/system_tray.h"
 #include "ash/common/system/tray/system_tray_item.h"
 #include "ash/common/system/tray/tray_constants.h"
@@ skipping 109 matching lines @@
 
   if (action_performed) {
     AnimateInkDrop(views::InkDropState::ACTION_TRIGGERED,
                    ui::LocatedEvent::FromIfValid(&event));
   } else {
     AnimateInkDrop(views::InkDropState::HIDDEN,
                    ui::LocatedEvent::FromIfValid(&event));
   }
 }
 
+ButtonListenerActionableView::ButtonListenerActionableView(
+    SystemTrayItem* owner,
+    views::ButtonListener* listener)
+    : ActionableView(owner), listener_(listener), destroyed_(nullptr) {}

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

did you run git cl format ash? This looks a bit off...

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

Yeah, this is the result of 'git cl format'.

+
+ButtonListenerActionableView::~ButtonListenerActionableView() {
+  if (destroyed_)
+    *destroyed_ = true;
+}
+
+bool ButtonListenerActionableView::PerformAction(const ui::Event& event) {
+  bool destroyed = false;
+  destroyed_ = &destroyed;
+  listener_->ButtonPressed(this, event);
+  if (destroyed)

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

I'm not sure what this gains you, since the only place you potentially access a
member in the destroyed case is on line 146, and there you're just nulling
something out.

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

Not sure why I had this here.  Removed.

+    return true;
+  destroyed_ = nullptr;
+  return true;
+}
+
 }  // namespace ash