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

Unified Diff: components/reading_list/ios/reading_list_store.h

Issue 2618483003: [Sync] Introduce ModelError for USS error handling. (Closed)
Patch Set: Address comments. Created 3 years, 11 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: components/reading_list/ios/reading_list_store.h
diff --git a/components/reading_list/ios/reading_list_store.h b/components/reading_list/ios/reading_list_store.h
index a32e39de67b1ba3ee526a6af51417f8c0e74e203..4eef45974d689ce88578e65a5e3c23a3ed06281b 100644
--- a/components/reading_list/ios/reading_list_store.h
+++ b/components/reading_list/ios/reading_list_store.h
@@ -8,6 +8,7 @@
#include "base/threading/non_thread_safe.h"
#include "components/reading_list/ios/reading_list_model_storage.h"
#include "components/reading_list/ios/reading_list_store_delegate.h"
+#include "components/sync/model/model_error.h"
#include "components/sync/model/model_type_store.h"
namespace syncer {
@@ -54,7 +55,7 @@ class ReadingListStore : public ReadingListModelStorage,
// combine all change atomically, should save the metadata after the data
// changes, so that this merge will be re-driven by sync if is not completely
// saved during the current run.
- syncer::SyncError MergeSyncData(
+ syncer::ModelError MergeSyncData(
std::unique_ptr<syncer::MetadataChangeList> metadata_change_list,
syncer::EntityDataMap entity_data_map) override;
@@ -63,7 +64,7 @@ class ReadingListStore : public ReadingListModelStorage,
// |metadata_change_list| in case when some of the data changes are filtered
// out, or even be empty in case when a commit confirmation is processed and
// only the metadata needs to persisted.
- syncer::SyncError ApplySyncChanges(
+ syncer::ModelError ApplySyncChanges(
std::unique_ptr<syncer::MetadataChangeList> metadata_change_list,
syncer::EntityChangeList entity_changes) override;
@@ -147,7 +148,7 @@ class ReadingListStore : public ReadingListModelStorage,
syncer::ModelTypeStore::Result result,
std::unique_ptr<syncer::ModelTypeStore::RecordList> entries);
void OnDatabaseSave(syncer::ModelTypeStore::Result result);
- void OnReadAllMetadata(syncer::SyncError sync_error,
+ void OnReadAllMetadata(syncer::ModelError error,
std::unique_ptr<syncer::MetadataBatch> metadata_batch);
void AddEntryToBatch(syncer::MutableDataBatch* batch,
« no previous file with comments | « components/reading_list/ios/reading_list_model_unittest.mm ('k') | components/reading_list/ios/reading_list_store.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698