Return CancelCallbackOnce in drive GetAllFileList

Return of GetAllFileList is not used anywhere so it's safe to convert to
CancelCallbackOnce.

Bug: 1007686
Change-Id: I57ff303b8d34fab9926199615a95eb0e81937e31
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2304952Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Commit-Queue: Josip Sokcevic <sokcevic@google.com>
Cr-Commit-Position: refs/heads/master@{#790649}

chrome/browser/sync_file_system/drive_backend/drive_service_on_worker.cc

@@ -306,11 +306,11 @@ google_apis::CancelCallbackOnce DriveServiceOnWorker::GetAllTeamDriveList(
   return google_apis::CancelCallbackOnce();
 }
 
-google_apis::CancelCallback DriveServiceOnWorker::GetAllFileList(
+google_apis::CancelCallbackOnce DriveServiceOnWorker::GetAllFileList(
     const std::string& team_drive_id,
     const google_apis::FileListCallback& callback) {
   NOTREACHED();
-  return google_apis::CancelCallback();
+  return google_apis::CancelCallbackOnce();
 }
 
 google_apis::CancelCallback DriveServiceOnWorker::Search(

chrome/browser/sync_file_system/drive_backend/drive_service_on_worker.h

@@ -117,7 +117,7 @@ class DriveServiceOnWorker : public drive::DriveServiceInterface {
   void ClearRefreshToken() override;
   google_apis::CancelCallbackOnce GetAllTeamDriveList(
       google_apis::TeamDriveListCallback callback) override;
-  google_apis::CancelCallback GetAllFileList(
+  google_apis::CancelCallbackOnce GetAllFileList(
       const std::string& team_drive_id,
       const google_apis::FileListCallback& callback) override;
   google_apis::CancelCallback Search(

components/drive/service/drive_api_service.cc

@@ -294,7 +294,7 @@ CancelCallbackOnce DriveAPIService::GetAllTeamDriveList(
   return sender_->StartRequestWithAuthRetry(std::move(request));
 }
 
-CancelCallback DriveAPIService::GetAllFileList(
+CancelCallbackOnce DriveAPIService::GetAllFileList(
     const std::string& team_drive_id,
     const FileListCallback& callback) {
   DCHECK(thread_checker_.CalledOnValidThread());

components/drive/service/drive_api_service.h

@@ -131,7 +131,7 @@ class DriveAPIService : public DriveServiceInterface,
   std::string GetRootResourceId() const override;
   google_apis::CancelCallbackOnce GetAllTeamDriveList(
       google_apis::TeamDriveListCallback callback) override;
-  google_apis::CancelCallback GetAllFileList(
+  google_apis::CancelCallbackOnce GetAllFileList(
       const std::string& team_drive_id,
       const google_apis::FileListCallback& callback) override;
   google_apis::CancelCallback GetFileListInDirectory(

components/drive/service/drive_service_interface.h

@@ -217,7 +217,7 @@ class DriveServiceInterface : public DriveServiceBatchOperationsInterface {
   // default corpus, otherwise will fetch the file list for the specified
   // team drive.
   // |callback| must not be null.
-  virtual google_apis::CancelCallback GetAllFileList(
+  virtual google_apis::CancelCallbackOnce GetAllFileList(
       const std::string& team_drive_id,
       const google_apis::FileListCallback& callback) = 0;
 

components/drive/service/dummy_drive_service.cc

@@ -58,10 +58,10 @@ CancelCallbackOnce DummyDriveService::GetAllTeamDriveList(
   return CancelCallbackOnce();
 }
 
-CancelCallback DummyDriveService::GetAllFileList(
+CancelCallbackOnce DummyDriveService::GetAllFileList(
     const std::string& team_drive_id,
     const FileListCallback& callback) {
-  return CancelCallback();
+  return CancelCallbackOnce();
 }
 
 CancelCallback DummyDriveService::GetFileListInDirectory(

components/drive/service/dummy_drive_service.h

@@ -35,7 +35,7 @@ class DummyDriveService : public DriveServiceInterface {
   std::string GetRootResourceId() const override;
   google_apis::CancelCallbackOnce GetAllTeamDriveList(
       google_apis::TeamDriveListCallback callback) override;
-  google_apis::CancelCallback GetAllFileList(
+  google_apis::CancelCallbackOnce GetAllFileList(
       const std::string& team_drive_id,
       const google_apis::FileListCallback& callback) override;
   google_apis::CancelCallback GetFileListInDirectory(

components/drive/service/fake_drive_service.cc

@@ -402,7 +402,7 @@ CancelCallbackOnce FakeDriveService::GetAllTeamDriveList(
   return CancelCallbackOnce();
 }
 
-CancelCallback FakeDriveService::GetAllFileList(
+CancelCallbackOnce FakeDriveService::GetAllFileList(
     const std::string& team_drive_id,
     const FileListCallback& callback) {
   DCHECK(thread_checker_.CalledOnValidThread());
@@ -420,7 +420,7 @@ CancelCallback FakeDriveService::GetAllFileList(
                         0,  // start offset
                         default_max_results_, &file_list_load_count_,
                         base::BindOnce(&FileListCallbackAdapter, callback));
-  return CancelCallback();
+  return CancelCallbackOnce();
 }
 
 CancelCallback FakeDriveService::GetFileListInDirectory(

components/drive/service/fake_drive_service.h

@@ -135,7 +135,7 @@ class FakeDriveService : public DriveServiceInterface {
   void ClearRefreshToken() override;
   google_apis::CancelCallbackOnce GetAllTeamDriveList(
       google_apis::TeamDriveListCallback callback) override;
-  google_apis::CancelCallback GetAllFileList(
+  google_apis::CancelCallbackOnce GetAllFileList(
       const std::string& team_drive_id,
       const google_apis::FileListCallback& callback) override;
   google_apis::CancelCallback GetFileListInDirectory(