Handle the ViewHostMsg_FindReply message which is sent by content(renderer) via the TabContentsDe...

chrome/browser/ui/browser.cc

 // Copyright (c) 2011 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/ui/browser.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 #include <shellapi.h>
 #endif  // OS_WIN
@@ skipping 2500 matching lines @@
 
   DLOG(INFO) << "Browser tab contents received intent:"
              << "\naction=" << UTF16ToASCII(action)
              << "\ntype=" << UTF16ToASCII(type)
              << "\nrenderer_id=" << routing_id
              << "\nid=" << intent_id;
 
   tcw->web_intent_picker_controller()->ShowDialog(action, type);
 }
 
+// static
+void Browser::FindReplyHelper(TabContents* tab,
+                              int request_id,
+                              int number_of_matches,
+                              const gfx::Rect& selection_rect,
+                              int active_match_ordinal,
+                              bool final_update) {
+  TabContentsWrapper* tcw = TabContentsWrapper::GetCurrentWrapperForContents(
+      tab);
+  if (!tcw || !tcw->find_tab_helper())
+    return;
+
+  tcw->find_tab_helper()->HandleFindReply(request_id, number_of_matches,
+                                          selection_rect, active_match_ordinal,
+                                          final_update);
+}
+
 void Browser::ExecuteCommandWithDisposition(
   int id, WindowOpenDisposition disposition) {
   // No commands are enabled if there is not yet any selected tab.
   // TODO(pkasting): It seems like we should not need this, because either
   // most/all commands should not have been enabled yet anyway or the ones that
   // are enabled should be global, or safe themselves against having no selected
   // tab.  However, Ben says he tried removing this before and got lots of
   // crashes, e.g. from Windows sending WM_COMMANDs at random times during
   // window construction.  This probably could use closer examination someday.
   if (!GetSelectedTabContentsWrapper())
@@ skipping 1235 matching lines @@
 void Browser::WebIntentDispatch(TabContents* tab,
                                 int routing_id,
                                 const string16& action,
                                 const string16& type,
                                 const string16& data,
                                 int intent_id) {
   Browser::WebIntentDispatchHelper(tab, routing_id, action, type, data,
                                    intent_id);
 }
 
+void Browser::FindReply(TabContents* tab,
+                        int request_id,
+                        int number_of_matches,
+                        const gfx::Rect& selection_rect,
+                        int active_match_ordinal,
+                        bool final_update) {
+  Browser::FindReplyHelper(tab, request_id, number_of_matches, selection_rect,

[jam, 2011/09/02 00:05:26]

nit: the Browser:: is redundant

[ananta, 2011/09/02 00:28:28]

Done. I also removed the Browser prefix from other TabContentsDelegate overrides
which call out to similar helpers.

+                           active_match_ordinal, final_update);
+}
+
 void Browser::ExitTabbedFullscreenModeIfNecessary() {
   if (tab_caused_fullscreen_)
     ToggleFullscreenMode();
   else
     NotifyTabOfFullscreenExitIfNecessary();
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 // Browser, TabContentsWrapperDelegate implementation:
 
@@ skipping 1341 matching lines @@
 }
 
 void Browser::ShowSyncSetup() {
   ProfileSyncService* service =
       profile()->GetOriginalProfile()->GetProfileSyncService();
   if (service->HasSyncSetupCompleted())
     ShowOptionsTab(chrome::kSyncSetupSubPage);
   else
     service->ShowLoginDialog();
 }