sync: remove IncrementErrorCount from ServerConnectionManager.

chrome/browser/sync/engine/net/server_connection_manager.h

 // 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.
 
 #ifndef CHROME_BROWSER_SYNC_ENGINE_NET_SERVER_CONNECTION_MANAGER_H_
 #define CHROME_BROWSER_SYNC_ENGINE_NET_SERVER_CONNECTION_MANAGER_H_
 #pragma once
 
 #include <iosfwd>
 #include <string>
@@ skipping 13 matching lines @@
 }
 
 namespace sync_pb {
 class ClientToServerMessage;
 }
 
 namespace browser_sync {
 
 class ClientToServerMessage;
 
-// How many connection errors are accepted before network handles are closed
-// and reopened.
-static const int32 kMaxConnectionErrorsBeforeReset = 10;
-
 static const int32 kUnsetResponseCode = -1;
 static const int32 kUnsetContentLength = -1;
 static const int32 kUnsetPayloadLength = -1;
 
 // HttpResponse gathers the relevant output properties of an HTTP request.
 // Depending on the value of the server_status code, response_code, and
 // content_length may not be valid.
 struct HttpResponse {
   enum ServerConnectionCode {
     // For uninitialized state.
@@ skipping 244 matching lines @@
 
  protected:
   inline std::string proto_sync_path() const {
     return proto_sync_path_;
   }
 
   std::string get_time_path() const {
     return get_time_path_;
   }
 
-  // Called wherever a failure should be taken as an indication that we may
-  // be experiencing connection difficulties.
-  virtual bool IncrementErrorCount();
-
   // NOTE: Tests rely on this protected function being virtual.
   //
   // Internal PostBuffer base function.
   virtual bool PostBufferToPath(PostBufferParams*,
                                 const std::string& path,
                                 const std::string& auth_token,
                                 ScopedServerStatusWatcher* watcher);
 
   // Helper to check terminated flags and build a Connection object, installing
   // it as the |active_connection_|.  If this ServerConnectionManager has been
@@ skipping 22 matching lines @@
   // The paths we post to.
   std::string proto_sync_path_;
   std::string get_time_path_;
 
   // The auth token to use in authenticated requests. Set by the AuthWatcher.
   std::string auth_token_;
 
   // The previous auth token that is invalid now.
   std::string previously_invalidated_token;
 
-  int error_count_;  // Tracks the number of connection errors.
-
   ObserverList<ServerConnectionEventListener> listeners_;
 
   HttpResponse::ServerConnectionCode server_status_;
   bool server_reachable_;
 
   base::ThreadChecker thread_checker_;
 
   // Protects all variables below to allow bailing out of active connections.
   base::Lock terminate_connection_lock_;
 
@@ skipping 35 matching lines @@
 // settings.
 bool FillMessageWithShareDetails(sync_pb::ClientToServerMessage* csm,
                                  syncable::DirectoryManager* manager,
                                  const std::string& share);
 
 std::ostream& operator<<(std::ostream& s, const struct HttpResponse& hr);
 
 }  // namespace browser_sync
 
 #endif  // CHROME_BROWSER_SYNC_ENGINE_NET_SERVER_CONNECTION_MANAGER_H_