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

Unified Diff: chrome/browser/google_apis/drive_api_operations.h

Issue 11799003: Add DriveApiUrlGenerator. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 7 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/google_apis/drive_api_operations.h
diff --git a/chrome/browser/google_apis/drive_api_operations.h b/chrome/browser/google_apis/drive_api_operations.h
index 718660824c064de3ba1324f5c785222a35ba3ed4..45a91d9b61d80603ad414d408a55a3a24f398406 100644
--- a/chrome/browser/google_apis/drive_api_operations.h
+++ b/chrome/browser/google_apis/drive_api_operations.h
@@ -8,6 +8,7 @@
#include <string>
#include "chrome/browser/google_apis/base_operations.h"
+#include "chrome/browser/google_apis/drive_api_url_generator.h"
namespace net {
class URLRequestContextGetter;
@@ -22,6 +23,7 @@ class GetAboutOperation : public GetDataOperation {
public:
GetAboutOperation(OperationRegistry* registry,
net::URLRequestContextGetter* url_request_context_getter,
+ const DriveApiUrlGenerator& url_generator,
const GetDataCallback& callback);
virtual ~GetAboutOperation();
@@ -30,6 +32,8 @@ class GetAboutOperation : public GetDataOperation {
virtual GURL GetURL() const OVERRIDE;
private:
+ DriveApiUrlGenerator url_generator_;
satorux1 2013/01/08 03:29:13 const?
hidehiko 2013/01/08 03:54:48 Done.
+
DISALLOW_COPY_AND_ASSIGN(GetAboutOperation);
};
@@ -40,6 +44,7 @@ class GetApplistOperation : public GetDataOperation {
public:
GetApplistOperation(OperationRegistry* registry,
net::URLRequestContextGetter* url_request_context_getter,
+ const DriveApiUrlGenerator& url_generator,
const GetDataCallback& callback);
virtual ~GetApplistOperation();
@@ -48,6 +53,8 @@ class GetApplistOperation : public GetDataOperation {
virtual GURL GetURL() const OVERRIDE;
private:
+ DriveApiUrlGenerator url_generator_;
satorux1 2013/01/08 03:29:13 ditto. please check other places too
hidehiko 2013/01/08 03:54:48 Done. Ditto for below similar ones.
+
DISALLOW_COPY_AND_ASSIGN(GetApplistOperation);
};
@@ -64,6 +71,7 @@ class GetChangelistOperation : public GetDataOperation {
GetChangelistOperation(
OperationRegistry* registry,
net::URLRequestContextGetter* url_request_context_getter,
+ const DriveApiUrlGenerator& url_generator,
const GURL& url,
int64 start_changestamp,
const GetDataCallback& callback);
@@ -74,6 +82,7 @@ class GetChangelistOperation : public GetDataOperation {
virtual GURL GetURL() const OVERRIDE;
private:
+ DriveApiUrlGenerator url_generator_;
GURL url_;
int64 start_changestamp_;
@@ -88,6 +97,7 @@ class GetFilelistOperation : public GetDataOperation {
GetFilelistOperation(
OperationRegistry* registry,
net::URLRequestContextGetter* url_request_context_getter,
+ const DriveApiUrlGenerator& url_generator,
const GURL& url,
const std::string& search_string,
const GetDataCallback& callback);
@@ -98,6 +108,7 @@ class GetFilelistOperation : public GetDataOperation {
virtual GURL GetURL() const OVERRIDE;
private:
+ DriveApiUrlGenerator url_generator_;
GURL url_;
std::string search_string_;
@@ -111,6 +122,7 @@ class GetFileOperation : public GetDataOperation {
public:
GetFileOperation(OperationRegistry* registry,
net::URLRequestContextGetter* url_request_context_getter,
+ const DriveApiUrlGenerator& url_generator,
const std::string& file_id,
const GetDataCallback& callback);
virtual ~GetFileOperation();
@@ -120,6 +132,7 @@ class GetFileOperation : public GetDataOperation {
virtual GURL GetURL() const OVERRIDE;
private:
+ DriveApiUrlGenerator url_generator_;
std::string file_id_;
DISALLOW_COPY_AND_ASSIGN(GetFileOperation);
« no previous file with comments | « chrome/browser/chromeos/drive/drive_api_service.cc ('k') | chrome/browser/google_apis/drive_api_operations.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698