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

Unified Diff: chrome/browser/sync/engine/syncer.cc

Issue 9158004: Detect sync server communication errors (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Update for review comments Created 8 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: chrome/browser/sync/engine/syncer.cc
diff --git a/chrome/browser/sync/engine/syncer.cc b/chrome/browser/sync/engine/syncer.cc
index f6490c5831233111a556c58f75ee7ad812cce9b7..bbce9c34cb09d861f3903017ed21651f3f4bb44b 100644
--- a/chrome/browser/sync/engine/syncer.cc
+++ b/chrome/browser/sync/engine/syncer.cc
@@ -144,7 +144,8 @@ void Syncer::SyncShare(sessions::SyncSession* session,
}
case DOWNLOAD_UPDATES: {
DownloadUpdatesCommand download_updates;
- download_updates.Execute(session);
+ session->mutable_status_controller()->set_last_download_updates_result(
+ download_updates.Execute(session));
next_step = PROCESS_CLIENT_COMMAND;
break;
}
@@ -222,14 +223,17 @@ void Syncer::SyncShare(sessions::SyncSession* session,
}
case POST_COMMIT_MESSAGE: {
PostCommitMessageCommand post_commit_command;
- post_commit_command.Execute(session);
+ session->mutable_status_controller()->set_last_post_commit_result(
+ post_commit_command.Execute(session));
next_step = PROCESS_COMMIT_RESPONSE;
break;
}
case PROCESS_COMMIT_RESPONSE: {
session->mutable_status_controller()->reset_num_conflicting_commits();
ProcessCommitResponseCommand process_response_command;
- process_response_command.Execute(session);
+ session->mutable_status_controller()->
+ set_last_process_commit_response_result(
+ process_response_command.Execute(session));
next_step = BUILD_AND_PROCESS_CONFLICT_SETS;
break;
}
« no previous file with comments | « chrome/browser/sync/engine/download_updates_command.cc ('k') | chrome/browser/sync/engine/syncer_proto_util.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698