No longer trigger signin errors on connection_failed for sync.

chrome/browser/signin/signin_tracker_unittest.cc

 // Copyright (c) 2012 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/signin/signin_tracker.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/compiler_specific.h"
 #include "chrome/browser/profiles/profile.h"
@@ skipping 345 matching lines @@
       Return(false));
   EXPECT_CALL(*mock_pss_, sync_initialized()).WillRepeatedly(Return(true));
   // Any error should result in SigninFailed() being called.
   EXPECT_CALL(observer_, SigninFailed(error));
   tracker_.reset(new SigninTracker(profile_.get(), &observer_,
                                    SigninTracker::SERVICES_INITIALIZING));
 }
 
 TEST_F(SigninTrackerTest, SigninFailedWhenInitializing) {
   tracker_.reset();
-  // SigninFailed() should be called.
+  // SigninFailed() should be called because we are not signed in.
   GoogleServiceAuthError error(GoogleServiceAuthError::REQUEST_CANCELED);
   EXPECT_CALL(observer_, SigninFailed(error));
   tracker_.reset(new SigninTracker(profile_.get(), &observer_,
                                    SigninTracker::SERVICES_INITIALIZING));
   tracker_->OnStateChanged();
 }
+
+TEST_F(SigninTrackerTest, ConnectionErrorWhenInitializing) {
+  // SigninFailed() should not be called for a CONNECTION_FAILED error.
+  GoogleServiceAuthError error(GoogleServiceAuthError::CONNECTION_FAILED);
+  ExpectSignedInSyncService(mock_pss_, mock_token_service_, error);
+  mock_signin_manager_->SetAuthenticatedUsername("username@gmail.com");
+  EXPECT_CALL(observer_, SigninFailed(_)).Times(0);
+  tracker_.reset(new SigninTracker(profile_.get(), &observer_,
+                                   SigninTracker::SERVICES_INITIALIZING));
+  tracker_->OnStateChanged();
+}