Chromium Code Reviews| Index: content/browser/loader/async_revalidation_manager.cc |
| diff --git a/content/browser/loader/async_revalidation_manager.cc b/content/browser/loader/async_revalidation_manager.cc |
| index abdf659fd228059833fa8c422dedf0099ee207a0..b77bb50b2aa9707cc41802aa5447cf60e13c0c89 100644 |
| --- a/content/browser/loader/async_revalidation_manager.cc |
| +++ b/content/browser/loader/async_revalidation_manager.cc |
| @@ -9,7 +9,6 @@ |
| #include "base/logging.h" |
| #include "content/browser/loader/async_revalidation_driver.h" |
| -#include "content/browser/loader/resource_message_filter.h" |
| #include "content/browser/loader/resource_request_info_impl.h" |
| #include "content/browser/loader/resource_scheduler.h" |
| #include "content/common/resource_messages.h" |
| @@ -58,7 +57,8 @@ void AsyncRevalidationManager::BeginAsyncRevalidation( |
| const ResourceRequestInfoImpl* info = |
| ResourceRequestInfoImpl::ForRequest(&for_request); |
| DCHECK(info); |
| - if (!info->filter()) { |
| + if (!info->requester_info().IsRenderer() || |
|
mmenke
2016/11/21 19:51:55
First check should actually be a DCHECK - the bool
horo
2016/11/22 01:12:07
Done.
|
| + !info->requester_info().filter()) { |
| // The child has gone away and we can no longer get ResourceContext and |
| // URLRequestContext to perform async revalidation. |
| // This can happen in the following cases, ordered from bad to not-so-bad |
| @@ -89,8 +89,8 @@ void AsyncRevalidationManager::BeginAsyncRevalidation( |
| // The embedder of //content needs to ensure that the URLRequestContext object |
| // remains valid until after the ResourceContext object is destroyed. |
| - info->filter()->GetContexts(info->GetResourceType(), &resource_context, |
| - &request_context); |
| + info->requester_info().GetContexts(info->GetResourceType(), &resource_context, |
| + &request_context); |
| AsyncRevalidationKey async_revalidation_key( |
| resource_context, request_context->http_transaction_factory()->GetCache(), |