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

Side by Side Diff: chrome/browser/sync/engine/conflict_resolver.cc

Issue 351026: Remove context-based dependencies on SyncerSession from ApplyUpdatesCommand.... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 1 month 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2009 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE entry. 3 // found in the LICENSE entry.
4 4
5 #include "chrome/browser/sync/engine/conflict_resolver.h" 5 #include "chrome/browser/sync/engine/conflict_resolver.h"
6 6
7 #include <map> 7 #include <map>
8 #include <set> 8 #include <set>
9 9
10 #include "chrome/browser/sync/engine/syncer.h" 10 #include "chrome/browser/sync/engine/syncer.h"
(...skipping 294 matching lines...) Expand 10 before | Expand all | Expand 10 after
305 // locally_named takes on the ID of the server entry. 305 // locally_named takes on the ID of the server entry.
306 server_named->Put(syncable::ID, trans->directory()->NextId()); 306 server_named->Put(syncable::ID, trans->directory()->NextId());
307 locally_named->Put(syncable::ID, desired_id); 307 locally_named->Put(syncable::ID, desired_id);
308 locally_named->Put(syncable::IS_UNSYNCED, false); 308 locally_named->Put(syncable::IS_UNSYNCED, false);
309 CopyServerFields(server_named, locally_named); 309 CopyServerFields(server_named, locally_named);
310 ClearServerData(server_named); 310 ClearServerData(server_named);
311 server_named->Put(syncable::IS_DEL, true); 311 server_named->Put(syncable::IS_DEL, true);
312 server_named->Put(syncable::BASE_VERSION, 0); 312 server_named->Put(syncable::BASE_VERSION, 0);
313 CHECK(SUCCESS == 313 CHECK(SUCCESS ==
314 SyncerUtil::AttemptToUpdateEntryWithoutMerge( 314 SyncerUtil::AttemptToUpdateEntryWithoutMerge(
315 trans, locally_named, NULL, NULL)); 315 trans, locally_named, NULL));
316 return true; 316 return true;
317 } 317 }
318 318
319 ConflictResolver::ServerClientNameClashReturn 319 ConflictResolver::ServerClientNameClashReturn
320 ConflictResolver::ProcessServerClientNameClash(WriteTransaction* trans, 320 ConflictResolver::ProcessServerClientNameClash(WriteTransaction* trans,
321 MutableEntry* locally_named, 321 MutableEntry* locally_named,
322 MutableEntry* server_named, 322 MutableEntry* server_named,
323 SyncerSession* session) { 323 SyncerSession* session) {
324 if (!locally_named->ExistsOnClientBecauseDatabaseNameIsNonEmpty()) 324 if (!locally_named->ExistsOnClientBecauseDatabaseNameIsNonEmpty())
325 return NO_CLASH; // Locally_named is a server update. 325 return NO_CLASH; // Locally_named is a server update.
(...skipping 420 matching lines...) Expand 10 before | Expand all | Expand 10 after
746 conflict_set_count_map_.erase(i++); 746 conflict_set_count_map_.erase(i++);
747 // METRIC self resolved conflict sets ++. 747 // METRIC self resolved conflict sets ++.
748 } else { 748 } else {
749 ++i; 749 ++i;
750 } 750 }
751 } 751 }
752 return rv; 752 return rv;
753 } 753 }
754 754
755 } // namespace browser_sync 755 } // namespace browser_sync
OLDNEW
« no previous file with comments | « chrome/browser/sync/engine/build_and_process_conflict_sets_command.cc ('k') | chrome/browser/sync/engine/syncer_util.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698