Remove more usage of the base::ListValue::Append(Value*) overload.

chrome/test/base/javascript_browser_test.cc

Index: chrome/test/base/javascript_browser_test.cc
diff --git a/chrome/test/base/javascript_browser_test.cc b/chrome/test/base/javascript_browser_test.cc
index 022956d460117900ac5cd1793b3c79df0c96a140..3e131f355c9f69f26ec899c327056235d182f983 100644
--- a/chrome/test/base/javascript_browser_test.cc
+++ b/chrome/test/base/javascript_browser_test.cc
@@ -112,11 +112,8 @@ base::string16 JavaScriptBrowserTest::BuildRunTestJSCall(
   arguments.push_back(function_name_arg);
   base::ListValue* baked_argument_list = new base::ListValue();
   ConstValueVector::const_iterator arguments_iterator;
-  for (arguments_iterator = test_func_args.begin();
-       arguments_iterator != test_func_args.end();
-       ++arguments_iterator) {
-    baked_argument_list->Append((*arguments_iterator)->DeepCopy());
-  }
+  for (const auto& arg : test_func_args)

[danakj, 2016/08/27 00:07:14]

auto*?

[dcheng, 2016/08/30 06:47:48]

Done.

+    baked_argument_list->Append(arg->CreateDeepCopy());
   arguments.push_back(baked_argument_list);
   return content::WebUI::GetJavascriptCall(std::string("runTest"),
                                            arguments.get());