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

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

Issue 10640019: Remove the HANDLED_EXTERNALLY status code. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: rebase + fix nits Created 8 years, 3 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/redirect_to_file_resource_handler.cc
diff --git a/content/browser/renderer_host/redirect_to_file_resource_handler.cc b/content/browser/renderer_host/redirect_to_file_resource_handler.cc
index d7f0ea3a7a383178e1bfc1d451f135a0a1a09f6c..a527ac22f581470f66c278ba1c1efba8eea8cb53 100644
--- a/content/browser/renderer_host/redirect_to_file_resource_handler.cc
+++ b/content/browser/renderer_host/redirect_to_file_resource_handler.cc
@@ -61,7 +61,8 @@ bool RedirectToFileResourceHandler::OnResponseStarted(
int request_id,
ResourceResponse* response,
bool* defer) {
- if (response->head.status.is_success()) {
+ if (response->head.error_code == net::OK ||
+ response->head.error_code == net::ERR_IO_PENDING) {
DCHECK(deletable_file_ && !deletable_file_->path().empty());
response->head.download_file_path = deletable_file_->path();
}

Powered by Google App Engine
This is Rietveld 408576698