Comment #14 on issue 15228 by m...@chromium.org: Alt+Shift+T: Esc flashes  
find-in bar continuously
http://code.google.com/p/chromium/issues/detail?id=15228

I tried to do the second approach, regarding eliminating the hack we  
implemented. The
thing is, it still the same thing, but rather than the IsFocusable()  
requirement for
the ToolBarView it now requires it to be part of ImageButton since now I am  
doing
RequestFocus on that ImageButton. And the traversal wont work like this:

void ToolbarView::InitiateFocus() {
   acc_focused_view_ = back_;
   acc_focused_view_->RequestFocus();
}

To make it work, it should be:
void ToolbarView::InitiateFocus() {
   acc_focused_view_ = back_;
   GetRootView()->FocusView(back_);
}

But still the hack is still needed. Unless I am misunderstanding the whole  
concept :(

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to