diff options
author | finnur@chromium.org <finnur@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-22 20:28:12 +0000 |
---|---|---|
committer | finnur@chromium.org <finnur@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-22 20:28:12 +0000 |
commit | e491f1cb8247090047bd8c895e9bd2bf4de5ee5a (patch) | |
tree | df56a34ca44028d65cde514853cb3feda0c8b839 /chrome/browser/views/find_bar_view.cc | |
parent | 44c875a167d5ce78000d8843190a8c22d2236e54 (diff) | |
download | chromium_src-e491f1cb8247090047bd8c895e9bd2bf4de5ee5a.zip chromium_src-e491f1cb8247090047bd8c895e9bd2bf4de5ee5a.tar.gz chromium_src-e491f1cb8247090047bd8c895e9bd2bf4de5ee5a.tar.bz2 |
Fix bug 12468 where F3 (FindNext) in a brand new tab was not using prepopulated search (what you have searched for in other tabs).
TEST=Covered by in-process-browser test now. To test manually: open a tab, search for something in FindInPage, Press Ctrl+T and then F3. It should search for the same thing in the newly opened tab.
BUG=12468
Review URL: http://codereview.chromium.org/115714
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@16786 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/views/find_bar_view.cc')
-rw-r--r-- | chrome/browser/views/find_bar_view.cc | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/chrome/browser/views/find_bar_view.cc b/chrome/browser/views/find_bar_view.cc index 2c62a27..c2db6d2 100644 --- a/chrome/browser/views/find_bar_view.cc +++ b/chrome/browser/views/find_bar_view.cc @@ -401,7 +401,8 @@ void FindBarView::ButtonPressed(views::Button* sender) { if (!find_text_->GetText().empty()) { container_->GetFindBarController()->tab_contents()->StartFinding( find_text_->GetText(), - sender->tag() == FIND_NEXT_TAG); + sender->tag() == FIND_NEXT_TAG, + false); // Not case sensitive. } // Move the focus back to the text-field, we don't want the button // focused. @@ -435,7 +436,8 @@ void FindBarView::ContentsChanged(views::TextField* sender, // if the textbox contains something we set it as the new search string and // initiate search (even though old searches might be in progress). if (!new_contents.empty()) { - controller->tab_contents()->StartFinding(new_contents, true); + // The last two params here are forward (true) and case sensitive (false). + controller->tab_contents()->StartFinding(new_contents, true, false); } else { // The textbox is empty so we reset. true = clear selection on page. controller->tab_contents()->StopFinding(true); @@ -462,7 +464,8 @@ bool FindBarView::HandleKeystroke(views::TextField* sender, // Search forwards for enter, backwards for shift-enter. container_->GetFindBarController()->tab_contents()->StartFinding( find_string, - GetKeyState(VK_SHIFT) >= 0); + GetKeyState(VK_SHIFT) >= 0, + false); // Not case sensitive. } } #endif |