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

Unified Diff: sync/engine/process_updates_command.cc

Issue 11190013: sync: Remove UpdateProgress and related code (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Formatting improvements Created 8 years, 2 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
« no previous file with comments | « sync/engine/apply_updates_command_unittest.cc ('k') | sync/engine/update_applicator.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sync/engine/process_updates_command.cc
diff --git a/sync/engine/process_updates_command.cc b/sync/engine/process_updates_command.cc
index 22d465c08713410940fbef329df93f5c0c182ea7..15a14cb54cc64e7bb517e7c6e4dff8ec53886c15 100644
--- a/sync/engine/process_updates_command.cc
+++ b/sync/engine/process_updates_command.cc
@@ -30,7 +30,6 @@ namespace syncer {
using sessions::SyncSession;
using sessions::StatusController;
-using sessions::UpdateProgress;
using syncable::GET_BY_ID;
@@ -141,7 +140,6 @@ SyncerError ProcessUpdatesCommand::ModelChangingExecuteImpl(
VerifyResult verify_result = VerifyUpdate(&trans, update,
requested_types,
session->routing_info());
- status->mutable_update_progress()->AddVerifyResult(verify_result, update);
status->increment_num_updates_downloaded_by(1);
if (!UpdateContainsNewVersion(&trans, update))
status->increment_num_reflected_updates_downloaded_by(1);
@@ -158,7 +156,6 @@ SyncerError ProcessUpdatesCommand::ModelChangingExecuteImpl(
process_result == SUCCESS_STORED);
}
- status->mutable_update_progress()->ClearVerifiedUpdates();
return SYNCER_OK;
}
« no previous file with comments | « sync/engine/apply_updates_command_unittest.cc ('k') | sync/engine/update_applicator.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698