Commit be97f45e authored by Carlos IL's avatar Carlos IL Committed by Commit Bot

Changed variations net error to INVALID_REDIRECT

Variations service does not follow redirects when fetching seeds,
previously they were cancelled with ERR_ABORTED, this CL changes the
code to ERR_INVALID_REDIRECT to make them distinguishable from other
errors in metrics.

Bug: 902727
Change-Id: Ibaf9144d60333eb9fdfa4fa0c5420e9703ef19b4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1540407Reviewed-by: default avatarBrian White <bcwhite@chromium.org>
Commit-Queue: Carlos IL <carlosil@chromium.org>
Cr-Commit-Position: refs/heads/master@{#644861}
parent 79c38bbb
......@@ -698,7 +698,7 @@ void VariationsService::OnSimpleLoaderCompleteOrRedirect(
initial_request_completed_ = true;
bool is_success = false;
int net_error = net::ERR_ABORTED;
int net_error = net::ERR_INVALID_REDIRECT;
scoped_refptr<net::HttpResponseHeaders> headers;
int response_code = -1;
......@@ -710,7 +710,7 @@ void VariationsService::OnSimpleLoaderCompleteOrRedirect(
// Variations seed fetches should not follow redirects, so if this request was
// redirected, keep the default values for |net_error| and |is_success| (treat
// it as a net::ERR_ABORTED), and the fetch will be cancelled when
// it as a net::ERR_INVALID_REDIRECT), and the fetch will be cancelled when
// pending_seed_request is reset.
if (!was_redirect) {
final_url_was_https =
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment