Follow-up on http://codereview.chromium.org/2861019 to address Powel's commen...

chrome/browser/automation/automation_provider.h

Index: chrome/browser/automation/automation_provider.h
===================================================================
--- chrome/browser/automation/automation_provider.h	(revision 50678)
+++ chrome/browser/automation/automation_provider.h	(working copy)
@@ -327,10 +327,6 @@
                       int64 id,
                       bool* success);
 
-  // Util for creating a JSON error return string (dict with key
-  // 'error' and error string value).  No need to quote input.
-  std::string JSONErrorString(std::string err);
-
   // Set window dimensions.
   // Uses the JSON interface for input/output.
   void SetWindowDimensions(Browser* browser,
@@ -442,6 +438,10 @@
                        DictionaryValue* args,
                        IPC::Message* reply_message);
 
+  // Util for creating a JSON error return string (dict with key
+  // 'error' and error string value).  No need to quote input.
+  static std::string JSONErrorString(std::string err);

[Paweł Hajdan Jr., 2010/06/29 17:43:19]

nit: "std::string err" -> "const std::string& err"

+
   // Generic pattern for pyautolib
   // Uses the JSON interface for input/output.
   void SendJSONRequest(int handle,