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

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

Issue 2576663004: content::ResourceType clean up (Closed)
Patch Set: Rebase Created 4 years 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/loader/resource_dispatcher_host_impl.cc
diff --git a/content/browser/loader/resource_dispatcher_host_impl.cc b/content/browser/loader/resource_dispatcher_host_impl.cc
index 415a2092a8e234ddd61578ae3960c6a681cdbf71..a699e0ed2db218b9c051c0c6bc55de1f66b25099 100644
--- a/content/browser/loader/resource_dispatcher_host_impl.cc
+++ b/content/browser/loader/resource_dispatcher_host_impl.cc
@@ -1221,10 +1221,7 @@ void ResourceDispatcherHostImpl::CompleteTransfer(
DCHECK(request_data.transferred_request_child_id != -1 ||
request_data.transferred_request_request_id != -1);
- bool is_navigational_request =
- request_data.resource_type == RESOURCE_TYPE_MAIN_FRAME ||
- request_data.resource_type == RESOURCE_TYPE_SUB_FRAME;
- if (!is_navigational_request) {
+ if (!IsResourceTypeFrame(request_data.resource_type)) {
// Transfers apply only to navigational requests - the renderer seems to
// have sent bogus IPC data.
bad_message::ReceivedBadMessage(

Powered by Google App Engine
This is Rietveld 408576698