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

Unified Diff: chrome/browser/sync_file_system/drive_backend/local_to_remote_syncer.cc

Issue 87463002: Only re-enable remote service if the task used the network. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased 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/sync_file_system/drive_backend/local_to_remote_syncer.cc
diff --git a/chrome/browser/sync_file_system/drive_backend/local_to_remote_syncer.cc b/chrome/browser/sync_file_system/drive_backend/local_to_remote_syncer.cc
index 4dab572539b72f19401bb918dc6eebc8b4f46492..7e94d9288c6efebcfea05bd4fca56ceb8e7b630f 100644
--- a/chrome/browser/sync_file_system/drive_backend/local_to_remote_syncer.cc
+++ b/chrome/browser/sync_file_system/drive_backend/local_to_remote_syncer.cc
@@ -268,6 +268,7 @@ void LocalToRemoteSyncer::DeleteRemoteFile(
DCHECK(remote_file_tracker_);
DCHECK(remote_file_tracker_->has_synced_details());
+ set_used_network(true);
drive_service()->DeleteResource(
remote_file_tracker_->file_id(),
remote_file_tracker_->synced_details().etag(),
@@ -320,6 +321,7 @@ void LocalToRemoteSyncer::DidGetMD5ForUpload(
return;
}
+ set_used_network(true);
drive_uploader()->UploadExistingFile(
remote_file_tracker_->file_id(),
local_path_,
@@ -351,6 +353,7 @@ void LocalToRemoteSyncer::DidUploadExistingFile(
void LocalToRemoteSyncer::UpdateRemoteMetadata(
const SyncStatusCallback& callback) {
DCHECK(remote_file_tracker_);
+ set_used_network(true);
drive_service()->GetResourceEntry(
remote_file_tracker_->file_id(),
base::Bind(&LocalToRemoteSyncer::DidGetRemoteMetadata,
@@ -405,6 +408,7 @@ void LocalToRemoteSyncer::DidDeleteForCreateFolder(
void LocalToRemoteSyncer::UploadNewFile(const SyncStatusCallback& callback) {
DCHECK(remote_parent_folder_tracker_);
+ set_used_network(true);
base::FilePath title = fileapi::VirtualPath::BaseName(target_path_);
drive_uploader()->UploadNewFile(
remote_parent_folder_tracker_->file_id(),

Powered by Google App Engine
This is Rietveld 408576698