Make element removal methods in DictionaryValue and ListValue take scoped_ptr's as outparams.

chrome/test/webdriver/webdriver_automation.cc

Index: chrome/test/webdriver/webdriver_automation.cc
diff --git a/chrome/test/webdriver/webdriver_automation.cc b/chrome/test/webdriver/webdriver_automation.cc
index 7ecff746128a8992e952b6d0b4fe748368eb07f7..58d212b25dc9dd0f4680136818ff0e0603f6d0ab 100644
--- a/chrome/test/webdriver/webdriver_automation.cc
+++ b/chrome/test/webdriver/webdriver_automation.cc
@@ -544,15 +544,14 @@ void Automation::ExecuteScript(const WebViewId& view_id,
   if (*error)
     return;
 
-  Value* unscoped_value;
   automation::Error auto_error;
+  scoped_ptr<Value> value;
   if (!SendExecuteJavascriptJSONRequest(automation(), view_locator,
                                         frame_path.value(), script,
-                                        &unscoped_value, &auto_error)) {
+                                        &value, &auto_error)) {
     *error = Error::FromAutomationError(auto_error);
     return;
   }
-  scoped_ptr<Value> value(unscoped_value);
   if (!value->GetAsString(result))
     *error = new Error(kUnknownError, "Execute script did not return string");
 }
@@ -844,7 +843,7 @@ void Automation::Reload(const WebViewId& view_id, Error** error) {
 }
 
 void Automation::GetCookies(const std::string& url,
-                            ListValue** cookies,
+                            scoped_ptr<ListValue>* cookies,
                             Error** error) {
   automation::Error auto_error;
   if (!SendGetCookiesJSONRequest(automation(), url, cookies, &auto_error))