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

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: Fix test 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 b1cab715eda4cb8e8daf0d199550287be6b35f92..5382f45a5d1a50d6c14e9885ec0c928b19135e52 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.
« no previous file with comments | « chrome/browser/sync/engine/apply_updates_command_unittest.cc ('k') | chrome/browser/sync/engine/conflict_resolver.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698