diff options
author | jdduke@chromium.org <jdduke@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-02-27 02:34:22 +0000 |
---|---|---|
committer | jdduke@chromium.org <jdduke@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-02-27 02:34:22 +0000 |
commit | c265e4d96d01c79aaeed9b1334402d7fdb9e0147 (patch) | |
tree | d1da7a264d39ef73a71eae36c085cb3fe39a0bcd /content/browser/loader | |
parent | ee828d831e93b58b3a79e7dfc27ab8795f7b5fcc (diff) | |
download | chromium_src-c265e4d96d01c79aaeed9b1334402d7fdb9e0147.zip chromium_src-c265e4d96d01c79aaeed9b1334402d7fdb9e0147.tar.gz chromium_src-c265e4d96d01c79aaeed9b1334402d7fdb9e0147.tar.bz2 |
Drop unhandled ResourceHostMsg's in the ResourceDispatcherHostImpl
Certain ResourceHostMsg's may go unhandled in ResourceDispatcherHostImpl,
depending on whether an appropriate ResourceMessageDelegate exists. If it goes
unhandled, it is forwarded to all other filters and from there to the
RenderProcessHostImpl. All of that is needless work as the message will simply
be dropped, so drop the message instead in ResourceDispatcherHostImpl.
BUG=343343
Review URL: https://codereview.chromium.org/164333016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@253692 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/browser/loader')
-rw-r--r-- | content/browser/loader/resource_dispatcher_host_impl.cc | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/content/browser/loader/resource_dispatcher_host_impl.cc b/content/browser/loader/resource_dispatcher_host_impl.cc index 653b2db..c486431 100644 --- a/content/browser/loader/resource_dispatcher_host_impl.cc +++ b/content/browser/loader/resource_dispatcher_host_impl.cc @@ -859,6 +859,10 @@ bool ResourceDispatcherHostImpl::OnMessageReceived( handled = delegate->OnMessageReceived(message, message_was_ok); } } + + // As the unhandled resource message effectively has no consumer, mark it as + // handled to prevent needless propagation through the filter pipeline. + handled = true; } filter_ = NULL; |