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

Unified Diff: chrome/renderer/chrome_render_process_observer.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: chrome/renderer/chrome_render_process_observer.cc
diff --git a/chrome/renderer/chrome_render_process_observer.cc b/chrome/renderer/chrome_render_process_observer.cc
index 70e7bed0dc1b86045ece856d9cce855a814053e5..128ecd5f12da4ca4bdb9c2c87e431d1d1a4857e5 100644
--- a/chrome/renderer/chrome_render_process_observer.cc
+++ b/chrome/renderer/chrome_render_process_observer.cc
@@ -76,7 +76,7 @@ class RendererResourceDelegate : public content::ResourceDispatcherDelegate {
virtual webkit_glue::ResourceLoaderBridge::Peer* OnRequestComplete(
webkit_glue::ResourceLoaderBridge::Peer* current_peer,
ResourceType::Type resource_type,
- const net::URLRequestStatus& status) {
+ int error_code) {
// Update the browser about our cache.
// Rate limit informing the host of our cache stats.
if (!weak_factory_.HasWeakPtrs()) {
@@ -87,14 +87,13 @@ class RendererResourceDelegate : public content::ResourceDispatcherDelegate {
base::TimeDelta::FromMilliseconds(kCacheStatsDelayMS));
}
- if (status.status() != net::URLRequestStatus::CANCELED ||
- status.error() == net::ERR_ABORTED) {
+ if (error_code == net::ERR_ABORTED) {
return NULL;
}
// Resource canceled with a specific error are filtered.
return SecurityFilterPeer::CreateSecurityFilterPeerForDeniedRequest(
- resource_type, current_peer, status.error());
+ resource_type, current_peer, error_code);
}
virtual webkit_glue::ResourceLoaderBridge::Peer* OnReceivedResponse(
« no previous file with comments | « chrome/common/extensions/extension_localization_peer_unittest.cc ('k') | chrome/renderer/security_filter_peer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698