Change ExecuteJavaScript* helper functions in browser_test_utils.{h,cc}

chrome/browser/instant/instant_browsertest.cc

 // Copyright 2012 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/content_settings/host_content_settings_map.h"
 #include "chrome/browser/history/history_service_factory.h"
 #include "chrome/browser/instant/instant_loader.h"
 #include "chrome/browser/instant/instant_model_observer.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/search_engines/template_url_service.h"
@@ skipping 87 matching lines @@
         instant()->GetPreviewContents()->GetRenderProcessHost()->GetHandle(),
         content::RESULT_CODE_KILLED,
         false);
   }
 
   void FocusOmnibox() {
     // If the omnibox already has focus, just notify Instant.
     if (omnibox()->model()->has_focus()) {
       instant()->OmniboxFocusChanged(OMNIBOX_FOCUS_VISIBLE,
                                      OMNIBOX_FOCUS_CHANGE_EXPLICIT, NULL);
-    }
-    else {
+    } else {
       browser()->window()->GetLocationBar()->FocusLocation(false);
     }
   }
 
   void FocusOmniboxAndWaitForInstantSupport() {
     content::WindowedNotificationObserver observer(
         chrome::NOTIFICATION_INSTANT_SUPPORT_DETERMINED,
         content::NotificationService::AllSources());
     FocusOmnibox();
     observer.Wait();
   }
 
   void SetOmniboxText(const std::string& text) {
     FocusOmnibox();
     omnibox()->SetUserText(UTF8ToUTF16(text));
   }
 
   void SetOmniboxTextAndWaitForInstantToShow(const std::string& text) {
     InstantTestModelObserver observer(
         instant()->model(), chrome::search::Mode::MODE_SEARCH_SUGGESTIONS);
     SetOmniboxText(text);
     observer.WaitUntilDesiredPreviewState();
   }
 
-  std::wstring WrapScript(const std::string& script) const {
-    return UTF8ToWide("domAutomationController.send(" + script + ")");
+  std::string WrapScript(const std::string& script) const {
+    return "domAutomationController.send(" + script + ")";
   }
 
   bool GetBoolFromJS(content::RenderViewHost* rvh,
                      const std::string& script,
                      bool* result) WARN_UNUSED_RESULT {
-    return content::ExecuteJavaScriptAndExtractBool(rvh, std::wstring(),
-                                                    WrapScript(script), result);
+    return content::ExecuteJavaScriptAndExtractBool(rvh, "", WrapScript(script),
+                                                    result);
   }
 
   bool GetIntFromJS(content::RenderViewHost* rvh,
                     const std::string& script,
                     int* result) WARN_UNUSED_RESULT {
-    return content::ExecuteJavaScriptAndExtractInt(rvh, std::wstring(),
-                                                   WrapScript(script), result);
+    return content::ExecuteJavaScriptAndExtractInt(rvh, "", WrapScript(script),
+                                                   result);
   }
 
   bool GetStringFromJS(content::RenderViewHost* rvh,
                        const std::string& script,
                        std::string* result) WARN_UNUSED_RESULT {
-    return content::ExecuteJavaScriptAndExtractString(
-        rvh, std::wstring(), WrapScript(script), result);
+    return content::ExecuteJavaScriptAndExtractString(rvh, "",
+                                                      WrapScript(script),
+                                                      result);
   }
 
   bool UpdateSearchState(content::WebContents* contents) WARN_UNUSED_RESULT {
     content::RenderViewHost* rvh = contents->GetRenderViewHost();
     return GetIntFromJS(rvh, "onvisibilitycalls", &onvisibilitycalls_) &&
            GetIntFromJS(rvh, "onchangecalls", &onchangecalls_) &&
            GetIntFromJS(rvh, "onsubmitcalls", &onsubmitcalls_) &&
            GetIntFromJS(rvh, "oncancelcalls", &oncancelcalls_) &&
            GetIntFromJS(rvh, "onresizecalls", &onresizecalls_) &&
            GetStringFromJS(rvh, "value", &value_) &&
            GetBoolFromJS(rvh, "verbatim", &verbatim_) &&
            GetIntFromJS(rvh, "height", &height_);
   }
 
   bool ExecuteScript(const std::string& script) WARN_UNUSED_RESULT {
     return content::ExecuteJavaScript(
-        instant()->GetPreviewContents()->GetRenderViewHost(), std::wstring(),
-        UTF8ToWide(script));
+        instant()->GetPreviewContents()->GetRenderViewHost(),
+        "",
+        script);
   }
 
   bool CheckVisibilityIs(content::WebContents* contents,
                          bool expected) WARN_UNUSED_RESULT {
     bool actual = !expected;  // Purposely start with a mis-match.
     // We can only use ASSERT_*() in a method that returns void, hence this
     // convoluted check.
     return GetBoolFromJS(contents->GetRenderViewHost(),
                          "!document.webkitHidden", &actual) &&
            actual == expected;
@@ skipping 899 matching lines @@
   SetOmniboxTextAndWaitForInstantToShow("q");
   EXPECT_EQ(ASCIIToUTF16("query suggestion"), omnibox()->GetText());
 
   // Kill the instant renderer and wait for instant support again.
   KillInstantRenderView();
   FocusOmniboxAndWaitForInstantSupport();
 
   SetOmniboxTextAndWaitForInstantToShow("qu");
   EXPECT_EQ(ASCIIToUTF16("query suggestion"), omnibox()->GetText());
 }