Remove ScopedVector from base::JSONValueConverter

chrome/browser/sync_file_system/drive_backend/folder_creator.cc

Index: chrome/browser/sync_file_system/drive_backend/folder_creator.cc
diff --git a/chrome/browser/sync_file_system/drive_backend/folder_creator.cc b/chrome/browser/sync_file_system/drive_backend/folder_creator.cc
index 8a7ec887f4654586aa3d086953616e3b2a2b3ebf..9dada9ab229f2dfcb433802ebc454b667063ddea 100644
--- a/chrome/browser/sync_file_system/drive_backend/folder_creator.cc
+++ b/chrome/browser/sync_file_system/drive_backend/folder_creator.cc
@@ -56,14 +56,16 @@ void FolderCreator::DidCreateFolder(
 
   drive_service_->SearchByTitle(
       title_, parent_folder_id_,
-      base::Bind(&FolderCreator::DidListFolders,
-                 weak_ptr_factory_.GetWeakPtr(), callback,
-                 base::Passed(ScopedVector<google_apis::FileResource>())));
+      base::Bind(
+          &FolderCreator::DidListFolders, weak_ptr_factory_.GetWeakPtr(),
+          callback,
+          base::Passed(
+              std::vector<std::unique_ptr<google_apis::FileResource>>())));
 }
 
 void FolderCreator::DidListFolders(
     const FileIDCallback& callback,
-    ScopedVector<google_apis::FileResource> candidates,
+    std::vector<std::unique_ptr<google_apis::FileResource>> candidates,
     google_apis::DriveApiErrorCode error,
     std::unique_ptr<google_apis::FileList> file_list) {
   SyncStatusCode status = DriveApiErrorCodeToSyncStatusCode(error);
@@ -79,10 +81,10 @@ void FolderCreator::DidListFolders(
   }
 
   candidates.reserve(candidates.size() + file_list->items().size());
-  candidates.insert(candidates.end(),
-                    file_list->items().begin(),
-                    file_list->items().end());
-  file_list->mutable_items()->weak_clear();
+  candidates.insert(candidates.end(), std::make_move_iterator(
+                                          file_list->mutable_items()->begin()),
+                    std::make_move_iterator(file_list->mutable_items()->end()));

[Avi (use Gerrit), 2017/01/09 17:18:29]

nit: Why not move + back_inserter like above?

[leonhsl(Using Gerrit), 2017/01/10 04:39:36]

Done. I was keeping the old code calling candidates.insert() here :) Also taking
this as a chance would you please let me know more about the difference between
this and 'move+back_inserter'? Thanks.

[Avi (use Gerrit), 2017/01/10 04:48:40]

They should be equivalent, though move_inserter allows you to insert anywhere
while back_inserter is only for the back and front_inserter won't work on a
vector that doesn't have push_front().

Either one is fine here; I was just wondering about consistency.

BTW I didn't know about move_iterators; thank you for teaching me.

+  file_list->mutable_items()->clear();
 
   if (!file_list->next_link().is_empty()) {
     drive_service_->GetRemainingFileList(