diff options
author | rlarocque@chromium.org <rlarocque@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-10-16 18:14:43 +0000 |
---|---|---|
committer | rlarocque@chromium.org <rlarocque@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-10-16 18:14:43 +0000 |
commit | 092ba99e28fdec0b84384e0fd59eaa3cd7b31a3b (patch) | |
tree | a3b1992d31f2a291388b7e009afd235d144d0c8f /sync/sessions/sync_session.cc | |
parent | ec06f819c6d2cbbe67821877b4b2c46f7ceb00c3 (diff) | |
download | chromium_src-092ba99e28fdec0b84384e0fd59eaa3cd7b31a3b.zip chromium_src-092ba99e28fdec0b84384e0fd59eaa3cd7b31a3b.tar.gz chromium_src-092ba99e28fdec0b84384e0fd59eaa3cd7b31a3b.tar.bz2 |
sync: Merge {Verify,Process}UpdatesCommand
The VerifyUpdatesCommand was used to ensure the validity of each update
that had been delivered from the server and to prepare for the
ProcessUpdatesCommand. The ProcessUpdatesCommand would then take all
updates that had been successfully verified and move them into the
SERVER_ fields of the sync directory.
It turns out that the logic can be greatly simplified by performing both
tasks within the same command. This patch does not take full advantage
of the merge. This patch is intended simply to merge the two files,
with the goal of performing more significant refactorings later.
BUG=154654
Review URL: https://chromiumcodereview.appspot.com/11091009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@162176 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'sync/sessions/sync_session.cc')
-rw-r--r-- | sync/sessions/sync_session.cc | 18 |
1 files changed, 0 insertions, 18 deletions
diff --git a/sync/sessions/sync_session.cc b/sync/sessions/sync_session.cc index 5586819..5a2b572 100644 --- a/sync/sessions/sync_session.cc +++ b/sync/sessions/sync_session.cc @@ -219,24 +219,6 @@ std::set<ModelSafeGroup> SyncSession::GetEnabledGroupsWithConflicts() const { return enabled_groups_with_conflicts; } -std::set<ModelSafeGroup> - SyncSession::GetEnabledGroupsWithVerifiedUpdates() const { - const std::set<ModelSafeGroup>& enabled_groups = GetEnabledGroups(); - std::set<ModelSafeGroup> enabled_groups_with_verified_updates; - for (std::set<ModelSafeGroup>::const_iterator it = - enabled_groups.begin(); it != enabled_groups.end(); ++it) { - const UpdateProgress* update_progress = - status_controller_->GetUnrestrictedUpdateProgress(*it); - if (update_progress && - (update_progress->VerifiedUpdatesBegin() != - update_progress->VerifiedUpdatesEnd())) { - enabled_groups_with_verified_updates.insert(*it); - } - } - - return enabled_groups_with_verified_updates; -} - namespace { // Returns false iff one of the command results had an error. |