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

Unified Diff: ui/app_list/views/app_list_item_view.cc

Issue 1534303002: CustomButton cleanup: make protected members private, create accessors (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: tidy up Created 5 years 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 side-by-side diff with in-line comments
Download patch
Index: ui/app_list/views/app_list_item_view.cc
diff --git a/ui/app_list/views/app_list_item_view.cc b/ui/app_list/views/app_list_item_view.cc
index 340bc5d93101202d89f0827d5db0e66fe3b3f3e4..d669696021d205df760340802174d7aaa596bcea 100644
--- a/ui/app_list/views/app_list_item_view.cc
+++ b/ui/app_list/views/app_list_item_view.cc
@@ -143,11 +143,11 @@ void AppListItemView::SetIcon(const gfx::ImageSkia& icon) {
shadow_animator_.SetOriginalImage(resized);
}
-void AppListItemView::SetUIState(UIState state) {
- if (ui_state_ == state)
+void AppListItemView::SetUIState(UIState ui_state) {
+ if (ui_state_ == ui_state)
return;
- ui_state_ = state;
+ ui_state_ = ui_state;
switch (ui_state_) {
case UI_STATE_NORMAL:
@@ -466,14 +466,14 @@ void AppListItemView::OnGestureEvent(ui::GestureEvent* event) {
}
break;
case ui::ET_GESTURE_TAP_DOWN:
- if (::switches::IsTouchFeedbackEnabled() && state_ != STATE_DISABLED) {
+ if (::switches::IsTouchFeedbackEnabled() && state() != STATE_DISABLED) {
SetState(STATE_PRESSED);
event->SetHandled();
}
break;
case ui::ET_GESTURE_TAP:
case ui::ET_GESTURE_TAP_CANCEL:
- if (::switches::IsTouchFeedbackEnabled() && state_ != STATE_DISABLED)
+ if (::switches::IsTouchFeedbackEnabled() && state() != STATE_DISABLED)
SetState(STATE_NORMAL);
break;
case ui::ET_GESTURE_LONG_PRESS:

Powered by Google App Engine
This is Rietveld 408576698