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

chrome/test/chromedriver/element_util.cc

 // Copyright (c) 2013 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.
 
 #include "chrome/test/chromedriver/element_util.h"
 
 #include <utility>
 
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
@@ skipping 54 matching lines @@
       !dict_value->GetDouble("width", &width) ||
       !dict_value->GetDouble("height", &height))
     return false;
   rect->origin.x = static_cast<int>(x);
   rect->origin.y = static_cast<int>(y);
   rect->size.width = static_cast<int>(width);
   rect->size.height = static_cast<int>(height);
   return true;
 }
 
-base::Value* CreateValueFrom(const WebRect& rect) {
-  base::DictionaryValue* dict = new base::DictionaryValue();
+std::unique_ptr<base::DictionaryValue> CreateValueFrom(const WebRect& rect) {
+  std::unique_ptr<base::DictionaryValue> dict(new base::DictionaryValue());
   dict->SetInteger("left", rect.X());
   dict->SetInteger("top", rect.Y());
   dict->SetInteger("width", rect.Width());
   dict->SetInteger("height", rect.Height());
   return dict;
 }
 
 Status CallAtomsJs(const std::string& frame,
                    WebView* web_view,
                    const char* const* atom_function,
@@ skipping 113 matching lines @@
   base::StringToInt(border_top_str, &border_top_tmp);
   if (border_left_tmp == -1 || border_top_tmp == -1)
     return Status(kUnknownError, "failed to get border width of element");
   *border_left = border_left_tmp;
   *border_top = border_top_tmp;
   return Status(kOk);
 }
 
 }  // namespace
 
-base::DictionaryValue* CreateElement(const std::string& element_id) {
-  base::DictionaryValue* element = new base::DictionaryValue();
+std::unique_ptr<base::DictionaryValue> CreateElement(
+    const std::string& element_id) {
+  std::unique_ptr<base::DictionaryValue> element(new base::DictionaryValue());
   element->SetString(kElementKey, element_id);
   return element;
 }
 
-base::Value* CreateValueFrom(const WebPoint& point) {
-  base::DictionaryValue* dict = new base::DictionaryValue();
+std::unique_ptr<base::DictionaryValue> CreateValueFrom(const WebPoint& point) {
+  std::unique_ptr<base::DictionaryValue> dict(new base::DictionaryValue());
   dict->SetInteger("x", point.x);
   dict->SetInteger("y", point.y);
   return dict;
 }
 
 Status FindElement(int interval_ms,
                    bool only_one,
                    const std::string* root_element_id,
                    Session* session,
                    WebView* web_view,
@@ skipping 418 matching lines @@
     status = ScrollElementRegionIntoViewHelper(
         rit->parent_frame_id, web_view, frame_element_id,
         WebRect(region_offset, region_size),
         center, frame_element_id, &region_offset);
     if (status.IsError())
       return status;
   }
   *location = region_offset;
   return Status(kOk);
 }