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

Unified Diff: chrome/browser/chromeos/drive/file_system/remove_operation.h

Issue 68543002: drive: Support offline delete (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Move to drive/other and mark as deleted Created 7 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
Index: chrome/browser/chromeos/drive/file_system/remove_operation.h
diff --git a/chrome/browser/chromeos/drive/file_system/remove_operation.h b/chrome/browser/chromeos/drive/file_system/remove_operation.h
index 4564ef8a53b3433d2b26cbc47ef954f1938b00ef..b9cb2bcc625a12d0dcd15aab935fcbae724b48fc 100644
--- a/chrome/browser/chromeos/drive/file_system/remove_operation.h
+++ b/chrome/browser/chromeos/drive/file_system/remove_operation.h
@@ -19,7 +19,6 @@ class SequencedTaskRunner;
namespace drive {
-class JobScheduler;
class ResourceEntry;
namespace internal {
@@ -31,14 +30,10 @@ namespace file_system {
class OperationObserver;
-// This class encapsulates the drive Remove function. It is responsible for
-// sending the request to the drive API, then updating the local state and
-// metadata to reflect the new state.
class RemoveOperation {
kinaba 2013/11/12 06:21:29 Please keep some brief class comment.
hashimoto 2013/11/25 10:13:58 Done.
public:
RemoveOperation(base::SequencedTaskRunner* blocking_task_runner,
OperationObserver* observer,
- JobScheduler* scheduler,
internal::ResourceMetadata* metadata,
internal::FileCache* cache);
~RemoveOperation();
@@ -53,26 +48,14 @@ class RemoveOperation {
const FileOperationCallback& callback);
private:
- // Part of Remove(). Called after CheckLocalState() completion.
- void RemoveAfterCheckLocalState(const FileOperationCallback& callback,
- const std::string* parent_resource_id,
- const ResourceEntry* entry,
- FileError error);
-
- // Part of Remove(). Called after server-side removal is done.
- void RemoveAfterUpdateRemoteState(
- const FileOperationCallback& callback,
- const base::Callback<FileError(base::FilePath*)>& local_update_task,
- google_apis::GDataErrorCode status);
-
// Part of Remove(). Called after UpdateLocalState() completion.
void RemoveAfterUpdateLocalState(const FileOperationCallback& callback,
+ const std::string* local_id,
const base::FilePath* changed_directory_path,
FileError error);
scoped_refptr<base::SequencedTaskRunner> blocking_task_runner_;
OperationObserver* observer_;
- JobScheduler* scheduler_;
internal::ResourceMetadata* metadata_;
internal::FileCache* cache_;

Powered by Google App Engine
This is Rietveld 408576698