Fixing eager execution of asynchronous deps on dynamic imports (#38)

Co-authored-by: Nicolò Ribaudo <nribaudo@igalia.com>

Author: caiolima

spec.emu

@@ -257,17 +257,31 @@ contributors: Nicolò Ribaudo
                 1. Let _namespace_ be GetModuleNamespace(_module_, _phase_, <ins>~all~</ins>).
                 1. Perform ! Call(_promiseCapability_.[[Resolve]], *undefined*, &laquo; _namespace_ &raquo;).
                 1. Return ~unused~.
-              1. If _phase_ is ~defer~, then
-                1. Let _evaluationList_ be GatherAsynchronousTransitiveDependencies(_module_).
-                1. If _evaluationList_ is empty, then
-                  1. Perform _fulfilledClosure_().
-                  1. Return ~unused~.
-                1. Let _asyncDepsEvaluationPromises_ be a new empty List.
-                1. For each Module Record _dep_ of _evaluationList_, append _dep_.Evaluate() to _asyncDepsEvaluationPromises_.
-                1. Let _evaluatePromise_ be SafePerformPromiseAll(_asyncDepsEvaluationPromises_).
-              1. Else,
-                1. Assert: _phase_ is ~evaluation~.
-                1. Let _evaluatePromise_ be _module_.Evaluate().
+              1. <del>If _phase_ is ~defer~, then </del>
+                1. <del>Let _evaluationList_ be GatherAsynchronousTransitiveDependencies(_module_).</del>
+                1. <del>If _evaluationList_ is empty, then</del>
+                  1. <del>Perform _fulfilledClosure_().</del>
+                  1. <del>Return ~unused~.</del>
+                1. <del>Let _asyncDepsEvaluationPromises_ be a new empty List.</del>
+                1. <del>For each Module Record _dep_ of _evaluationList_, append _dep_.Evaluate() to _asyncDepsEvaluationPromises_.</del>
+                1. <del>Let _evaluatePromise_ be SafePerformPromiseAll(_asyncDepsEvaluationPromises_).</del>
+              1. <del>Else,</del>
+                1. <del>Assert: _phase_ is ~evaluation~.</del>
+                1. <del>Let _evaluatePromise_ be _module_.Evaluate().</del>
+              1. <ins>If _phase_ is ~defer~, let _evaluationList_ be GatherAsynchronousTransitiveDependencies(_module_).</ins>
+              1. <ins>Else, let _evaluationList_ be « _module_ ».</ins>
+              1. <ins>Let _optionalIndirectRequests_ be _module_.GetOptionalIndirectExportsModuleRequests(~all~).</ins>
+              1. <ins>Perform ListAppendUnique(_evaluationList_, GatherAsynchronousTransitiveDependenciesForRequests(_module_, _optionalIndirectRequests_, « »)).</ins>
+              1. <ins>If _evaluationList_ is empty, then</ins>
+                1. <ins>Assert: _phase_ is ~defer~.</ins>
+                1. <ins>Perform _fulfilledClosure_().</ins>
+                1. <ins>Return ~unused~.</ins>
+              1. <ins>If the length of _evaluationList_ = 1, then</ins>
+                1. <ins>Let _evaluatePromise_ be _module_.Evaluate().</ins>
+              1. <ins>Else,</ins>
+                1. <ins>Let _evaluationPromises_ be a new empty List.</ins>
+                1. <ins>For each Module Record _dep_ of _evaluationList_, append _dep_.Evaluate() to _evaluationPromises_.</ins>
+                1. <ins>Let _evaluatePromise_ be SafePerformPromiseAll(_evaluationPromises_).</ins>
               1. Let _onFulfilled_ be CreateBuiltinFunction(_fulfilledClosure_, *""*, 0, &laquo; &raquo;).
               1. Perform PerformPromiseThen(_evaluatePromise_, _onFulfilled_, _onRejected_).
               1. Return ~unused~.