[Sync] Sync should run canary cycle after any network related error

sync/internal_api/syncapi_server_connection_manager.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 "sync/internal_api/syncapi_server_connection_manager.h"
 
 #include <stdint.h>
 
 #include "net/base/net_errors.h"
 #include "net/http/http_status_code.h"
@@ skipping 35 matching lines @@
   }
 
   // Must be octet-stream, or the payload may be parsed for a cookie.
   http->SetPostPayload("application/octet-stream", payload.length(),
                        payload.data());
 
   // Issue the POST, blocking until it finishes.
   int error_code = 0;
   int response_code = 0;
   if (!http->MakeSynchronousPost(&error_code, &response_code)) {
+    DCHECK_NE(error_code, net::OK);
     DVLOG(1) << "Http POST failed, error returns: " << error_code;
-    response->server_status = HttpResponse::ServerConnectionCodeFromNetError(
-        error_code);
+    response->server_status = HttpResponse::CONNECTION_UNAVAILABLE;
     return false;
   }
 
   // We got a server response, copy over response codes and content.
   response->response_code = response_code;
   response->content_length =
       static_cast<int64_t>(http->GetResponseContentLength());
   response->payload_length =
       static_cast<int64_t>(http->GetResponseContentLength());
   if (response->response_code < 400)
@@ skipping 28 matching lines @@
 }
 
 SyncAPIServerConnectionManager::~SyncAPIServerConnectionManager() {}
 
 ServerConnectionManager::Connection*
 SyncAPIServerConnectionManager::MakeConnection() {
   return new SyncAPIBridgedConnection(this, post_provider_factory_.get());
 }
 
 }  // namespace syncer