base::ListValue::Append cleanup: pass unique_ptr instead of the released pointer.

google_apis/drive/drive_api_requests.cc

 // Copyright (c) 2012 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 "google_apis/drive/drive_api_requests.h"
 
 #include <stddef.h>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 107 matching lines @@
     const base::Time& modified_date,
     const base::Time& last_viewed_by_me_date,
     const Properties& properties) {
   base::DictionaryValue root;
   if (!title.empty())
     root.SetString("title", title);
 
   // Fill parent link.
   if (!parent_resource_id.empty()) {
     std::unique_ptr<base::ListValue> parents(new base::ListValue);
-    parents->Append(
-        google_apis::util::CreateParentValue(parent_resource_id).release());
+    parents->Append(google_apis::util::CreateParentValue(parent_resource_id));
     root.Set("parents", parents.release());
   }
 
   if (!modified_date.is_null()) {
     root.SetString("modifiedDate",
                    google_apis::util::FormatTimeAsString(modified_date));
   }
 
   if (!last_viewed_by_me_date.is_null()) {
     root.SetString("lastViewedByMeDate", google_apis::util::FormatTimeAsString(
@@ skipping 686 matching lines @@
 bool InitiateUploadNewFileRequest::GetContentData(
     std::string* upload_content_type,
     std::string* upload_content) {
   *upload_content_type = util::kContentTypeApplicationJson;
 
   base::DictionaryValue root;
   root.SetString("title", title_);
 
   // Fill parent link.
   std::unique_ptr<base::ListValue> parents(new base::ListValue);
-  parents->Append(util::CreateParentValue(parent_resource_id_).release());
+  parents->Append(util::CreateParentValue(parent_resource_id_));
   root.Set("parents", parents.release());
 
   if (!modified_date_.is_null())
     root.SetString("modifiedDate", util::FormatTimeAsString(modified_date_));
 
   if (!last_viewed_by_me_date_.is_null()) {
     root.SetString("lastViewedByMeDate",
                    util::FormatTimeAsString(last_viewed_by_me_date_));
   }
 
@@ skipping 39 matching lines @@
   headers.push_back(util::GenerateIfMatchHeader(etag_));
   return headers;
 }
 
 bool InitiateUploadExistingFileRequest::GetContentData(
     std::string* upload_content_type,
     std::string* upload_content) {
   base::DictionaryValue root;
   if (!parent_resource_id_.empty()) {
     std::unique_ptr<base::ListValue> parents(new base::ListValue);
-    parents->Append(util::CreateParentValue(parent_resource_id_).release());
+    parents->Append(util::CreateParentValue(parent_resource_id_));
     root.Set("parents", parents.release());
   }
 
   if (!title_.empty())
     root.SetString("title", title_);
 
   if (!modified_date_.is_null())
     root.SetString("modifiedDate", util::FormatTimeAsString(modified_date_));
 
   if (!last_viewed_by_me_date_.is_null()) {
@@ skipping 546 matching lines @@
     } else if (last_progress_value_ < child->data_offset + child->data_size &&
                child->data_offset + child->data_size < current) {
       child->request->NotifyUploadProgress(source, child->data_size,
                                            child->data_size);
     }
   }
   last_progress_value_ = current;
 }
 }  // namespace drive
 }  // namespace google_apis