Modify extension request IPC messages to pass a ListValue instead of a string.

chrome/browser/extensions/extension_function.h

 // Copyright (c) 2009 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_EXTENSIONS_EXTENSION_FUNCTION_H_
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_FUNCTION_H_
 
 #include <string>
 
 #include "base/values.h"
@@ skipping 18 matching lines @@
 // APIs that live beyond a single stack frame.
 class ExtensionFunction : public base::RefCounted<ExtensionFunction> {
  public:
   ExtensionFunction() : request_id_(-1), name_(""), has_callback_(false) {}
   virtual ~ExtensionFunction() {}
 
   // Specifies the name of the function.
   void set_name(const std::string& name) { name_ = name; }
   const std::string name() { return name_; }
 
-  // Specifies the raw arguments to the function, as a JSON-encoded string.
-  virtual void SetArgs(const std::string& args) = 0;
+  // Specifies the raw arguments to the function, as a JSON value.
+  virtual void SetArgs(const Value* args) = 0;
 
   // Retrieves the results of the function as a JSON-encoded string (may
   // be empty).
   virtual const std::string GetResult() = 0;
 
   // Retrieves any error string from the function.
   virtual const std::string GetError() = 0;
 
   void set_dispatcher_peer(ExtensionFunctionDispatcher::Peer* peer) {
     peer_ = peer;
@@ skipping 45 matching lines @@
 // the browser's UI thread*.
 // Note that once Run() returns, dispatcher() can be NULL, so be sure to
 // NULL-check.
 // TODO(aa) Remove this extra level of inheritance once the browser stops
 // parsing JSON (and instead uses custom serialization of Value objects).
 class AsyncExtensionFunction : public ExtensionFunction {
  public:
   AsyncExtensionFunction() : args_(NULL), bad_message_(false) {}
   virtual ~AsyncExtensionFunction() {}
 
-  virtual void SetArgs(const std::string& args);
+  virtual void SetArgs(const Value* args);
   virtual const std::string GetResult();
   virtual const std::string GetError() { return error_; }
   virtual void Run() {
     if (!RunImpl())
       SendResponse(false);
   }
 
   // Derived classes should implement this method to do their work and return
   // success/failure.
   virtual bool RunImpl() = 0;
@@ skipping 42 matching lines @@
 
   virtual void Run() {
     SendResponse(RunImpl());
   }
 
  private:
   DISALLOW_COPY_AND_ASSIGN(SyncExtensionFunction);
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_FUNCTION_H_