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

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: eof Created 8 years, 2 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 c9caa750439ee51379445879cc7c966ee85a47b9..c789dcca10083d417018f92aab29cf2dff8ba4af 100644
--- a/sync/internal_api/public/sessions/model_neutral_state.cc
+++ b/sync/internal_api/public/sessions/model_neutral_state.cc
@@ -30,5 +30,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
« no previous file with comments | « sync/internal_api/public/sessions/model_neutral_state.h ('k') | sync/internal_api/public/sessions/sync_session_snapshot.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698