Remove wstring from RVH's run Javascript command....

chrome/browser/automation/automation_provider_observers.h

 // Copyright (c) 2010 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.
 
 #ifndef CHROME_BROWSER_AUTOMATION_AUTOMATION_PROVIDER_OBSERVERS_H_
 #define CHROME_BROWSER_AUTOMATION_AUTOMATION_PROVIDER_OBSERVERS_H_
 #pragma once
 
 #include <deque>
 #include <map>
@@ skipping 890 matching lines @@
 
  private:
   // Overriden from DomOperationObserver.
   virtual void OnDomOperationCompleted(const std::string& json);
 
   // Called by the ThumbnailGenerator when the requested snapshot has been
   // generated.
   void OnSnapshotTaken(const SkBitmap& bitmap);
 
   // Helper method to send arbitrary javascript to the renderer for evaluation.
-  void ExecuteScript(const std::wstring& javascript);
+  void ExecuteScript(const std::string& javascript);

[Evan Martin, 2011/02/04 21:37:29]

In theory this can be Unicode.  (In particular, consider passing a JSON object
down.)  It looks like you're using UTF-8 elsewhere.  Should this be a string16
instead?

[Nirnimesh, 2011/02/04 21:59:19]

+1. We have tests that might pass non-ascii

[Avi (use Gerrit), 2011/02/04 23:02:11]

OK

 
   // Helper method to send a response back to the client. Deletes this.
   void SendMessage(bool success);
 
   AutomationProvider* automation_;
   IPC::Message* reply_message_;
   RenderViewHost* render_view_;
   FilePath image_path_;
   bool received_width_;
   gfx::Size entire_page_size_;
@@ skipping 164 matching lines @@
   void RunOnProcessLauncherThread();
   void RunOnUIThread();
 
   scoped_refptr<AutomationProvider> automation_;
   IPC::Message* reply_message_;
 
   DISALLOW_COPY_AND_ASSIGN(WaitForProcessLauncherThreadToGoIdleObserver);
 };
 
 #endif  // CHROME_BROWSER_AUTOMATION_AUTOMATION_PROVIDER_OBSERVERS_H_