Implement new methods to replace GetResourceList on GData WAPI.

chrome/browser/google_apis/gdata_wapi_service.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/gdata_wapi_service.h"
 
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
@@ skipping 173 matching lines @@
           url_request_context_getter_,
           url_generator_,
           url,
           static_cast<int>(start_changestamp),
           search_query,
           shared_with_me,
           directory_resource_id,
           callback));
 }
 
+// Because GData WAPI support is expected to be gone somehow soon by migration
+// to the Drive API v2, so we'll reuse GetResourceListOperation to implement
+// following methods, instead of cleaning the operation class.
+
 void GDataWapiService::GetAllResourceList(
     const GetResourceListCallback& callback) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   DCHECK(!callback.is_null());
 
-  // TODO(hidehiko): Implement this.
-  NOTIMPLEMENTED();
+  runner_->StartOperationWithRetry(
+      new GetResourceListOperation(
+          operation_registry(),
+          url_request_context_getter_,
+          url_generator_,
+          GURL(),  // No override url
+          0,  // start changestamp
+          "",  // empty search query
+          false,  // shared_with_me

[hashimoto, 2013/04/04 09:24:45]

nit: Please be consistent to use one of "shared_width_me" or "no shared with
me", not both. (either would be fine)

[hidehiko, 2013/04/05 05:17:56]

Acknowledged.
Fortunately (?), shared_with_me was removed.

+          "",  // no directory resource id
+          callback));
 }
 
 void GDataWapiService::GetResourceListInDirectory(
     const std::string& directory_resource_id,
     const GetResourceListCallback& callback) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   DCHECK(!directory_resource_id.empty());
   DCHECK(!callback.is_null());
 
-  // TODO(hidehiko): Implement this.
-  NOTIMPLEMENTED();
+  runner_->StartOperationWithRetry(
+      new GetResourceListOperation(
+          operation_registry(),
+          url_request_context_getter_,
+          url_generator_,
+          GURL(),  // No override url
+          0,  // start changestamp
+          "",  // empty search query
+          false,  // no shared with me
+          directory_resource_id,
+          callback));
 }
 
 void GDataWapiService::Search(const std::string& search_query,
                               const GetResourceListCallback& callback) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   DCHECK(!search_query.empty());
   DCHECK(!callback.is_null());
 
-  // TODO(hidehiko): Implement this.
-  NOTIMPLEMENTED();
+  runner_->StartOperationWithRetry(
+      new GetResourceListOperation(
+          operation_registry(),
+          url_request_context_getter_,
+          url_generator_,
+          GURL(),  // No override url
+          0,  // start changestamp
+          search_query,
+          false,  // no shared with me
+          "",  // no directory resource id
+          callback));
 }
 
 void GDataWapiService::SearchInDirectory(
     const std::string& search_query,
     const std::string& directory_resource_id,
     const GetResourceListCallback& callback) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   DCHECK(!search_query.empty());
   DCHECK(!directory_resource_id.empty());
   DCHECK(!callback.is_null());
 
-  // TODO(hidehiko): Implement this.
-  NOTIMPLEMENTED();
+  runner_->StartOperationWithRetry(
+      new GetResourceListOperation(
+          operation_registry(),
+          url_request_context_getter_,
+          url_generator_,
+          GURL(),  // No override url
+          0,  // start changestamp
+          search_query,
+          false,  // no shared with me
+          directory_resource_id,
+          callback));
 }
 
 void GDataWapiService::GetChangeList(int64 start_changestamp,
                                      const GetResourceListCallback& callback) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   DCHECK(!callback.is_null());
 
-  // TODO(hidehiko): Implement this.
-  NOTIMPLEMENTED();
+  runner_->StartOperationWithRetry(
+      new GetResourceListOperation(
+          operation_registry(),
+          url_request_context_getter_,
+          url_generator_,
+          GURL(),  // No override url
+          start_changestamp,
+          "",  // empty search query
+          false,  // shared_with_me
+          "",  // no directory resource id
+          callback));
 }
 
 void GDataWapiService::ContinueGetResourceList(
     const GURL& override_url,
     const GetResourceListCallback& callback) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+  DCHECK(!override_url.is_empty());
   DCHECK(!callback.is_null());
 
-  // TODO(hidehiko): Implement this.
-  NOTIMPLEMENTED();
+  runner_->StartOperationWithRetry(
+      new GetResourceListOperation(
+          operation_registry(),
+          url_request_context_getter_,
+          url_generator_,
+          override_url,
+          0,  // start changestamp
+          "",  // empty search query
+          false,  // shared_with_me
+          "",  // no directory resource id
+          callback));
 }
 
 void GDataWapiService::GetResourceEntry(
     const std::string& resource_id,
     const GetResourceEntryCallback& callback) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   DCHECK(!callback.is_null());
 
   runner_->StartOperationWithRetry(
       new GetResourceEntryOperation(
@@ skipping 309 matching lines @@
 }
 
 void GDataWapiService::OnProgressUpdate(
     const OperationProgressStatusList& list) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   FOR_EACH_OBSERVER(
       DriveServiceObserver, observers_, OnProgressUpdate(list));
 }
 
 }  // namespace google_apis