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

Side by Side Diff: chrome/browser/views/accessible_toolbar_view.h

Issue 1751018: Remove unused local variables now that subclasses don't need to track their o... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 10 years, 8 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) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 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 #ifndef CHROME_BROWSER_VIEWS_ACCESSIBLE_TOOLBAR_VIEW_H_ 5 #ifndef CHROME_BROWSER_VIEWS_ACCESSIBLE_TOOLBAR_VIEW_H_
6 #define CHROME_BROWSER_VIEWS_ACCESSIBLE_TOOLBAR_VIEW_H_ 6 #define CHROME_BROWSER_VIEWS_ACCESSIBLE_TOOLBAR_VIEW_H_
7 7
8 #include "views/view.h" 8 #include "views/view.h"
9 9
10 // This class provides keyboard access to any view that extends it by intiating 10 // This class provides keyboard access to any view that extends it by intiating
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
48 private: 48 private:
49 // Sets the focus to the currently |acc_focused_view_| view. 49 // Sets the focus to the currently |acc_focused_view_| view.
50 void SetFocusToAccessibleView(); 50 void SetFocusToAccessibleView();
51 51
52 // Retrieve the focused view from the storage so we can request focus back 52 // Retrieve the focused view from the storage so we can request focus back
53 // to it. If |focus_view| is null, we place focus on the default view. 53 // to it. If |focus_view| is null, we place focus on the default view.
54 // |selected_focused_view_| doesn't need to reset here since it will be 54 // |selected_focused_view_| doesn't need to reset here since it will be
55 // dealt within the WillLoseFocus method. 55 // dealt within the WillLoseFocus method.
56 void SetFocusToLastFocusedView(); 56 void SetFocusToLastFocusedView();
57 57
58 // Storage of strings needed for accessibility.
59 std::wstring accessible_name_;
60
61 // Selected child view currently having accessibility focus. 58 // Selected child view currently having accessibility focus.
62 views::View* selected_focused_view_; 59 views::View* selected_focused_view_;
63 60
64 // Last focused view that issued this traversal. 61 // Last focused view that issued this traversal.
65 int last_focused_view_storage_id_; 62 int last_focused_view_storage_id_;
66 63
67 DISALLOW_COPY_AND_ASSIGN(AccessibleToolbarView); 64 DISALLOW_COPY_AND_ASSIGN(AccessibleToolbarView);
68 }; 65 };
69 66
70 #endif // CHROME_BROWSER_VIEWS_ACCESSIBLE_TOOLBAR_VIEW_H_ 67 #endif // CHROME_BROWSER_VIEWS_ACCESSIBLE_TOOLBAR_VIEW_H_
OLDNEW
« no previous file with comments | « chrome/browser/chromeos/frame/normal_browser_frame_view.h ('k') | chrome/browser/views/bookmark_bar_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698