Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1157)

Side by Side Diff: chrome/browser/drive/drive_api_service.cc

Issue 23958006: Use "Files: list" for full fetch. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/browser/chromeos/drive/change_list_loader.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/drive/drive_api_service.h" 5 #include "chrome/browser/drive/drive_api_service.h"
6 6
7 #include <string> 7 #include <string>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 301 matching lines...) Expand 10 before | Expand all | Expand 10 after
312 312
313 std::string DriveAPIService::GetRootResourceId() const { 313 std::string DriveAPIService::GetRootResourceId() const {
314 return kDriveApiRootDirectoryResourceId; 314 return kDriveApiRootDirectoryResourceId;
315 } 315 }
316 316
317 CancelCallback DriveAPIService::GetAllResourceList( 317 CancelCallback DriveAPIService::GetAllResourceList(
318 const GetResourceListCallback& callback) { 318 const GetResourceListCallback& callback) {
319 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 319 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
320 DCHECK(!callback.is_null()); 320 DCHECK(!callback.is_null());
321 321
322 // The simplest way to fetch the all resources list looks files.list method, 322 FilesListRequest* request = new FilesListRequest(
323 // but it seems impossible to know the returned list's changestamp.
324 // Thus, instead, we use changes.list method with includeDeleted=false here.
325 // The returned list should contain only resources currently existing.
326 ChangesListRequest* request = new ChangesListRequest(
327 sender_.get(), url_generator_, 323 sender_.get(), url_generator_,
328 base::Bind(&ConvertChangeListToResourceListOnBlockingPoolAndRun, 324 base::Bind(&ConvertFileListToResourceListOnBlockingPoolAndRun,
329 blocking_task_runner_, callback)); 325 blocking_task_runner_, callback));
330 request->set_include_deleted(false); 326 request->set_max_results(kMaxNumFilesResourcePerRequest);
327 request->set_q("trashed = false"); // Exclude trashed files.
331 return sender_->StartRequestWithRetry(request); 328 return sender_->StartRequestWithRetry(request);
332 } 329 }
333 330
334 CancelCallback DriveAPIService::GetResourceListInDirectory( 331 CancelCallback DriveAPIService::GetResourceListInDirectory(
335 const std::string& directory_resource_id, 332 const std::string& directory_resource_id,
336 const GetResourceListCallback& callback) { 333 const GetResourceListCallback& callback) {
337 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 334 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
338 DCHECK(!directory_resource_id.empty()); 335 DCHECK(!directory_resource_id.empty());
339 DCHECK(!callback.is_null()); 336 DCHECK(!callback.is_null());
340 337
(...skipping 492 matching lines...) Expand 10 before | Expand all | Expand 10 after
833 if (CanSendRequest()) { 830 if (CanSendRequest()) {
834 FOR_EACH_OBSERVER( 831 FOR_EACH_OBSERVER(
835 DriveServiceObserver, observers_, OnReadyToSendRequests()); 832 DriveServiceObserver, observers_, OnReadyToSendRequests());
836 } else if (!HasRefreshToken()) { 833 } else if (!HasRefreshToken()) {
837 FOR_EACH_OBSERVER( 834 FOR_EACH_OBSERVER(
838 DriveServiceObserver, observers_, OnRefreshTokenInvalid()); 835 DriveServiceObserver, observers_, OnRefreshTokenInvalid());
839 } 836 }
840 } 837 }
841 838
842 } // namespace drive 839 } // namespace drive
OLDNEW
« no previous file with comments | « chrome/browser/chromeos/drive/change_list_loader.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698