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

remoting/host/gcd_rest_client.cc

 // Copyright 2015 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 "remoting/host/gcd_rest_client.h"
 
 #include <stdint.h>
 
 #include <utility>
 
@@ skipping 39 matching lines @@
   //   }]
   // }
   //
   // Note that |now| is deliberately using a double to hold an integer
   // value because |DictionaryValue| doesn't support int64_t values, and
   // GCD doesn't accept fractional values.
   double now = clock_->Now().ToJavaTime();
   std::unique_ptr<base::DictionaryValue> patch_dict(new base::DictionaryValue);
   patch_dict->SetDouble("requestTimeMs", now);
   std::unique_ptr<base::ListValue> patch_list(new base::ListValue);
-  base::DictionaryValue* patch_item = new base::DictionaryValue;
-  patch_list->Append(patch_item);
+  std::unique_ptr<base::DictionaryValue> patch_item(new base::DictionaryValue);
   patch_item->Set("patch", std::move(patch_details));
   patch_item->SetDouble("timeMs", now);
+  patch_list->Append(std::move(patch_item));
   patch_dict->Set("patches", std::move(patch_list));
 
   // Stringify the message.
   std::string patch_string;
   if (!base::JSONWriter::Write(*patch_dict, &patch_string)) {
     LOG(ERROR) << "Error building GCD device state patch.";
     callback.Run(OTHER_ERROR);
     return;
   }
   DLOG(INFO) << "sending state patch: " << patch_string;
@@ skipping 63 matching lines @@
                << ") fetching URL: " << request_url;
     status = NETWORK_ERROR;
   } else {
     LOG(ERROR) << "Error (" << response << ") fetching URL: " << request_url;
   }
 
   FinishCurrentRequest(status);
 }
 
 }  // namespace remoting