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

Unified Diff: chrome/browser/sync_file_system/sync_task_manager.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/sync_task_manager.cc
diff --git a/chrome/browser/sync_file_system/sync_task_manager.cc b/chrome/browser/sync_file_system/sync_task_manager.cc
index 581106b0ba91f83f8117e90db2affc95db11ca1f..bba910b5e56970352c23c828cf6a6efb204c827a 100644
--- a/chrome/browser/sync_file_system/sync_task_manager.cc
+++ b/chrome/browser/sync_file_system/sync_task_manager.cc
@@ -152,8 +152,13 @@ void SyncTaskManager::NotifyTaskDone(
<< " (" << SyncStatusCodeToString(status) << ")"
<< " " << token_->location().ToString();
+ bool task_used_network = false;
+ if (task)
+ task_used_network = task->used_network();
+
if (client_)
- client_->NotifyLastOperationStatus(last_operation_status_);
+ client_->NotifyLastOperationStatus(last_operation_status_,
+ task_used_network);
if (!current_callback_.is_null()) {
SyncStatusCallback callback = current_callback_;
« no previous file with comments | « chrome/browser/sync_file_system/sync_task_manager.h ('k') | chrome/browser/sync_file_system/sync_task_manager_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698