Prepopulate the Find box with whatever was searched for last

chrome/browser/tab_contents/web_contents.cc

-// Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
+// 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/web_contents.h"
 
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/file_version_info.h"
 #include "base/process_util.h"
 #include "base/string_util.h"
@@ skipping 189 matching lines @@
       received_page_title_(false),
       is_starred_(false),
 #if defined(OS_WIN)
       message_box_active_(CreateEvent(NULL, TRUE, FALSE, NULL)),
 #endif
       ALLOW_THIS_IN_INITIALIZER_LIST(fav_icon_helper_(this)),
       suppress_javascript_messages_(false),
       load_state_(net::LOAD_STATE_IDLE),
       find_ui_active_(false),
       find_op_aborted_(false),
-      current_find_request_id_(find_request_id_counter_++) {
+      current_find_request_id_(find_request_id_counter_++),
+      find_prepopulate_text_(NULL) {
   pending_install_.page_id = 0;
   pending_install_.callback_functor = NULL;
 
   render_manager_.Init(profile, site_instance, routing_id, modal_dialog_event);
 
   view_->CreateView();
 
   // Register for notifications about all interested prefs change.
   PrefService* prefs = profile->GetPrefs();
   if (prefs) {
     for (int i = 0; i < kPrefsToObserveLength; ++i)
       prefs->AddPrefObserver(kPrefsToObserve[i], this);
   }
 
   // Register for notifications about URL starredness changing on any profile.
   NotificationService::current()->AddObserver(
       this, NotificationType::URLS_STARRED, NotificationService::AllSources());
   NotificationService::current()->AddObserver(
       this, NotificationType::BOOKMARK_MODEL_LOADED,
       NotificationService::AllSources());
   NotificationService::current()->AddObserver(
       this, NotificationType::RENDER_WIDGET_HOST_DESTROYED,
       NotificationService::AllSources());
+
+  // Keep a global copy of the previous search string (if any).
+  static string16 global_last_search = string16();
+  find_prepopulate_text_ = &global_last_search;

[Ben Goodger (Google), 2009/04/13 20:48:25]

is there a reason to save a pointer to this rather than just making the member
static?

 }
 
 WebContents::~WebContents() {
   if (pending_install_.callback_functor)
     pending_install_.callback_functor->Cancel();
   NotificationService::current()->RemoveObserver(
       this, NotificationType::RENDER_WIDGET_HOST_DESTROYED,
       NotificationService::AllSources());
 }
 
@@ skipping 373 matching lines @@
   bool find_next = (find_text_ == find_text || find_text.empty()) &&
                    !find_op_aborted_;
   if (!find_next)
     current_find_request_id_ = find_request_id_counter_++;
 
   if (!find_text.empty())
     find_text_ = find_text;
 
   find_op_aborted_ = false;
 
+  // Keep track of what the last search was across the tabs.
+  *find_prepopulate_text_ = find_text;
+
   render_view_host()->StartFinding(current_find_request_id_,
                                    find_text_,
                                    forward_direction,
                                    false,  // case sensitive
                                    find_next);
 }
 
 void WebContents::StopFinding(bool clear_selection) {
   find_ui_active_ = false;
   find_op_aborted_ = true;
@@ skipping 1394 matching lines @@
   //    -> Use pending DOM UI if any.
   //
   //  - Normal state with no load: committed nav entry + no pending nav entry:
   //    -> Use committed DOM UI.
   if (controller()->GetPendingEntry() &&
       (controller()->GetLastCommittedEntry() ||
        render_manager_.pending_dom_ui()))
     return render_manager_.pending_dom_ui();
   return render_manager_.dom_ui();
 }