Backed out changeset 41ded240a012 (
bug 1342012) for wrong bug number in commit message
--- a/js/src/builtin/Module.js
+++ b/js/src/builtin/Module.js
@@ -294,27 +294,26 @@ function ModuleInstantiate()
let stack = [];
// Steps 4-5
try {
InnerModuleInstantiation(module, stack, 0);
} catch (error) {
for (let i = 0; i < stack.length; i++) {
let m = stack[i];
- if (m.status === MODULE_STATUS_INSTANTIATING) {
- HandleModuleInstantiationFailure(m);
- }
+ assert(m.status === MODULE_STATUS_INSTANTIATING,
+ "Expected instantiating status during failed instantiation");
+ HandleModuleInstantiationFailure(m);
}
// Handle OOM when appending to the stack or over-recursion errors.
- if (stack.length === 0 && module.status === MODULE_STATUS_INSTANTIATING) {
+ if (stack.length === 0)
HandleModuleInstantiationFailure(module);
- }
- assert(module.status !== MODULE_STATUS_INSTANTIATING,
+ assert(module.status === MODULE_STATUS_UNINSTANTIATED,
"Expected uninstantiated status after failed instantiation");
throw error;
}
// Step 6
assert(module.status === MODULE_STATUS_INSTANTIATED ||
module.status === MODULE_STATUS_EVALUATED ||
deleted file mode 100644
--- a/js/src/jit-test/tests/modules/dynamic-import-oom.js
+++ /dev/null
@@ -1,4 +0,0 @@
-// |jit-test| skip-if: !('oomTest' in this)
-
-oomTest(() => import("module1.js"));
-oomTest(() => import("cyclicImport1.js"));