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

Side by Side Diff: chrome/browser/ui/views/tabs/tab_strip.cc

Issue 393033006: Added tab notification to accessibility_event_router_views. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: updated test class Created 6 years, 5 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/tabs/tab_strip.h" 5 #include "chrome/browser/ui/views/tabs/tab_strip.h"
6 6
7 #if defined(OS_WIN) 7 #if defined(OS_WIN)
8 #include <windowsx.h> 8 #include <windowsx.h>
9 #endif 9 #endif
10 10
(...skipping 1165 matching lines...) Expand 10 before | Expand all | Expand 10 after
1176 clip->SetRect(x, 0, tab_bounds.width() - x, tab_bounds.height()); 1176 clip->SetRect(x, 0, tab_bounds.width() - x, tab_bounds.height());
1177 } 1177 }
1178 } 1178 }
1179 return true; 1179 return true;
1180 } 1180 }
1181 1181
1182 bool TabStrip::IsImmersiveStyle() const { 1182 bool TabStrip::IsImmersiveStyle() const {
1183 return immersive_style_; 1183 return immersive_style_;
1184 } 1184 }
1185 1185
1186 void TabStrip::UpdateTabAccessibilityState(const Tab* tab,
1187 ui::AXViewState* state) {
sky 2014/07/23 17:38:42 nit: indentation.
evy 2014/07/23 18:44:11 Done.
1188 state->count = tab_count();
1189 state->index = GetModelIndexOfTab(tab);
sky 2014/07/23 17:38:42 Do you care if this returns -1?
evy 2014/07/23 18:44:11 Is there a case where the user hovers over/touches
1190 }
1191
1186 void TabStrip::MouseMovedOutOfHost() { 1192 void TabStrip::MouseMovedOutOfHost() {
1187 ResizeLayoutTabs(); 1193 ResizeLayoutTabs();
1188 if (reset_to_shrink_on_exit_) { 1194 if (reset_to_shrink_on_exit_) {
1189 reset_to_shrink_on_exit_ = false; 1195 reset_to_shrink_on_exit_ = false;
1190 SetStackedLayout(false); 1196 SetStackedLayout(false);
1191 controller_->StackedLayoutMaybeChanged(); 1197 controller_->StackedLayoutMaybeChanged();
1192 } 1198 }
1193 } 1199 }
1194 1200
1195 /////////////////////////////////////////////////////////////////////////////// 1201 ///////////////////////////////////////////////////////////////////////////////
(...skipping 1517 matching lines...) Expand 10 before | Expand all | Expand 10 after
2713 action = TouchUMA::GESTURE_TABSWITCH_TAP; 2719 action = TouchUMA::GESTURE_TABSWITCH_TAP;
2714 TouchUMA::RecordGestureAction(action); 2720 TouchUMA::RecordGestureAction(action);
2715 break; 2721 break;
2716 } 2722 }
2717 2723
2718 default: 2724 default:
2719 break; 2725 break;
2720 } 2726 }
2721 event->SetHandled(); 2727 event->SetHandled();
2722 } 2728 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698