Set isCancelling flag on WebURLError when request has net::ERR_ABORTED as an error_code

webkit/glue/weburlloader_impl.cc

Index: webkit/glue/weburlloader_impl.cc
diff --git a/webkit/glue/weburlloader_impl.cc b/webkit/glue/weburlloader_impl.cc
index dd5b20bbb8e7b9e39feb8190271d9046ffc1793e..6284de7afe1a471de4cc396f9b41f8fea9f73be0 100644
--- a/webkit/glue/weburlloader_impl.cc
+++ b/webkit/glue/weburlloader_impl.cc
@@ -654,6 +654,8 @@ void WebURLLoaderImpl::Context::OnCompletedRequest(
         error_code = status.os_error();
       }
       WebURLError error;
+      if (status.status() == net::URLRequestStatus::CANCELED)
+        error.isCanceling = true;

[darin (slow to review), 2011/06/15 16:35:03]

The net error code ERR_ABORTED is ordinarily used to convey the canceled state. 
Can you use that?

You can see that when status.status() is HANDLED_EXTERNALLY that we force
error_code to be ERR_ABORTED.  You could do the same thing for the CANCELED
state.

Also, maybe I don't know what you are trying to do because I don't see anywhere
in this CL where you check isCanceling.  Is there another part to this CL?

[darin (slow to review), 2011/06/15 16:48:44]

OK... after some more coffee, I now understand what is going on here.  Sorry for
the noise.

It seems like there are two options here:

1) Test URLRequestStatus::status() against URLRequestStatus::CANCELED as you are
doing here, or...

2) Test URLRequestStatus::os_error() against net::ERR_ABORTED.

It is important to understand that URLRequestStatus existed before we had
net_error_list.h.  In the old days, os_error() was a WinINet error code.  Now,
with net_error_list.h, the various status levels for URLRequestStatus are
largely redundant with actual network error codes.  We should be able to kill
off the status field in fact.

My point is that since net::ERR_ABORTED means "user cancellation", it would be
unfortunate if CANCELED were false when os_error is ERR_ABORTED.  Or, put
another way, it would be unfortunate to have a WebURLError conveying ERR_ABORTED
without isCancellation set to true.

So, I suspect that implementation #2 would actually be better.

       error.domain = WebString::fromUTF8(net::kErrorDomain);
       error.reason = error_code;
       error.unreachableURL = request_.url();