Unify MenuButton Pushed state logic

ui/views/controls/button/menu_button.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 UI_VIEWS_CONTROLS_BUTTON_MENU_BUTTON_H_
 #define UI_VIEWS_CONTROLS_BUTTON_MENU_BUTTON_H_
 
 #include <string>
 
 #include "base/memory/weak_ptr.h"
@@ skipping 67 matching lines @@
   virtual bool OnKeyReleased(const ui::KeyEvent& event) OVERRIDE;
   virtual void GetAccessibleState(ui::AXViewState* state) OVERRIDE;
 
  protected:
   // Paint the menu marker image.
   void PaintMenuMarker(gfx::Canvas* canvas);
 
   // Overridden from LabelButton:
   virtual gfx::Rect GetChildAreaBounds() OVERRIDE;
 
+  // Overridden from MenuButton:

[flackr, 2014/10/14 14:11:06]

Overridden from MenuButton? Isn't this MenuButton?

[jonross, 2014/10/14 16:00:48]

Yup, meant CustomButton.

+  virtual bool ShouldEnterPushedState(const ui::Event& event) OVERRIDE;
+
   // Offset of the associated menu position.
   gfx::Point menu_offset_;
 
  private:
   friend class PressedLock;
 
   // Increment/decrement the number of "pressed" locks this button has, and
   // set the state accordingly.
   void IncrementPressedLocked();
   void DecrementPressedLocked();
@@ skipping 27 matching lines @@
   int pressed_lock_count_;
 
   base::WeakPtrFactory<MenuButton> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(MenuButton);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_CONTROLS_BUTTON_MENU_BUTTON_H_