Server directed error handling backend code.

chrome/browser/sync/sessions/status_controller_unittest.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/sync/sessions/sync_session.h"
 #include "chrome/test/sync/engine/test_id_factory.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace browser_sync {
 namespace sessions {
@@ skipping 90 matching lines @@
   status.set_commit_set(commits);
   EXPECT_FALSE(status.TestAndClearIsDirty());
 }
 
 // This test is useful, as simple as it sounds, due to the copy-paste prone
 // nature of status_controller.cc (we have had bugs in the past where a set_foo
 // method was actually setting |bar_| instead!).
 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());
 
   EXPECT_FALSE(status.syncer_status().invalid_store);
   status.set_invalid_store(true);
   EXPECT_TRUE(status.syncer_status().invalid_store);
 
   EXPECT_FALSE(status.syncer_status().syncer_stuck);
   status.set_syncer_stuck(true);
@@ skipping 68 matching lines @@
 // Basic test that non group-restricted state accessors don't cause violations.
 TEST_F(StatusControllerTest, Unrestricted) {
   StatusController status(routes_);
   status.GetUnrestrictedUpdateProgress(
       GROUP_UI)->SuccessfullyAppliedUpdateCount();
   status.mutable_commit_message();
   status.commit_response();
   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();
   status.HasBookmarkCommitActivity();
   status.download_updates_succeeded();
   status.ServerSaysNothingMoreToDownload();
   status.group_restriction();
 }
 
 }  // namespace sessions
 }  // namespace browser_sync