Fixed focus requesting for the findbar.

chrome/browser/ui/views/find_bar_view.cc

 // Copyright (c) 2010 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 "app/l10n_util.h"
 #include "app/resource_bundle.h"
 #include "base/string_number_conversions.h"
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/find_bar_controller.h"
 #include "chrome/browser/find_bar_state.h"
 #include "chrome/browser/profile.h"
 #include "chrome/browser/tab_contents/tab_contents.h"
 #include "chrome/browser/themes/browser_theme_provider.h"
 #include "chrome/browser/view_ids.h"
 #include "chrome/browser/views/find_bar_host.h"
 #include "chrome/browser/views/frame/browser_view.h"
 #include "gfx/canvas.h"
 #include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
 #include "third_party/skia/include/effects/SkGradientShader.h"
 #include "views/background.h"
 #include "views/controls/button/image_button.h"
 #include "views/controls/label.h"
+#include "views/focus/focus_manager.h"
 #include "views/widget/widget.h"
 
 // The amount of whitespace to have before the find button.
 static const int kWhiteSpaceAfterMatchCountLabel = 1;
 
 // The margins around the search field and the close button.
 static const int kMarginLeftOfCloseButton = 3;
 static const int kMarginRightOfCloseButton = 7;
 static const int kMarginLeftOfFindTextfield = 12;
 
@@ skipping 178 matching lines @@
 
   // The match_count label may have increased/decreased in size so we need to
   // do a layout and repaint the dialog so that the find text field doesn't
   // partially overlap the match-count label when it increases on no matches.
   Layout();
   SchedulePaint();
 }
 
 void FindBarView::SetFocusAndSelection(bool select_all) {
   find_text_->RequestFocus();
+  // Storing is needed here because the view that has a focus before the wrench

[Finnur, 2010/11/15 16:20:30]

nit: 'a focus' implies there might be more than one focus (focai?) :) active at
any given point. That's not the case. I would just drop the 'a' here since the
meaning here is 'the focus'.

[altimofeev, 2010/11/16 14:28:34]

Done.

+  // menu activation will get the focus just after the wrench menu is closed.

[Finnur, 2010/11/15 16:20:30]

tiny-nit: 'focus' is enough, 'the focus' is not needed here. :)

[altimofeev, 2010/11/16 14:28:34]

Done.

+  // FindBar has it's own focus tracker, so it will focus a correct view on

[Finnur, 2010/11/15 16:20:30]

nit: 'The FindBar'
nit: 'the correct view'

[altimofeev, 2010/11/16 14:28:34]

Done.

+  // the close.

[Finnur, 2010/11/15 16:20:30]

nit: 'the' is not appropriate here (the pendulum has swung the other way now).
:)

[altimofeev, 2010/11/16 14:28:34]

Done.

+  find_text_->GetFocusManager()->StoreFocusedView();
   if (select_all && !find_text_->text().empty())
     find_text_->SelectAll();
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 // FindBarView, views::View overrides:
 
 void FindBarView::Paint(gfx::Canvas* canvas) {
   SkPaint paint;
 
@@ skipping 281 matching lines @@
 
 void FindBarView::OnThemeChanged() {
   ResourceBundle& rb = ResourceBundle::GetSharedInstance();
   if (GetThemeProvider()) {
     close_button_->SetBackground(
         GetThemeProvider()->GetColor(BrowserThemeProvider::COLOR_TAB_TEXT),
         rb.GetBitmapNamed(IDR_CLOSE_BAR),
         rb.GetBitmapNamed(IDR_CLOSE_BAR_MASK));
   }
 }