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

Unified Diff: sync/engine/resolve_conflicts_command.cc

Issue 10825137: FYI: Control Data + Per-Device Metadata (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add PER_USER_METADATA, refactor some encryption code Created 8 years, 4 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
Index: sync/engine/resolve_conflicts_command.cc
diff --git a/sync/engine/resolve_conflicts_command.cc b/sync/engine/resolve_conflicts_command.cc
index a1ffbde3f9686fddfeec91ef5817f13f56575020..9d403ed9a8914865c9aec5345662bb3f4317bf5a 100644
--- a/sync/engine/resolve_conflicts_command.cc
+++ b/sync/engine/resolve_conflicts_command.cc
@@ -17,7 +17,13 @@ ResolveConflictsCommand::~ResolveConflictsCommand() {}
std::set<ModelSafeGroup> ResolveConflictsCommand::GetGroupsToChange(
const sessions::SyncSession& session) const {
- return session.GetEnabledGroupsWithConflicts();
+ std::set<ModelSafeGroup> groups_with_conflicts =
+ session.GetEnabledGroupsWithConflicts();
+
+ // GROUP_CONTROL conflicts are not handled by this command.
+ groups_with_conflicts.erase(GROUP_CONTROL);
+
+ return groups_with_conflicts;
}
SyncerError ResolveConflictsCommand::ModelChangingExecuteImpl(

Powered by Google App Engine
This is Rietveld 408576698