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

Side by Side Diff: chrome/browser/ui/views/accessible_pane_view.cc

Issue 6622002: Do all OOLing in the views code. linux_views now builds clean with the clang plugin. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 9 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 "base/logging.h" 5 #include "base/logging.h"
6 #include "chrome/browser/ui/view_ids.h" 6 #include "chrome/browser/ui/view_ids.h"
7 #include "chrome/browser/ui/views/accessible_pane_view.h" 7 #include "chrome/browser/ui/views/accessible_pane_view.h"
8 #include "chrome/browser/ui/views/frame/browser_view.h" 8 #include "chrome/browser/ui/views/frame/browser_view.h"
9 #include "chrome/browser/ui/views/location_bar/location_bar_view.h" 9 #include "chrome/browser/ui/views/location_bar/location_bar_view.h"
10 #include "views/controls/button/menu_button.h" 10 #include "views/controls/button/menu_button.h"
(...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after
80 focus_manager_->AddFocusChangeListener(this); 80 focus_manager_->AddFocusChangeListener(this);
81 81
82 return true; 82 return true;
83 } 83 }
84 84
85 bool AccessiblePaneView::SetPaneFocusAndFocusDefault( 85 bool AccessiblePaneView::SetPaneFocusAndFocusDefault(
86 int view_storage_id) { 86 int view_storage_id) {
87 return SetPaneFocus(view_storage_id, GetDefaultFocusableChild()); 87 return SetPaneFocus(view_storage_id, GetDefaultFocusableChild());
88 } 88 }
89 89
90 views::View* AccessiblePaneView::GetDefaultFocusableChild() {
91 return NULL;
92 }
93
90 void AccessiblePaneView::RemovePaneFocus() { 94 void AccessiblePaneView::RemovePaneFocus() {
91 focus_manager_->RemoveFocusChangeListener(this); 95 focus_manager_->RemoveFocusChangeListener(this);
92 pane_has_focus_ = false; 96 pane_has_focus_ = false;
93 97
94 focus_manager_->UnregisterAccelerator(home_key_, this); 98 focus_manager_->UnregisterAccelerator(home_key_, this);
95 focus_manager_->UnregisterAccelerator(end_key_, this); 99 focus_manager_->UnregisterAccelerator(end_key_, this);
96 focus_manager_->UnregisterAccelerator(escape_key_, this); 100 focus_manager_->UnregisterAccelerator(escape_key_, this);
97 focus_manager_->UnregisterAccelerator(left_key_, this); 101 focus_manager_->UnregisterAccelerator(left_key_, this);
98 focus_manager_->UnregisterAccelerator(right_key_, this); 102 focus_manager_->UnregisterAccelerator(right_key_, this);
99 } 103 }
(...skipping 142 matching lines...) Expand 10 before | Expand all | Expand 10 after
242 246
243 views::FocusTraversable* AccessiblePaneView::GetFocusTraversableParent() { 247 views::FocusTraversable* AccessiblePaneView::GetFocusTraversableParent() {
244 DCHECK(pane_has_focus_); 248 DCHECK(pane_has_focus_);
245 return NULL; 249 return NULL;
246 } 250 }
247 251
248 views::View* AccessiblePaneView::GetFocusTraversableParentView() { 252 views::View* AccessiblePaneView::GetFocusTraversableParentView() {
249 DCHECK(pane_has_focus_); 253 DCHECK(pane_has_focus_);
250 return NULL; 254 return NULL;
251 } 255 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698