Revert of Revert of Drive: Let DriveUploader use batch request API. (patchset #1...
commit09b621eaa36477b9b157bbb12fc83865fc6da4bc
authorhirono <hirono@chromium.org>
Wed, 13 May 2015 10:19:24 +0000 (13 03:19 -0700)
committerCommit bot <commit-bot@chromium.org>
Wed, 13 May 2015 10:19:34 +0000 (13 10:19 +0000)
treeb0f4fc4460a213fe9bebbc7d5368fb87787acc51
parente52bf693ef07aa9977ae9c3a60a60a56382c98f5
Revert of Revert of Drive: Let DriveUploader use batch request API. (patchset #1 id:1 of https://codereview.chromium.org/1135353004/)

Reason for revert:
The failure is not caused by the patch. MigrationSingleClientTest.AllTypesIndividually looks flaky.

Original issue's description:
> Revert of Drive: Let DriveUploader use batch request API. (patchset #7 id:120001 of https://codereview.chromium.org/1134633003/)
>
> Reason for revert:
> MigrationSingleClientTest.AllTypesIndividually is broken by this change on Linux GN:
>
> http://build.chromium.org/p/chromium.linux/builders/Linux%20GN/builds/28327
>
> Original issue's description:
> > Drive: Let DriveUploader use batch request API.
> >
> > The CL lets DriveUploader use batch request API. DriveUploader does asynchronous
> > preparation before calling API on service interface. To hold batch request
> > during the asynchronous preparation, the CL introduce refcounted hepler class,
> > which manages life time of batch request configurator.
> >
> > BUG=451917
> > TEST=DriveUploaderTest
> >
> > Committed: https://crrev.com/9cf453b74c8790f6d7a91f958ec998a9fa5a1fa0
> > Cr-Commit-Position: refs/heads/master@{#329599}
>
> TBR=kinaba@chromium.org,tzik@chromium.org,hirono@chromium.org
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=451917
>
> Committed: https://crrev.com/e244812ae9a09d001f24f9aaaf780c80f976d790
> Cr-Commit-Position: refs/heads/master@{#329610}

TBR=kinaba@chromium.org,tzik@chromium.org,sergeyv@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=451917

Review URL: https://codereview.chromium.org/1138883004

Cr-Commit-Position: refs/heads/master@{#329613}
chrome/browser/chromeos/drive/job_queue.cc
chrome/browser/chromeos/drive/job_scheduler.cc
chrome/browser/drive/drive_uploader.cc
chrome/browser/drive/drive_uploader.h
chrome/browser/drive/drive_uploader_unittest.cc
chrome/browser/sync_file_system/drive_backend/drive_uploader_on_worker.cc
chrome/browser/sync_file_system/drive_backend/drive_uploader_on_worker.h
chrome/browser/sync_file_system/drive_backend/fake_drive_uploader.cc
chrome/browser/sync_file_system/drive_backend/fake_drive_uploader.h
google_apis/drive/drive_api_requests.cc
google_apis/drive/drive_api_requests_unittest.cc