Find-in-page should not ding while deleting characters.

chrome/browser/tab_contents/tab_contents.cc

 // Copyright (c) 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/tab_contents/tab_contents.h"
 
 #include "app/l10n_util.h"
 #include "app/resource_bundle.h"
 #include "base/file_version_info.h"
 #include "base/process_util.h"
@@ skipping 1066 matching lines @@
                                bool case_sensitive) {
   // If find_text is empty, it means FindNext was pressed with a keyboard
   // shortcut so unless we have something to search for we return early.
   if (find_text.empty() && find_text_.empty()) {
     if (last_search_prepopulate_text_->empty())
       return;
     // Try whatever we searched for last in any tab.
     find_text = *last_search_prepopulate_text_;
   }
 
+  // Keep track of the previous search.
+  previous_find_text_ = find_text_;
+
   // This is a FindNext operation if we are searching for the same text again,
   // or if the passed in search text is empty (FindNext keyboard shortcut). The
   // exception to this is if the Find was aborted (then we don't want FindNext
   // because the highlighting has been cleared and we need it to reappear). We
   // therefore treat FindNext after an aborted Find operation as a full fledged
   // Find.
   bool find_next = (find_text_ == find_text || find_text.empty()) &&
                    (last_search_case_sensitive_ == case_sensitive) &&
                    !find_op_aborted_;
   if (!find_next)
@@ skipping 13 matching lines @@
                                    forward_direction,
                                    case_sensitive,
                                    find_next);
 }
 
 void TabContents::StopFinding(bool clear_selection) {
   // When |clear_selection| is true, it means the find string has been cleared
   // by the user, but the UI has not been dismissed.
   if (!clear_selection)
     find_ui_active_ = false;
+  find_text_ = string16();

[Peter Kasting, 2009/10/26 21:04:33]

Nit: Use "find_text_.clear();"

   find_op_aborted_ = true;
   last_search_result_ = FindNotificationDetails();
   render_view_host()->StopFinding(clear_selection);
 }
 
 void TabContents::GetPageLanguage() {
   render_view_host()->GetPageLanguage();
 }
 
 void TabContents::OnJavaScriptMessageBoxClosed(IPC::Message* reply_msg,
@@ skipping 1516 matching lines @@
 #endif
 
     default:
       NOTREACHED();
   }
 }
 
 void TabContents::set_encoding(const std::string& encoding) {
   encoding_ = CharacterEncoding::GetCanonicalEncodingNameByAliasName(encoding);
 }