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

Unified Diff: chrome/browser/sync/engine/conflict_resolver.h

Issue 8770032: [Sync] Implement encryption-aware conflict resolution. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Split out nigori conflict code and rebase Created 9 years 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/engine/conflict_resolver.h
diff --git a/chrome/browser/sync/engine/conflict_resolver.h b/chrome/browser/sync/engine/conflict_resolver.h
index acc37d908fd55b096e1ab1821f766f08f7bebda7..e506c72cc5c27953cd6504b41714ccee6873818d 100644
--- a/chrome/browser/sync/engine/conflict_resolver.h
+++ b/chrome/browser/sync/engine/conflict_resolver.h
@@ -39,6 +39,19 @@ class ConflictResolver {
FRIEND_TEST_ALL_PREFIXES(SyncerTest,
ConflictResolverMergeOverwritesLocalEntry);
public:
+ // Enumeration of different conflict resolutions. Used for histogramming.
+ enum SimpleConflictResolutions {
+ OVERWRITE_LOCAL, // Resolved by overwriting local changes.
+ OVERWRITE_SERVER, // Resolved by overwriting server changes.
+ UNDELETE, // Resolved by undeleting local item.
+ IGNORE_ENCRYPTION, // Resolved by ignoring an encryption-only server
+ // change.
+ NIGORI_MERGE, // Resolved by merging nigori nodes.
+ CHANGES_MATCH, // Resolved by ignoring both local and server
+ // changes because they matched.
+ CONFLICT_RESOLUTION_SIZE,
+ };
+
ConflictResolver();
~ConflictResolver();
// Called by the syncer at the end of a update/commit cycle.

Powered by Google App Engine
This is Rietveld 408576698