Don't leave focus on the next/prior button on the find bar

chrome/browser/views/find_bar_view.cc

 // Copyright (c) 2006-2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/views/find_bar_view.h"
 
 #include <algorithm>
 
 #include "base/string_util.h"
 #include "chrome/browser/find_bar_controller.h"
@@ skipping 415 matching lines @@
 
 void FindBarView::ButtonPressed(views::Button* sender) {
   switch (sender->tag()) {
     case FIND_PREVIOUS_TAG:
     case FIND_NEXT_TAG:
       if (find_text_->GetText().length() > 0) {
         container_->GetFindBarController()->web_contents()->StartFinding(
             find_text_->GetText(),
             sender->tag() == FIND_NEXT_TAG);
       }
+      // Move the focus back to the text-field, we don't want the button
+      // focused.
+      find_text_->RequestFocus();

[Finnur, 2009/04/08 15:22:36]

Wait... what happens if I press Tab twice (starting at the text field)? Will I
make it to the FindNext button? Won't it go from the text-field to the
FindPrevious button and back to the text-field (in a loop).

       break;
     case CLOSE_TAG:
       container_->GetFindBarController()->EndFindSession();
       break;
     default:
       NOTREACHED() << L"Unknown button";
       break;
   }
 }
 
@@ skipping 50 matching lines @@
 }
 
 bool FindBarView::FocusForwarderView::OnMousePressed(
     const views::MouseEvent& event) {
   if (view_to_focus_on_mousedown_) {
     view_to_focus_on_mousedown_->ClearSelection();
     view_to_focus_on_mousedown_->RequestFocus();
   }
   return true;
 }