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

Unified Diff: chrome/browser/sync/sessions/status_controller_unittest.cc

Issue 7621085: Server directed error handling backend code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: upload before commit. Created 9 years, 4 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
« no previous file with comments | « chrome/browser/sync/sessions/status_controller.cc ('k') | chrome/browser/sync/sessions/sync_session.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/sessions/status_controller_unittest.cc
diff --git a/chrome/browser/sync/sessions/status_controller_unittest.cc b/chrome/browser/sync/sessions/status_controller_unittest.cc
index 668b66bf6d6b252215a7677e9a08671a4b96d0c7..bd7296b04012ff0d0499427fcea39ce13fccd1c6 100644
--- a/chrome/browser/sync/sessions/status_controller_unittest.cc
+++ b/chrome/browser/sync/sessions/status_controller_unittest.cc
@@ -108,21 +108,21 @@ TEST_F(StatusControllerTest, StaysClean) {
TEST_F(StatusControllerTest, ReadYourWrites) {
StatusController status(routes_);
status.increment_num_conflicting_commits_by(1);
- EXPECT_EQ(1, status.error_counters().num_conflicting_commits);
+ EXPECT_EQ(1, status.error().num_conflicting_commits);
status.set_num_consecutive_transient_error_commits(6);
- EXPECT_EQ(6, status.error_counters().consecutive_transient_error_commits);
+ EXPECT_EQ(6, status.error().consecutive_transient_error_commits);
status.increment_num_consecutive_transient_error_commits_by(1);
- EXPECT_EQ(7, status.error_counters().consecutive_transient_error_commits);
+ EXPECT_EQ(7, status.error().consecutive_transient_error_commits);
status.increment_num_consecutive_transient_error_commits_by(0);
- EXPECT_EQ(7, status.error_counters().consecutive_transient_error_commits);
+ EXPECT_EQ(7, status.error().consecutive_transient_error_commits);
status.set_num_consecutive_errors(8);
- EXPECT_EQ(8, status.error_counters().consecutive_errors);
+ EXPECT_EQ(8, status.error().consecutive_errors);
status.increment_num_consecutive_errors();
- EXPECT_EQ(9, status.error_counters().consecutive_errors);
+ EXPECT_EQ(9, status.error().consecutive_errors);
status.increment_num_consecutive_errors_by(2);
- EXPECT_EQ(11, status.error_counters().consecutive_errors);
+ EXPECT_EQ(11, status.error().consecutive_errors);
status.set_num_server_changes_remaining(13);
EXPECT_EQ(13, status.num_server_changes_remaining());
@@ -211,7 +211,7 @@ TEST_F(StatusControllerTest, Unrestricted) {
status.mutable_commit_response();
status.updates_response();
status.mutable_updates_response();
- status.error_counters();
+ status.error();
status.syncer_status();
status.num_server_changes_remaining();
status.commit_ids();
« no previous file with comments | « chrome/browser/sync/sessions/status_controller.cc ('k') | chrome/browser/sync/sessions/sync_session.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698