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

Unified Diff: sync/internal_api/public/sessions/model_neutral_state.cc

Issue 10917234: sync: make scheduling logic and job ownership more obvious. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: test + comment + rebase Created 8 years, 3 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: sync/internal_api/public/sessions/model_neutral_state.cc
diff --git a/sync/internal_api/public/sessions/model_neutral_state.cc b/sync/internal_api/public/sessions/model_neutral_state.cc
index 9ae84369211e0efa9e297915aa6c3fe8d4ce033e..2287c911d394766f70092a62ccd7bdf04b1cb9bf 100644
--- a/sync/internal_api/public/sessions/model_neutral_state.cc
+++ b/sync/internal_api/public/sessions/model_neutral_state.cc
@@ -26,5 +26,13 @@ ModelNeutralState::ModelNeutralState()
ModelNeutralState::~ModelNeutralState() {}
+bool HasSyncerError(const ModelNeutralState& state) {
+ const bool get_key_error = SyncerErrorIsError(state.last_get_key_result);
+ const bool download_updates_error =
+ SyncerErrorIsError(state.last_download_updates_result);
+ const bool commit_error = SyncerErrorIsError(state.commit_result);
+ return get_key_error || download_updates_error || commit_error;
+}
+
} // namespace sessions
} // namespace syncer

Powered by Google App Engine
This is Rietveld 408576698