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

Side by Side Diff: chrome/browser/google_apis/drive_api_requests.cc

Issue 23514007: Refactor TrashResourceRequest. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase 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
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/google_apis/drive_api_requests.h" 5 #include "chrome/browser/google_apis/drive_api_requests.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/callback.h" 8 #include "base/callback.h"
9 #include "base/json/json_writer.h" 9 #include "base/json/json_writer.h"
10 #include "base/location.h" 10 #include "base/location.h"
(...skipping 308 matching lines...) Expand 10 before | Expand all | Expand 10 after
319 max_results_(100) { 319 max_results_(100) {
320 DCHECK(!callback.is_null()); 320 DCHECK(!callback.is_null());
321 } 321 }
322 322
323 FilesListRequest::~FilesListRequest() {} 323 FilesListRequest::~FilesListRequest() {}
324 324
325 GURL FilesListRequest::GetURL() const { 325 GURL FilesListRequest::GetURL() const {
326 return url_generator_.GetFilesListUrl(max_results_, page_token_, q_); 326 return url_generator_.GetFilesListUrl(max_results_, page_token_, q_);
327 } 327 }
328 328
329 //============================ FilesTrashRequest =============================
330
331 FilesTrashRequest::FilesTrashRequest(
332 RequestSender* sender,
333 const DriveApiUrlGenerator& url_generator,
334 const FileResourceCallback& callback)
335 : GetDataRequest(sender,
336 base::Bind(&ParseJsonAndRun<FileResource>, callback)),
337 url_generator_(url_generator) {
338 DCHECK(!callback.is_null());
339 }
340
341 FilesTrashRequest::~FilesTrashRequest() {}
342
343 net::URLFetcher::RequestType FilesTrashRequest::GetRequestType() const {
344 return net::URLFetcher::POST;
345 }
346
347 GURL FilesTrashRequest::GetURL() const {
348 return url_generator_.GetFilesTrashUrl(file_id_);
349 }
350
329 //============================== AboutGetRequest ============================= 351 //============================== AboutGetRequest =============================
330 352
331 AboutGetRequest::AboutGetRequest( 353 AboutGetRequest::AboutGetRequest(
332 RequestSender* sender, 354 RequestSender* sender,
333 const DriveApiUrlGenerator& url_generator, 355 const DriveApiUrlGenerator& url_generator,
334 const AboutResourceCallback& callback) 356 const AboutResourceCallback& callback)
335 : GetDataRequest(sender, 357 : GetDataRequest(sender,
336 base::Bind(&ParseJsonAndRun<AboutResource>, callback)), 358 base::Bind(&ParseJsonAndRun<AboutResource>, callback)),
337 url_generator_(url_generator) { 359 url_generator_(url_generator) {
338 DCHECK(!callback.is_null()); 360 DCHECK(!callback.is_null());
(...skipping 172 matching lines...) Expand 10 before | Expand all | Expand 10 after
511 parent_value->SetString("id", parent_resource_id_); 533 parent_value->SetString("id", parent_resource_id_);
512 } 534 }
513 535
514 base::JSONWriter::Write(&root, upload_content); 536 base::JSONWriter::Write(&root, upload_content);
515 537
516 DVLOG(1) << "MoveResource data: " << *upload_content_type << ", [" 538 DVLOG(1) << "MoveResource data: " << *upload_content_type << ", ["
517 << *upload_content << "]"; 539 << *upload_content << "]";
518 return true; 540 return true;
519 } 541 }
520 542
521 //=========================== TrashResourceRequest ===========================
522
523 TrashResourceRequest::TrashResourceRequest(
524 RequestSender* sender,
525 const DriveApiUrlGenerator& url_generator,
526 const std::string& resource_id,
527 const EntryActionCallback& callback)
528 : EntryActionRequest(sender, callback),
529 url_generator_(url_generator),
530 resource_id_(resource_id) {
531 DCHECK(!callback.is_null());
532 }
533
534 TrashResourceRequest::~TrashResourceRequest() {}
535
536 GURL TrashResourceRequest::GetURL() const {
537 return url_generator_.GetFileTrashUrl(resource_id_);
538 }
539
540 net::URLFetcher::RequestType TrashResourceRequest::GetRequestType() const {
541 return net::URLFetcher::POST;
542 }
543
544 //========================== InsertResourceRequest =========================== 543 //========================== InsertResourceRequest ===========================
545 544
546 InsertResourceRequest::InsertResourceRequest( 545 InsertResourceRequest::InsertResourceRequest(
547 RequestSender* sender, 546 RequestSender* sender,
548 const DriveApiUrlGenerator& url_generator, 547 const DriveApiUrlGenerator& url_generator,
549 const std::string& parent_resource_id, 548 const std::string& parent_resource_id,
550 const std::string& resource_id, 549 const std::string& resource_id,
551 const EntryActionCallback& callback) 550 const EntryActionCallback& callback)
552 : EntryActionRequest(sender, callback), 551 : EntryActionRequest(sender, callback),
553 url_generator_(url_generator), 552 url_generator_(url_generator),
(...skipping 225 matching lines...) Expand 10 before | Expand all | Expand 10 after
779 progress_callback, 778 progress_callback,
780 url_generator.GenerateDownloadFileUrl(resource_id), 779 url_generator.GenerateDownloadFileUrl(resource_id),
781 output_file_path) { 780 output_file_path) {
782 } 781 }
783 782
784 DownloadFileRequest::~DownloadFileRequest() { 783 DownloadFileRequest::~DownloadFileRequest() {
785 } 784 }
786 785
787 } // namespace drive 786 } // namespace drive
788 } // namespace google_apis 787 } // namespace google_apis
OLDNEW
« no previous file with comments | « chrome/browser/google_apis/drive_api_requests.h ('k') | chrome/browser/google_apis/drive_api_requests_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698