Skip to content

Commit ba56e05

Browse files
authored
Merge pull request #17151 from lvivski/chunk-loading
Reduce the amount of generated code for chunk loading
2 parents 79fe72c + c722c2c commit ba56e05

4 files changed

Lines changed: 216 additions & 210 deletions

File tree

lib/esm/ModuleChunkLoadingRuntimeModule.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,7 @@ class ModuleChunkLoadingRuntimeModule extends RuntimeModule {
127127
"",
128128
"// object to store loaded and loading chunks",
129129
"// undefined = chunk not loaded, null = chunk preloaded/prefetched",
130-
"// [resolve, reject, Promise] = chunk loading, 0 = chunk loaded",
130+
"// [resolve, Promise] = chunk loading, 0 = chunk loaded",
131131
`var installedChunks = ${
132132
stateExpression ? `${stateExpression} = ${stateExpression} || ` : ""
133133
}{`,
@@ -210,7 +210,9 @@ class ModuleChunkLoadingRuntimeModule extends RuntimeModule {
210210
)})])`,
211211
`promises.push(installedChunkData[1] = promise);`
212212
]),
213-
"} else installedChunks[chunkId] = 0;"
213+
hasJsMatcher === true
214+
? "}"
215+
: "} else installedChunks[chunkId] = 0;"
214216
]),
215217
"}"
216218
]),

lib/node/ReadFileChunkLoadingRuntimeModule.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -178,7 +178,9 @@ class ReadFileChunkLoadingRuntimeModule extends RuntimeModule {
178178
"});",
179179
"promises.push(installedChunkData[2] = promise);"
180180
]),
181-
"} else installedChunks[chunkId] = 0;"
181+
hasJsMatcher === true
182+
? "}"
183+
: "} else installedChunks[chunkId] = 0;"
182184
]),
183185
"}"
184186
]),

lib/web/JsonpChunkLoadingRuntimeModule.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,9 @@ class JsonpChunkLoadingRuntimeModule extends RuntimeModule {
189189
)};`,
190190
`${RuntimeGlobals.loadScript}(url, loadingEnded, "chunk-" + chunkId, chunkId);`
191191
]),
192-
"} else installedChunks[chunkId] = 0;"
192+
hasJsMatcher === true
193+
? "}"
194+
: "} else installedChunks[chunkId] = 0;"
193195
]),
194196
"}"
195197
]),

0 commit comments

Comments
 (0)