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

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

Issue 11419221: google_apis: Remove document_url_ from EntryActionOperation (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 1 month 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
« no previous file with comments | « chrome/browser/google_apis/base_operations.cc ('k') | chrome/browser/google_apis/gdata_wapi_operations.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/google_apis/gdata_wapi_operations.h
diff --git a/chrome/browser/google_apis/gdata_wapi_operations.h b/chrome/browser/google_apis/gdata_wapi_operations.h
index 36f1ee72985f8f8630362b012841248788eba533..6bb140e1f54533beadd52803d6d77a56cde2607b 100644
--- a/chrome/browser/google_apis/gdata_wapi_operations.h
+++ b/chrome/browser/google_apis/gdata_wapi_operations.h
@@ -157,6 +157,8 @@ class DeleteDocumentOperation : public EntryActionOperation {
virtual std::vector<std::string> GetExtraRequestHeaders() const OVERRIDE;
private:
+ GURL document_url_;
+
DISALLOW_COPY_AND_ASSIGN(DeleteDocumentOperation);
};
@@ -203,7 +205,7 @@ class CopyDocumentOperation : public GetDataOperation {
virtual ~CopyDocumentOperation();
protected:
- // Overridden from GetDataOperation.
+ // Overridden from UrlFetchOperationBase.
virtual net::URLFetcher::RequestType GetRequestType() const OVERRIDE;
// Overridden from UrlFetchOperationBase.
@@ -241,6 +243,7 @@ class RenameResourceOperation : public EntryActionOperation {
std::string* upload_content) OVERRIDE;
private:
+ GURL document_url_;
FilePath::StringType new_name_;
DISALLOW_COPY_AND_ASSIGN(RenameResourceOperation);
@@ -259,7 +262,7 @@ class AuthorizeAppOperation : public GetDataOperation {
virtual ~AuthorizeAppOperation();
protected:
- // Overridden from EntryActionOperation.
+ // Overridden from GetDataOperation.
virtual net::URLFetcher::RequestType GetRequestType() const OVERRIDE;
// Overridden from UrlFetchOperationBase.
@@ -306,6 +309,7 @@ class AddResourceToDirectoryOperation : public EntryActionOperation {
private:
GDataWapiUrlGenerator url_generator_;
GURL parent_content_url_;
+ GURL document_url_;
DISALLOW_COPY_AND_ASSIGN(AddResourceToDirectoryOperation);
};
@@ -314,6 +318,8 @@ class AddResourceToDirectoryOperation : public EntryActionOperation {
// This class performs the operation for adding a document/file/directory
// from a directory.
+//
+// TODO(satorux): Remove |document_url| parameter. crbug.com/163308
class RemoveResourceFromDirectoryOperation : public EntryActionOperation {
public:
RemoveResourceFromDirectoryOperation(OperationRegistry* registry,
@@ -332,6 +338,7 @@ class RemoveResourceFromDirectoryOperation : public EntryActionOperation {
private:
std::string resource_id_;
GURL parent_content_url_;
+ GURL document_url_;
DISALLOW_COPY_AND_ASSIGN(RemoveResourceFromDirectoryOperation);
};
« no previous file with comments | « chrome/browser/google_apis/base_operations.cc ('k') | chrome/browser/google_apis/gdata_wapi_operations.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698