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

Unified Diff: sync/sessions/sync_session.cc

Issue 10455012: [Sync] Add support for performing a GetKey on startup. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 8 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
Index: sync/sessions/sync_session.cc
diff --git a/sync/sessions/sync_session.cc b/sync/sessions/sync_session.cc
index 0281dcd469af95f6969e7b6a7ce44d7fd7af2d0b..ac549cac1243c492c5e8d9e738b658221a82d045 100644
--- a/sync/sessions/sync_session.cc
+++ b/sync/sessions/sync_session.cc
@@ -249,10 +249,11 @@ bool IsError(SyncerError error) {
// Returns false iff one of the command results had an error.
bool HadErrors(const ModelNeutralState& state) {
+ const bool get_key_error = IsError(state.last_get_key_result);
const bool download_updates_error =
IsError(state.last_download_updates_result);
const bool commit_error = IsError(state.commit_result);
- return download_updates_error || commit_error;
+ return get_key_error || download_updates_error || commit_error;
}
} // namespace
@@ -262,7 +263,8 @@ bool SyncSession::Succeeded() const {
bool SyncSession::SuccessfullyReachedServer() const {
const ModelNeutralState& state = status_controller_->model_neutral_state();
- bool reached_server = state.last_download_updates_result == SYNCER_OK;
+ bool reached_server = state.last_get_key_result == SYNCER_OK ||
+ state.last_download_updates_result == SYNCER_OK;
// It's possible that we reached the server on one attempt, then had an error
// on the next (or didn't perform some of the server-communicating commands).
// We want to verify that, for all commands attempted, we successfully spoke

Powered by Google App Engine
This is Rietveld 408576698