Chromium Code Reviews| Index: chrome/browser/ui/views/find_bar_view.cc |
| diff --git a/chrome/browser/ui/views/find_bar_view.cc b/chrome/browser/ui/views/find_bar_view.cc |
| index 8e74295faea968c688fe5799ed19196e93683040..5facdcb6689c72b202a557a51c3082c1a2760a50 100644 |
| --- a/chrome/browser/ui/views/find_bar_view.cc |
| +++ b/chrome/browser/ui/views/find_bar_view.cc |
| @@ -427,10 +427,10 @@ void FindBarView::ButtonPressed( |
| false); // Not case sensitive. |
| } |
| - if (event.IsMouseEvent()) { |
| - // Move focus to the find textfield. |
| + // Focus the textfield, unless the button was invoked with a key press. |
| + // That lets the user continuously press SPACE/ENTER on next/previous. |
|
Peter Kasting
2016/06/17 20:55:47
Nit: We might want to note why we'd want to move f
msw
2016/06/18 00:25:18
As discussed, I'm removing this workaround added f
|
| + if (!event.IsKeyEvent()) |
| find_text_->RequestFocus(); |
| - } |
| break; |
| case VIEW_ID_FIND_IN_PAGE_CLOSE_BUTTON: |
| find_bar_host()->GetFindBarController()->EndFindSession( |