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

Unified Diff: chrome/browser/views/toolbar_view.h

Issue 27175: Fix toolbar keyboard focus (shift-alt-t), which was broken by me, and toolbar... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 11 years, 10 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/views/frame/browser_view.cc ('k') | chrome/browser/views/toolbar_view.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/views/toolbar_view.h
===================================================================
--- chrome/browser/views/toolbar_view.h (revision 10342)
+++ chrome/browser/views/toolbar_view.h (working copy)
@@ -87,29 +87,27 @@
// (such as user editing) as well.
void Update(TabContents* tab, bool should_restore_state);
- void OnInputInProgress(bool in_progress);
+ virtual void OnInputInProgress(bool in_progress);
+ // Returns a brief, identifying string, containing a unique, readable name.
+ virtual bool GetAccessibleName(std::wstring* name);
+
// Returns the MSAA role of the current view. The role is what assistive
// technologies (ATs) use to determine what behavior to expect from a given
// control.
- bool GetAccessibleRole(VARIANT* role);
+ virtual bool GetAccessibleRole(VARIANT* role);
- // Returns a brief, identifying string, containing a unique, readable name.
- bool GetAccessibleName(std::wstring* name);
-
// Assigns an accessible string name.
- void SetAccessibleName(const std::wstring& name);
+ virtual void SetAccessibleName(const std::wstring& name);
+ virtual View* GetAccFocusedChildView() { return acc_focused_view_; }
+
// Returns the index of the next view of the toolbar, starting from the given
// view index (skipping the location bar), in the given navigation direction
// (nav_left true means navigation right to left, and vice versa). -1 finds
// first accessible child, based on the above policy.
int GetNextAccessibleViewIndex(int view_index, bool nav_left);
- views::View* acc_focused_view() {
- return acc_focused_view_;
- }
-
void set_acc_focused_view(views::View* acc_focused_view) {
acc_focused_view_ = acc_focused_view;
}
« no previous file with comments | « chrome/browser/views/frame/browser_view.cc ('k') | chrome/browser/views/toolbar_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698