Merge pull request #8592 from hulkish/hoist-plugin-logic

perf: hoist option logic, reduce code paths in handler
This commit is contained in:
Tobias Koppers 2019-01-07 21:26:40 +01:00 committed by GitHub
commit b133ebeb25
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 11 deletions

View File

@ -23,6 +23,10 @@ class NodeTemplatePlugin {
* @returns {void}
*/
apply(compiler) {
const ChunkLoadingRuntimeModule = this.asyncChunkLoading
? ReadFileChunkLoadingRuntimeModule
: RequireChunkLoadingRuntimeModule;
compiler.hooks.thisCompilation.tap("NodeTemplatePlugin", compilation => {
new NodeChunkTemplatePlugin(compilation).apply(compilation.chunkTemplate);
@ -31,17 +35,10 @@ class NodeTemplatePlugin {
if (onceForChunkSet.has(chunk)) return;
onceForChunkSet.add(chunk);
set.add(RuntimeGlobals.moduleFactories);
if (this.asyncChunkLoading) {
compilation.addRuntimeModule(
chunk,
new ReadFileChunkLoadingRuntimeModule(chunk, set)
);
} else {
compilation.addRuntimeModule(
chunk,
new RequireChunkLoadingRuntimeModule(chunk, set)
);
}
compilation.addRuntimeModule(
chunk,
new ChunkLoadingRuntimeModule(chunk, set)
);
};
compilation.hooks.runtimeRequirementInTree