Commit 2dacc374 authored by Eugene But's avatar Eugene But Committed by Commit Bot

Remove crbug.com/546165 and crbug.com/546167 TODOs.

crbug.com/546165 and crbug.com/546167 will be closed as WontFix, per
suggestion from net team here: crbug.com/546165#c8 and during another
codereview here: crrev.com/c/1060646

Bug: 546165, 546167
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I058aa14616538b445ecfa702ba660e5640015721
Reviewed-on: https://chromium-review.googlesource.com/1062714Reviewed-by: default avatarRyan Sleevi <rsleevi@chromium.org>
Commit-Queue: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#559537}
parent 9327148c
......@@ -91,14 +91,12 @@ bool GetNetErrorFromIOSErrorCode(NSInteger ios_error_code,
*net_error_code = net::ERR_INVALID_RESPONSE;
break;
case kCFURLErrorInternationalRoamingOff:
// TODO(crbug.com/546165): Create new net error for disabled intl roaming.
*net_error_code = net::ERR_INTERNET_DISCONNECTED;
break;
case kCFURLErrorCallIsActive:
*net_error_code = net::ERR_CONNECTION_FAILED;
break;
case kCFURLErrorDataNotAllowed:
// TODO(crbug.com/546167): Create new net error for disabled data usage.
*net_error_code = net::ERR_INTERNET_DISCONNECTED;
break;
case kCFURLErrorRequestBodyStreamExhausted:
......
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