sync/internal_api/http_bridge.cc - Issue 1553433002: [Sync] Sync should run canary cycle after any network related error

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: sync/internal_api/http_bridge.cc

Issue 1553433002: [Sync] Sync should run canary cycle after any network related error (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Adjust initial delay for CONNECTION_UNAVAILABLE error. Created 5 years 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 | « sync/engine/net/server_connection_manager.cc ('k') | sync/internal_api/syncapi_server_connection_manager.h » ('j') | sync/internal_api/syncapi_server_connection_manager.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: sync/internal_api/http_bridge.cc

diff --git a/sync/internal_api/http_bridge.cc b/sync/internal_api/http_bridge.cc

index 842adf3aba06a5b39cbdb2d157c93c1dba1cb8aa..b8d57cc5b26d0bc965134460a85bd4e7afd41eb9 100644

--- a/sync/internal_api/http_bridge.cc

+++ b/sync/internal_api/http_bridge.cc

@@ -512,7 +512,7 @@ void HttpBridge::OnURLFetchTimedOut() {

fetch_state_.request_completed = true;

fetch_state_.request_succeeded = false;

fetch_state_.http_response_code = -1;

- fetch_state_.error_code = net::URLRequestStatus::FAILED;

+ fetch_state_.error_code = net::ERR_TIMED_OUT;

// This method is called by the timer, not the url fetcher implementation,

// so it's safe to delete the fetcher here.