Commit 9ec94596 authored by Mythri Alle's avatar Mythri Alle Committed by Commit Bot

Revert "Update ResourceLoader to also fetch cached code for inline scripts"

This reverts commit f17b2cd3.

Reason for revert: <INSERT REASONING HERE>

Original change's description:
> Update ResourceLoader to also fetch cached code for inline scripts
> 
> V8 also stores code caches for inline scripts in the html resources.
> This cl updates ResourceLoader to fetch cached code for inline scripts
> in MainResources.
> 
> Bug: chromium:880212, chromium:812168
> Change-Id: I822453b52897aac292444df20e50937d8198064d
> Reviewed-on: https://chromium-review.googlesource.com/1208391
> Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
> Commit-Queue: Mythri Alle <mythria@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#589137}

TBR=kinuko@chromium.org,mythria@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: chromium:880212, chromium:812168
Change-Id: I4e24815c37378a86e334c8fe88129dff39ee87dd
Reviewed-on: https://chromium-review.googlesource.com/1213094Reviewed-by: default avatarMythri Alle <mythria@chromium.org>
Commit-Queue: Mythri Alle <mythria@chromium.org>
Cr-Commit-Position: refs/heads/master@{#589684}
parent 8a8c0cad
......@@ -306,10 +306,7 @@ bool ResourceLoader::ShouldFetchCodeCache() {
return false;
if (request.DownloadToBlob())
return false;
// If the resource type is script or MainResource (for inline scripts) fetch
// code cache. For others we need not fetch code cache.
if (resource_->GetType() != Resource::Type::kScript &&
resource_->GetType() != Resource::Type::kMainResource)
if (resource_->GetType() != Resource::Type::kScript)
return false;
return true;
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment