Fix memory leak in AsyncExtensionFunction.

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 106 matching lines @@
 
   // Derived classes should implement this method to do their work and return
   // success/failure.
   virtual bool RunImpl() = 0;
 
  protected:
   virtual ~AsyncExtensionFunction() {}
 
   void SendResponse(bool success);
 
+  const ListValue* args_as_list() {
+    return static_cast<ListValue*>(args_.get());
+  }
+  const DictionaryValue* args_as_dictionary() {
+    return static_cast<DictionaryValue*>(args_.get());
+  }
+
   // Note: After Run() returns, dispatcher() can be NULL.  Since these getters
   // rely on dispatcher(), make sure it is valid before using them.
   std::string extension_id();
   Profile* profile();
 
   // The arguments to the API. Only non-null if argument were specified.
-  Value* args_;
+  scoped_ptr<Value> args_;
 
   // The result of the API. This should be populated by the derived class before
   // SendResponse() is called.
   scoped_ptr<Value> result_;
 
   // Any detailed error from the API. This should be populated by the derived
   // class before Run() returns.
   std::string error_;
 
   // Any class that gets a malformed message should set this to true before
@@ skipping 23 matching lines @@
   }
 
  protected:
   virtual ~SyncExtensionFunction() {}
 
  private:
   DISALLOW_COPY_AND_ASSIGN(SyncExtensionFunction);
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_FUNCTION_H_