| 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
|
|
|