Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(999)

Unified Diff: content/browser/renderer_host/resource_dispatcher_host_impl.cc

Issue 10640019: Remove the HANDLED_EXTERNALLY status code. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: revert removing DCHECK from URLRequest::DoCancel Created 8 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/resource_dispatcher_host_impl.cc
diff --git a/content/browser/renderer_host/resource_dispatcher_host_impl.cc b/content/browser/renderer_host/resource_dispatcher_host_impl.cc
index a8973534cab184a71ead2c220edcece6ce0dd050..7cc48167136d745a1a6a03310cbff0faaa7b3830 100644
--- a/content/browser/renderer_host/resource_dispatcher_host_impl.cc
+++ b/content/browser/renderer_host/resource_dispatcher_host_impl.cc
@@ -125,11 +125,9 @@ void AbortRequestBeforeItStarts(ResourceMessageFilter* filter,
IPC::Message* sync_result,
int route_id,
int request_id) {
- net::URLRequestStatus status(net::URLRequestStatus::FAILED,
- net::ERR_ABORTED);
if (sync_result) {
SyncLoadResult result;
- result.status = status;
+ result.error_code = net::ERR_ABORTED;
ResourceHostMsg_SyncLoad::WriteReplyParams(sync_result, result);
filter->Send(sync_result);
} else {
@@ -137,7 +135,8 @@ void AbortRequestBeforeItStarts(ResourceMessageFilter* filter,
filter->Send(new ResourceMsg_RequestComplete(
route_id,
request_id,
- status,
+ net::ERR_ABORTED,
+ false,
std::string(), // No security info needed, connection not established.
base::TimeTicks()));
}

Powered by Google App Engine
This is Rietveld 408576698