drive: Introduce chrome/browser/google_apis/drive_common_callbacks.h

chrome/browser/drive/drive_service_interface.h

 // 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.
 
 #ifndef CHROME_BROWSER_DRIVE_DRIVE_SERVICE_INTERFACE_H_
 #define CHROME_BROWSER_DRIVE_DRIVE_SERVICE_INTERFACE_H_
 
 #include <string>
 
 #include "chrome/browser/google_apis/base_requests.h"
+#include "chrome/browser/google_apis/drive_common_callbacks.h"
 
 class Profile;
 
 namespace google_apis {

[tfarina, 2013/06/19 01:27:53]

do you plan to update this namespace to just drive in future?

[satorux1, 2013/06/19 01:30:10]

yes.

 
-class AboutResource;
-class AppList;
-class ResourceEntry;
-class ResourceList;
-
 // Observer interface for DriveServiceInterface.
 class DriveServiceObserver {

[tfarina, 2013/06/19 01:27:53]

this could get this own header file, but I guess it's here for convenience,
since it's tied to DriveServiceInterface. At least this isn't a nested class :)

[satorux1, 2013/06/19 01:30:10]

yes. we removed nested classes from drive related code.

  public:
   // Triggered when the service gets ready to send requests.
   virtual void OnReadyToSendRequests() {}
 
   // Called when the refresh token was found to be invalid.
   virtual void OnRefreshTokenInvalid() {}
 
  protected:
   virtual ~DriveServiceObserver() {}
 };
 
-// Callback used for GetResourceList().
-typedef base::Callback<void(GDataErrorCode error,
-                            scoped_ptr<ResourceList> resource_list)>
-    GetResourceListCallback;
-
-// Callback used for GetResourceEntry().
-typedef base::Callback<void(GDataErrorCode error,
-                            scoped_ptr<ResourceEntry> entry)>
-    GetResourceEntryCallback;
-
-// Callback used for GetAboutResource().
-typedef base::Callback<void(GDataErrorCode error,
-                            scoped_ptr<AboutResource> about_resource)>
-    GetAboutResourceCallback;
-
-// Callback used for GetApplicationInfo().
-typedef base::Callback<void(GDataErrorCode error,
-                            scoped_ptr<AppList> app_list)>
-    GetAppListCallback;
-
-// Callback used for ResumeUpload() and GetUploadStatus().
-typedef base::Callback<void(
-    const UploadRangeResponse& response,
-    scoped_ptr<ResourceEntry> new_entry)> UploadRangeCallback;
-
-// Callback used for AuthorizeApp(). |open_url| is used to open the target
-// file with the authorized app.
-typedef base::Callback<void(GDataErrorCode error,
-                            const GURL& open_url)>
-    AuthorizeAppCallback;
-
-// Closure for canceling a certain request. Each request-issuing method returns
-// this type of closure. If it is called during the request is in-flight, the
-// callback passed with the request is invoked with GDATA_CANCELLED. If the
-// request is already finished, nothing happens.
-typedef base::Closure CancelCallback;
-
 // This defines an interface for sharing by DriveService and MockDriveService
 // so that we can do testing of clients of DriveService.
 //
 // All functions must be called on UI thread. DriveService is built on top of
 // URLFetcher that runs on UI thread.
 class DriveServiceInterface {
  public:
   virtual ~DriveServiceInterface() {}
 
   // Common service:
@@ skipping 285 matching lines @@
   // the provided app. |callback| must not be null.
   virtual CancelCallback AuthorizeApp(
       const std::string& resource_id,
       const std::string& app_id,
       const AuthorizeAppCallback& callback) = 0;
 };
 
 }  // namespace google_apis
 
 #endif  // CHROME_BROWSER_DRIVE_DRIVE_SERVICE_INTERFACE_H_