Adding a UI test to catch the crash described in issue 1341577.This test is disabled, and will be turned on once we fix the issue.I added to TabProxy the ability to do FindNext, which was necessary to reproduce the crash, and changed the automation IPC to take a FindInPageRequest struct, which makes it identical to the IPC we pass to render_view.BUG=1341577

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@1425 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 8 insertions, 3 deletions

diff --git a/chrome/browser/automation/automation_provider.h b/chrome/browser/automation/automation_provider.h
index 6d40253..0b08627 100644
--- a/chrome/browser/automation/automation_provider.h
+++ b/chrome/browser/automation/automation_provider.h
@@ -185,14 +185,19 @@ class AutomationProvider : public base::RefCounted<AutomationProvider>,
   void GetConstrainedWindowBounds(const IPC::Message& message,
                                   int handle);
 
-  // Responds to the FindInPage request, retrieves the search query parameters,
-  // launches an observer to listen for results and issues a StartFind request.
+  // This function has been deprecated, please use HandleFindRequest.
   void HandleFindInPageRequest(const IPC::Message& message,
                                int handle,
                                const std::wstring& find_request,
                                int forward,
                                int match_case);
 
+  // Responds to the FindInPage request, retrieves the search query parameters,
+  // launches an observer to listen for results and issues a StartFind request.
+  void HandleFindRequest(const IPC::Message& message,
+                         int handle,
+                         const FindInPageRequest& request);
+
   // Responds to requests to open the FindInPage window.
   void HandleOpenFindInPageRequest(const IPC::Message& message,
                                    int handle);
@@ -332,7 +337,7 @@ class AutomationProvider : public base::RefCounted<AutomationProvider>,
 
   Profile* profile_;
 
-  DISALLOW_EVIL_CONSTRUCTORS(AutomationProvider);
+  DISALLOW_COPY_AND_ASSIGN(AutomationProvider);
 };
 
 // When life started, the AutomationProvider class was a singleton and was meant