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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/sync/glue/session_model_associator.h

Issue 7453014: [Sync] Refactor sync datatype error handling. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix unit test Created 9 years, 5 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
« chrome/browser/sync/api/sync_error.cc ('K') | « chrome/browser/sync/glue/preference_data_type_controller_unittest.cc ('k') | chrome/browser/sync/glue/session_model_associator.cc » ('j') | chrome/browser/sync/syncable/model_type.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/glue/session_model_associator.h
diff --git a/chrome/browser/sync/glue/session_model_associator.h b/chrome/browser/sync/glue/session_model_associator.h
index 05b6f459ac13860790c845aa1fcfc3439d74b505..443159211317b78e1149809f3bf923340ef27b51 100644
--- a/chrome/browser/sync/glue/session_model_associator.h
+++ b/chrome/browser/sync/glue/session_model_associator.h
@@ -117,7 +117,7 @@ class SessionModelAssociator
// with local client data. Processes/reuses any sync nodes owned by this
// client and creates any further sync nodes needed to store local header and
// tab info.
- virtual bool AssociateModels();
+ virtual bool AssociateModels(SyncError* error);
// Initializes the given sync node from the given chrome node id.
// Returns false if no sync node was found for the given chrome node id or
@@ -127,7 +127,7 @@ class SessionModelAssociator
// Clear local sync data buffers. Does not delete sync nodes to avoid
// tombstones. TODO(zea): way to eventually delete orphaned nodes.
- virtual bool DisassociateModels();
+ virtual bool DisassociateModels(SyncError* error);
// Returns the tag used to uniquely identify this machine's session in the
// sync model.
« chrome/browser/sync/api/sync_error.cc ('K') | « chrome/browser/sync/glue/preference_data_type_controller_unittest.cc ('k') | chrome/browser/sync/glue/session_model_associator.cc » ('j') | chrome/browser/sync/syncable/model_type.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698