Remove the HANDLED_EXTERNALLY status code.

content/browser/renderer_host/async_resource_handler.cc

Index: content/browser/renderer_host/async_resource_handler.cc
diff --git a/content/browser/renderer_host/async_resource_handler.cc b/content/browser/renderer_host/async_resource_handler.cc
index e37724d5d0086fa2c545270d4973a44870486cfc..061a5e5522a448163f23225ec4d3f66d224c9eba 100644
--- a/content/browser/renderer_host/async_resource_handler.cc
+++ b/content/browser/renderer_host/async_resource_handler.cc
@@ -288,9 +288,19 @@ bool AsyncResourceHandler::OnResponseCompleted(
   base::debug::Alias(url_buf);
 
   TimeTicks completion_time = TimeTicks::Now();
+
+  int error_code = status.error();
+  if (status.status() == net::URLRequestStatus::IO_PENDING)
+    error_code = net::ERR_IO_PENDING;
+  else if (status.status() == net::URLRequestStatus::CANCELED &&
+           error_code == net::OK)
+    error_code = net::ERR_ABORTED;
+  else if (status.status() == net::URLRequestStatus::FAILED &&
+           error_code == net::OK)
+    error_code = net::ERR_FAILED;

[mkosiba (inactive), 2012/06/26 17:01:42]

I was going back and forth on whether to do this here or to have some similar
DCHECK in URLRequestStatus and fix up callers. Ultimately settled for this
approach as fixing up callers to pass would introduce noise to the CL. We could
do it as a preliminary to this one.

   filter_->Send(new ResourceMsg_RequestComplete(routing_id_,
                                                 request_id,
-                                                status,
+                                                error_code,
                                                 security_info,
                                                 completion_time));