Focus comes on the 'close button' in find bar when selecting 'Enter' key.

chrome/browser/ui/views/find_bar_view.cc

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 816e93d3c310c6dc39b6dfd3dbc187ed04f957ef..d9406e24dd6639dd7d52e2f0623d1b49feff9e61 100644
--- a/chrome/browser/ui/views/find_bar_view.cc
+++ b/chrome/browser/ui/views/find_bar_view.cc
@@ -406,12 +406,14 @@ void FindBarView::ButtonPressed(
                                       sender->tag() == FIND_NEXT_TAG,
                                       false);  // Not case sensitive.
       }
-      if (event.IsMouseEvent()) {
-        // If mouse event, we move the focus back to the text-field, so that the
-        // user doesn't have to click on the text field to change the search. We
-        // don't want to do this for keyboard clicks on the button, since the
-        // user is more likely to press FindNext again than change the search
-        // query.
+      // As NotifyClick() now takes ui::Event directly so event can be mouse
+      // event or the key event.
+      if (event.IsMouseEvent() || (event.IsKeyEvent() &&
+                                   static_cast<const ui::KeyEvent&>(event)
+                                           .key_code() == ui::VKEY_RETURN)) {

[msw, 2015/08/26 18:44:32]

The same defect occurs for the space key, and perhaps others, so this isn't
complete, and likely the wrong fix. If we just want the old behavior, remove the
|event| conditions and always focus the textfield. But I think it should do:
A) If it's a mouse event, focus the textfield (as the code already did).
B) If it's a key event (Enter, Space, whatever...) leave the focus on the same
button (up or down), so users can focus the '^' (find previous) button and
repeatedly hit [RETURN] or [SPACE] to repeatedly find previous instances of the
search phrase. Otherwise, pressing a key to invoke '^' (find previous) will
focus the textfield and the following [ENTER] key event will find *next*
(switching direction from the last key press), or the following [SPACE] key
press will replace the textfield text with a space character (unexpected).

[Deepak, 2015/08/27 05:31:04]

Thanks for detailed explanation. As the current issue have release blocker tag,
so I am restoring it to the old behavior i.e moving the focus to text field for
the mouse and key events.
I will make separate patch for your suggestion for keeping the focus on the same
button so that user's [Return] or [Space] key will search previous/next instance
of the search phrase for key events.

[msw, 2015/08/27 18:09:02]

Acknowledged.

+        // If mouse event or key event with Enter key, we move the focus back
+        // to the text-field, so that the user doesn't have to click on the
+        // text field to change the search.
         find_text_->RequestFocus();
       }
       break;