Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(287)

Side by Side Diff: chrome/browser/ui/views/toolbar/toolbar_button.cc

Issue 1280953003: Enhance the material design ripple API so the ripple's state can be controlled by it's owning View. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed comments from patch set 15. Created 5 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/views/toolbar/toolbar_button.h" 5 #include "chrome/browser/ui/views/toolbar/toolbar_button.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/location.h" 8 #include "base/location.h"
9 #include "base/single_thread_task_runner.h" 9 #include "base/single_thread_task_runner.h"
10 #include "base/thread_task_runner_handle.h" 10 #include "base/thread_task_runner_handle.h"
11 #include "chrome/browser/themes/theme_properties.h" 11 #include "chrome/browser/themes/theme_properties.h"
12 #include "chrome/browser/ui/views/location_bar/location_bar_view.h" 12 #include "chrome/browser/ui/views/location_bar/location_bar_view.h"
13 #include "ui/accessibility/ax_view_state.h" 13 #include "ui/accessibility/ax_view_state.h"
14 #include "ui/base/l10n/l10n_util.h" 14 #include "ui/base/l10n/l10n_util.h"
15 #include "ui/base/models/menu_model.h" 15 #include "ui/base/models/menu_model.h"
16 #include "ui/base/resource/material_design/material_design_controller.h" 16 #include "ui/base/resource/material_design/material_design_controller.h"
17 #include "ui/base/theme_provider.h" 17 #include "ui/base/theme_provider.h"
18 #include "ui/gfx/display.h" 18 #include "ui/gfx/display.h"
19 #include "ui/gfx/screen.h" 19 #include "ui/gfx/screen.h"
20 #include "ui/strings/grit/ui_strings.h" 20 #include "ui/strings/grit/ui_strings.h"
21 #include "ui/views/animation/ink_drop_animation_controller.h" 21 #include "ui/views/animation/ink_drop_animation_controller.h"
22 #include "ui/views/animation/ink_drop_animation_controller_factory.h"
22 #include "ui/views/controls/button/label_button_border.h" 23 #include "ui/views/controls/button/label_button_border.h"
23 #include "ui/views/controls/menu/menu_item_view.h" 24 #include "ui/views/controls/menu/menu_item_view.h"
24 #include "ui/views/controls/menu/menu_model_adapter.h" 25 #include "ui/views/controls/menu/menu_model_adapter.h"
25 #include "ui/views/controls/menu/menu_runner.h" 26 #include "ui/views/controls/menu/menu_runner.h"
26 #include "ui/views/widget/widget.h" 27 #include "ui/views/widget/widget.h"
27 28
28 ToolbarButton::ToolbarButton(views::ButtonListener* listener, 29 ToolbarButton::ToolbarButton(views::ButtonListener* listener,
29 ui::MenuModel* model) 30 ui::MenuModel* model)
30 : views::LabelButton(listener, base::string16()), 31 : views::LabelButton(listener, base::string16()),
31 model_(model), 32 model_(model),
32 menu_showing_(false), 33 menu_showing_(false),
33 y_position_on_lbuttondown_(0), 34 y_position_on_lbuttondown_(0),
34 show_menu_factory_(this) { 35 show_menu_factory_(this) {
35 #if defined(OS_CHROMEOS)
36 // The ink drop animation is only targeted at ChromeOS because there is
37 // concern it will conflict with OS level touch feedback in a bad way.
38 if (ui::MaterialDesignController::IsModeMaterial()) { 36 if (ui::MaterialDesignController::IsModeMaterial()) {
39 ink_drop_animation_controller_.reset(
40 new views::InkDropAnimationController(this));
41 layer()->SetFillsBoundsOpaquely(false);
42 image()->SetPaintToLayer(true); 37 image()->SetPaintToLayer(true);
43 image()->SetFillsBoundsOpaquely(false); 38 image()->SetFillsBoundsOpaquely(false);
sadrul 2015/08/18 19:52:39 You should do these down in AddInkDropLayer too
bruthig 2015/08/18 20:58:04 Done.
44 } 39 }
45 #endif // defined(OS_CHROMEOS) 40 // TODO(bruthig): |ink_drop_animation_controller_| has to be initialized after
41 // SetPaintToLayer(true) becuase the InkDropAnimationControllerImpl
42 // constructor calls back in to AddInkDropLayer() to add the ink drop layer to
43 // layer(). This will be reworked so the InkDropAnimationController will call
44 // AddInkDropLayer() right before the ink drop becomes visible and then
45 // |ink_drop_animation_controller_| can be initialized in the initializer
46 // list.
sadrul 2015/08/18 19:52:39 Update the comment.
bruthig 2015/08/18 20:58:04 Done.
46 47
48 ink_drop_animation_controller_ = views::InkDropAnimationControllerFactory::
49 CreateInkDropAnimationController(this);
47 set_context_menu_controller(this); 50 set_context_menu_controller(this);
48 } 51 }
49 52
50 ToolbarButton::~ToolbarButton() { 53 ToolbarButton::~ToolbarButton() {
51 } 54 }
52 55
53 void ToolbarButton::Init() { 56 void ToolbarButton::Init() {
54 SetFocusable(false); 57 SetFocusable(false);
55 SetAccessibilityFocusable(true); 58 SetAccessibilityFocusable(true);
56 image()->EnableCanvasFlippingForRTLUI(true); 59 image()->EnableCanvasFlippingForRTLUI(true);
(...skipping 22 matching lines...) Expand all
79 ui::ThemeProvider* provider = GetThemeProvider(); 82 ui::ThemeProvider* provider = GetThemeProvider();
80 if (provider && provider->UsingSystemTheme()) { 83 if (provider && provider->UsingSystemTheme()) {
81 int inset = provider->GetDisplayProperty( 84 int inset = provider->GetDisplayProperty(
82 ThemeProperties::PROPERTY_TOOLBAR_BUTTON_BORDER_INSET); 85 ThemeProperties::PROPERTY_TOOLBAR_BUTTON_BORDER_INSET);
83 size.Enlarge(2 * inset, 2 * inset); 86 size.Enlarge(2 * inset, 2 * inset);
84 } 87 }
85 } 88 }
86 return size; 89 return size;
87 } 90 }
88 91
92 void ToolbarButton::Layout() {
93 LabelButton::Layout();
94 LayoutInkDrop();
95 }
96
89 bool ToolbarButton::OnMousePressed(const ui::MouseEvent& event) { 97 bool ToolbarButton::OnMousePressed(const ui::MouseEvent& event) {
90 if (enabled() && ShouldShowMenu() && 98 if (enabled() && ShouldShowMenu() &&
91 IsTriggerableEvent(event) && HitTestPoint(event.location())) { 99 IsTriggerableEvent(event) && HitTestPoint(event.location())) {
92 // Store the y pos of the mouse coordinates so we can use them later to 100 // Store the y pos of the mouse coordinates so we can use them later to
93 // determine if the user dragged the mouse down (which should pop up the 101 // determine if the user dragged the mouse down (which should pop up the
94 // drag down menu immediately, instead of waiting for the timer) 102 // drag down menu immediately, instead of waiting for the timer)
95 y_position_on_lbuttondown_ = event.y(); 103 y_position_on_lbuttondown_ = event.y();
96 104
97 // Schedule a task that will show the menu. 105 // Schedule a task that will show the menu.
98 const int kMenuTimerDelay = 500; 106 const int kMenuTimerDelay = 500;
99 base::ThreadTaskRunnerHandle::Get()->PostDelayedTask( 107 base::ThreadTaskRunnerHandle::Get()->PostDelayedTask(
100 FROM_HERE, base::Bind(&ToolbarButton::ShowDropDownMenu, 108 FROM_HERE, base::Bind(&ToolbarButton::ShowDropDownMenu,
101 show_menu_factory_.GetWeakPtr(), 109 show_menu_factory_.GetWeakPtr(),
102 ui::GetMenuSourceTypeForEvent(event)), 110 ui::GetMenuSourceTypeForEvent(event)),
103 base::TimeDelta::FromMilliseconds(kMenuTimerDelay)); 111 base::TimeDelta::FromMilliseconds(kMenuTimerDelay));
104 } 112 }
113
114 ink_drop_animation_controller_->AnimateToState(
115 views::InkDropState::ACTION_PENDING);
116
105 return LabelButton::OnMousePressed(event); 117 return LabelButton::OnMousePressed(event);
106 } 118 }
107 119
108 bool ToolbarButton::OnMouseDragged(const ui::MouseEvent& event) { 120 bool ToolbarButton::OnMouseDragged(const ui::MouseEvent& event) {
109 bool result = LabelButton::OnMouseDragged(event); 121 bool result = LabelButton::OnMouseDragged(event);
110 122
111 if (show_menu_factory_.HasWeakPtrs()) { 123 if (show_menu_factory_.HasWeakPtrs()) {
112 // If the mouse is dragged to a y position lower than where it was when 124 // If the mouse is dragged to a y position lower than where it was when
113 // clicked then we should not wait for the menu to appear but show 125 // clicked then we should not wait for the menu to appear but show
114 // it immediately. 126 // it immediately.
115 if (event.y() > y_position_on_lbuttondown_ + GetHorizontalDragThreshold()) { 127 if (event.y() > y_position_on_lbuttondown_ + GetHorizontalDragThreshold()) {
116 show_menu_factory_.InvalidateWeakPtrs(); 128 show_menu_factory_.InvalidateWeakPtrs();
117 ShowDropDownMenu(ui::GetMenuSourceTypeForEvent(event)); 129 ShowDropDownMenu(ui::GetMenuSourceTypeForEvent(event));
118 } 130 }
119 } 131 }
120 132
121 return result; 133 return result;
122 } 134 }
123 135
124 void ToolbarButton::OnMouseReleased(const ui::MouseEvent& event) { 136 void ToolbarButton::OnMouseReleased(const ui::MouseEvent& event) {
125 if (IsTriggerableEvent(event) || 137 if (IsTriggerableEvent(event) ||
126 (event.IsRightMouseButton() && !HitTestPoint(event.location()))) { 138 (event.IsRightMouseButton() && !HitTestPoint(event.location()))) {
127 LabelButton::OnMouseReleased(event); 139 LabelButton::OnMouseReleased(event);
128 } 140 }
129 141
130 if (IsTriggerableEvent(event)) 142 if (IsTriggerableEvent(event))
131 show_menu_factory_.InvalidateWeakPtrs(); 143 show_menu_factory_.InvalidateWeakPtrs();
144
145 ink_drop_animation_controller_->AnimateToState(views::InkDropState::HIDDEN);
132 } 146 }
133 147
134 void ToolbarButton::OnMouseCaptureLost() { 148 void ToolbarButton::OnMouseCaptureLost() {
135 } 149 }
136 150
137 void ToolbarButton::OnMouseExited(const ui::MouseEvent& event) { 151 void ToolbarButton::OnMouseExited(const ui::MouseEvent& event) {
138 // Starting a drag results in a MouseExited, we need to ignore it. 152 // Starting a drag results in a MouseExited, we need to ignore it.
139 // A right click release triggers an exit event. We want to 153 // A right click release triggers an exit event. We want to
140 // remain in a PUSHED state until the drop down menu closes. 154 // remain in a PUSHED state until the drop down menu closes.
141 if (state_ != STATE_DISABLED && !InDrag() && state_ != STATE_PRESSED) 155 if (state_ != STATE_DISABLED && !InDrag() && state_ != STATE_PRESSED)
142 SetState(STATE_NORMAL); 156 SetState(STATE_NORMAL);
143 } 157 }
144 158
145 void ToolbarButton::OnGestureEvent(ui::GestureEvent* event) { 159 void ToolbarButton::OnGestureEvent(ui::GestureEvent* event) {
146 if (menu_showing_) { 160 if (menu_showing_) {
147 // While dropdown menu is showing the button should not handle gestures. 161 // While dropdown menu is showing the button should not handle gestures.
148 event->StopPropagation(); 162 event->StopPropagation();
149 return; 163 return;
150 } 164 }
151 165
152 LabelButton::OnGestureEvent(event); 166 LabelButton::OnGestureEvent(event);
167
168 views::InkDropState ink_drop_state = views::InkDropState::HIDDEN;
169 switch (event->type()) {
170 case ui::ET_GESTURE_TAP_DOWN:
171 ink_drop_state = views::InkDropState::ACTION_PENDING;
172 // The ui::ET_GESTURE_TAP_DOWN event needs to be marked as handled so that
173 // subsequent events for the gesture are sent to |this|.
174 event->SetHandled();
175 break;
176 case ui::ET_GESTURE_LONG_PRESS:
177 ink_drop_state = views::InkDropState::SLOW_ACTION;
178 break;
179 case ui::ET_GESTURE_TAP:
180 ink_drop_state = views::InkDropState::QUICK_ACTION;
181 break;
182 case ui::ET_GESTURE_END:
183 case ui::ET_GESTURE_TAP_CANCEL:
184 ink_drop_state = views::InkDropState::HIDDEN;
185 break;
186 default:
187 return;
188 }
189 ink_drop_animation_controller_->AnimateToState(ink_drop_state);
153 } 190 }
154 191
155 void ToolbarButton::GetAccessibleState(ui::AXViewState* state) { 192 void ToolbarButton::GetAccessibleState(ui::AXViewState* state) {
156 CustomButton::GetAccessibleState(state); 193 CustomButton::GetAccessibleState(state);
157 state->role = ui::AX_ROLE_BUTTON_DROP_DOWN; 194 state->role = ui::AX_ROLE_BUTTON_DROP_DOWN;
158 state->default_action = l10n_util::GetStringUTF16(IDS_APP_ACCACTION_PRESS); 195 state->default_action = l10n_util::GetStringUTF16(IDS_APP_ACCACTION_PRESS);
159 state->AddStateFlag(ui::AX_STATE_HASPOPUP); 196 state->AddStateFlag(ui::AX_STATE_HASPOPUP);
160 } 197 }
161 198
162 scoped_ptr<views::LabelButtonBorder> 199 scoped_ptr<views::LabelButtonBorder>
(...skipping 15 matching lines...) Expand all
178 void ToolbarButton::ShowContextMenuForView(View* source, 215 void ToolbarButton::ShowContextMenuForView(View* source,
179 const gfx::Point& point, 216 const gfx::Point& point,
180 ui::MenuSourceType source_type) { 217 ui::MenuSourceType source_type) {
181 if (!enabled()) 218 if (!enabled())
182 return; 219 return;
183 220
184 show_menu_factory_.InvalidateWeakPtrs(); 221 show_menu_factory_.InvalidateWeakPtrs();
185 ShowDropDownMenu(source_type); 222 ShowDropDownMenu(source_type);
186 } 223 }
187 224
225 void ToolbarButton::AddInkDropLayer(ui::Layer* ink_drop_layer) {
226 SetPaintToLayer(true);
227 layer()->Add(ink_drop_layer);
228 layer()->StackAtBottom(ink_drop_layer);
229 }
230
231 void ToolbarButton::RemoveInkDropLayer(ui::Layer* ink_drop_layer) {
232 layer()->Remove(ink_drop_layer);
233 SetPaintToLayer(false);
234 }
235
188 bool ToolbarButton::ShouldEnterPushedState(const ui::Event& event) { 236 bool ToolbarButton::ShouldEnterPushedState(const ui::Event& event) {
189 // Enter PUSHED state on press with Left or Right mouse button or on taps. 237 // Enter PUSHED state on press with Left or Right mouse button or on taps.
190 // Remain in this state while the context menu is open. 238 // Remain in this state while the context menu is open.
191 return event.type() == ui::ET_GESTURE_TAP || 239 return event.type() == ui::ET_GESTURE_TAP ||
192 event.type() == ui::ET_GESTURE_TAP_DOWN || 240 event.type() == ui::ET_GESTURE_TAP_DOWN ||
193 (event.IsMouseEvent() && ((ui::EF_LEFT_MOUSE_BUTTON | 241 (event.IsMouseEvent() && ((ui::EF_LEFT_MOUSE_BUTTON |
194 ui::EF_RIGHT_MOUSE_BUTTON) & event.flags()) != 0); 242 ui::EF_RIGHT_MOUSE_BUTTON) & event.flags()) != 0);
195 } 243 }
196 244
197 bool ToolbarButton::ShouldShowMenu() { 245 bool ToolbarButton::ShouldShowMenu() {
(...skipping 64 matching lines...) Expand 10 before | Expand all | Expand 10 after
262 views::MenuRunner::RunResult result = 310 views::MenuRunner::RunResult result =
263 menu_runner_->RunMenuAt(GetWidget(), 311 menu_runner_->RunMenuAt(GetWidget(),
264 NULL, 312 NULL,
265 gfx::Rect(menu_position, gfx::Size(0, 0)), 313 gfx::Rect(menu_position, gfx::Size(0, 0)),
266 views::MENU_ANCHOR_TOPLEFT, 314 views::MENU_ANCHOR_TOPLEFT,
267 source_type); 315 source_type);
268 if (result == views::MenuRunner::MENU_DELETED) 316 if (result == views::MenuRunner::MENU_DELETED)
269 return; 317 return;
270 } 318 }
271 319
320 ink_drop_animation_controller_->AnimateToState(views::InkDropState::HIDDEN);
321
272 menu_showing_ = false; 322 menu_showing_ = false;
273 323
274 // Need to explicitly clear mouse handler so that events get sent 324 // Need to explicitly clear mouse handler so that events get sent
275 // properly after the menu finishes running. If we don't do this, then 325 // properly after the menu finishes running. If we don't do this, then
276 // the first click to other parts of the UI is eaten. 326 // the first click to other parts of the UI is eaten.
277 SetMouseHandler(NULL); 327 SetMouseHandler(NULL);
278 328
279 // Set the state back to normal after the drop down menu is closed. 329 // Set the state back to normal after the drop down menu is closed.
280 if (state_ != STATE_DISABLED) 330 if (state_ != STATE_DISABLED)
281 SetState(STATE_NORMAL); 331 SetState(STATE_NORMAL);
282 } 332 }
283 333
334 void ToolbarButton::LayoutInkDrop() {
335 ink_drop_animation_controller_->SetInkDropSize(size());
336 }
337
284 const char* ToolbarButton::GetClassName() const { 338 const char* ToolbarButton::GetClassName() const {
285 return "ToolbarButton"; 339 return "ToolbarButton";
286 } 340 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698