Refactor GetChangelistRequest.

chrome/browser/google_apis/drive_api_requests.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/google_apis/drive_api_requests.h"
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/json/json_writer.h"
 #include "base/location.h"
@@ skipping 96 matching lines @@
           scoped_ptr<FileResource>());
       return;
     }
   }
 
   callback.Run(response, file_resource.Pass());
 }
 
 }  // namespace
 
-//============================ GetChangelistRequest ==========================
-
-GetChangelistRequest::GetChangelistRequest(
-    RequestSender* sender,
-    const DriveApiUrlGenerator& url_generator,
-    bool include_deleted,
-    int64 start_changestamp,
-    int max_results,
-    const GetDataCallback& callback)
-    : GetDataRequest(sender, callback),
-      url_generator_(url_generator),
-      include_deleted_(include_deleted),
-      start_changestamp_(start_changestamp),
-      max_results_(max_results) {
-  DCHECK(!callback.is_null());
-}
-
-GetChangelistRequest::~GetChangelistRequest() {}
-
-GURL GetChangelistRequest::GetURL() const {
-  return url_generator_.GetChangelistUrl(
-      include_deleted_, start_changestamp_, max_results_);
-}
-
 namespace drive {
 
 //=============================== FilesGetRequest =============================
 
 FilesGetRequest::FilesGetRequest(
     RequestSender* sender,
     const DriveApiUrlGenerator& url_generator,
     const FileResourceCallback& callback)
     : GetDataRequest(sender,
                      base::Bind(&ParseJsonAndRun<FileResource>, callback)),
@@ skipping 109 matching lines @@
       url_generator_(url_generator) {
   DCHECK(!callback.is_null());
 }
 
 AboutGetRequest::~AboutGetRequest() {}
 
 GURL AboutGetRequest::GetURL() const {
   return url_generator_.GetAboutGetUrl();
 }
 
+//============================ ChangesListRequest ===========================
+
+ChangesListRequest::ChangesListRequest(
+    RequestSender* sender,
+    const DriveApiUrlGenerator& url_generator,
+    const ChangeListCallback& callback)
+    : GetDataRequest(
+          sender,
+          base::Bind(&ParseJsonOnBlockingPoolAndRun<ChangeList>,
+                     make_scoped_refptr(sender->blocking_task_runner()),
+                     callback)),
+      url_generator_(url_generator),
+      include_deleted_(true),
+      max_results_(100),
+      start_change_id_(0) {
+  DCHECK(!callback.is_null());
+}
+
+ChangesListRequest::~ChangesListRequest() {}
+
+GURL ChangesListRequest::GetURL() const {
+  return url_generator_.GetChangesListUrl(
+      include_deleted_, max_results_, page_token_, start_change_id_);
+}
+
 //============================== AppsListRequest ===========================
 
 AppsListRequest::AppsListRequest(
     RequestSender* sender,
     const DriveApiUrlGenerator& url_generator,
     const AppListCallback& callback)
     : GetDataRequest(sender,
                      base::Bind(&ParseJsonAndRun<AppList>, callback)),
       url_generator_(url_generator) {
   DCHECK(!callback.is_null());
@@ skipping 499 matching lines @@
           progress_callback,
           url_generator.GenerateDownloadFileUrl(resource_id),
           output_file_path) {
 }
 
 DownloadFileRequest::~DownloadFileRequest() {
 }
 
 }  // namespace drive
 }  // namespace google_apis