Commit 7dd04dea authored by Hiroshige Hayashizaki's avatar Hiroshige Hayashizaki Committed by Commit Bot

Do not use Resource for setting no_handler_reason

Instead, use ScriptSourceLocationType::kExternalFile,

Bug: 788828
Change-Id: Ibc69d49e372ff08fd5817163e157503f3e45d1ae
Reviewed-on: https://chromium-review.googlesource.com/786094
Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org>
Reviewed-by: default avatarJeremy Roman <jbroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520698}
parent 4a227aa2
......@@ -484,15 +484,14 @@ v8::MaybeLocal<v8::Script> V8ScriptRunner::CompileScript(
case ScriptSourceLocationType::kInlineInsideDocumentWrite:
no_handler_reason = v8::ScriptCompiler::kNoCacheBecauseInDocumentWrite;
break;
case ScriptSourceLocationType::kExternalFile:
no_handler_reason =
v8::ScriptCompiler::kNoCacheBecauseResourceWithNoCacheHandler;
break;
// TODO(leszeks): Possibly differentiate between the other kinds of script
// origin also.
default:
if (resource) {
no_handler_reason =
v8::ScriptCompiler::kNoCacheBecauseResourceWithNoCacheHandler;
} else {
no_handler_reason = v8::ScriptCompiler::kNoCacheBecauseNoResource;
}
no_handler_reason = v8::ScriptCompiler::kNoCacheBecauseNoResource;
break;
}
......
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