diff options
| author | mhm@chromium.org <mhm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-14 02:05:43 +0000 |
|---|---|---|
| committer | mhm@chromium.org <mhm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-14 02:05:43 +0000 |
| commit | 63596301ddb6f3f6192dce7e81cf5722312c1433 (patch) | |
| tree | 8b3c403332183cc758729dcc486d277ba165e167 | |
| parent | 72742e97889bc30db6d0ab5d0a23fc82db5480ad (diff) | |
| download | chromium_src-63596301ddb6f3f6192dce7e81cf5722312c1433.zip chromium_src-63596301ddb6f3f6192dce7e81cf5722312c1433.tar.gz chromium_src-63596301ddb6f3f6192dce7e81cf5722312c1433.tar.bz2 | |
Remove hot tracked toolbar button when toolbar removes focus.
WillLooseFocus needs to check if acc_focused_view is set, if it is, then it will properly remove it.
BUG=3510
TEST=ALT+SHIFT+T all the way to the menu button, select options/about chromium menu option and hit enter to select one of them. The menu will be not hottracked.
Review URL: http://codereview.chromium.org/270072
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28938 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | chrome/browser/views/toolbar_view.cc | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/chrome/browser/views/toolbar_view.cc b/chrome/browser/views/toolbar_view.cc index 1cdc877..ed8912a 100644 --- a/chrome/browser/views/toolbar_view.cc +++ b/chrome/browser/views/toolbar_view.cc @@ -646,6 +646,12 @@ void ToolbarView::WillLoseFocus() { // Any tooltips that are active should be hidden when toolbar loses focus. if (GetWidget() && GetWidget()->GetTooltipManager()) GetWidget()->GetTooltipManager()->HideKeyboardTooltip(); + + // Removes the Child MSAA view's focus when toolbar loses focus. + if (acc_focused_view_) { + acc_focused_view_->SetHotTracked(false); + acc_focused_view_ = NULL; + } } void ToolbarView::RequestFocus() { |