Remove an unnecessary TODO.

Bug: 721403
Change-Id: I5f6cf64be0a85e3d5ef8bf88d8dda1deea924635
Reviewed-on: https://chromium-review.googlesource.com/c/1356941Reviewed-by: Clark DuVall <cduvall@chromium.org>
Reviewed-by: rajendrant <rajendrant@chromium.org>
Commit-Queue: Eric Roman <eroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#612907}

components/data_reduction_proxy/content/common/data_reduction_proxy_url_loader_throttle.cc

@@ -144,9 +144,6 @@ void DataReductionProxyURLLoaderThrottle::MarkProxiesAsBad(
   for (const auto& proxy : bad_proxies)
     proxy_list.AddProxyServer(proxy);
 
-  // TODO(https://crbug.com/721403): Does this need to handle the case where
-  // |callback| is never invoked (which can happen on a connection error in
-  // |data_reduction_proxy_|).
   auto callback = base::BindOnce(
       &DataReductionProxyURLLoaderThrottle::OnMarkProxiesAsBadComplete,
       base::Unretained(this));

components/data_reduction_proxy/core/common/data_reduction_proxy_throttle_manager.cc

@@ -40,6 +40,9 @@ void DataReductionProxyThrottleManager::MarkProxiesAsBad(
     base::TimeDelta bypass_duration,
     const net::ProxyList& bad_proxies,
     mojom::DataReductionProxy::MarkProxiesAsBadCallback callback) {
+  // There is no need to handle the case where |callback| is never invoked
+  // (possible on connection error). That would imply disconnection from the
+  // browser, which is not recoverable.
   data_reduction_proxy_->MarkProxiesAsBad(bypass_duration, bad_proxies,
                                           std::move(callback));
 }