Editorial: Update 'wait for all' usage. (#1463)

Fixes #1363.
Fixes #1461.

The authors of this commit are Ms2ger and Jake Archibald.

Author: Ms2ger

docs/index.bs

@@ -1862,7 +1862,7 @@ spec: webappsec-referrer-policy; urlPrefix: https://w3c.github.io/webappsec-refe
 
         1. Let |requests| be an array containing only |request|.
         1. Let |responseArrayPromise| be the result of running the algorithm specified in {{Cache/addAll(requests)}} passing |requests| as the argument.
-        1. Return the result of [=transforming=] |responseArrayPromise| with a fulfillment handler that returns undefined.
+        1. Return the result of [=upon settling|transforming=] |responseArrayPromise| with a fulfillment handler that returns undefined.
     </section>
 
     <section algorithm="cache-addAll">
@@ -1903,8 +1903,8 @@ spec: webappsec-referrer-policy; urlPrefix: https://w3c.github.io/webappsec-refe
                     Note: The cache commit is allowed when the response's body is fully received.
 
             1. Add |responsePromise| to |responsePromises|.
-        1. Let |p| be [=waiting for all=] of |responsePromises|.
-        1. Return the result of [=transforming=] |p| with a fulfillment handler that, when called with argument |responses|, performs the following substeps:
+        1. Let |p| be the result of [=getting a promise to wait for all=] of |responsePromises|.
+        1. Return the result of [=upon settling|transforming=] |p| with a fulfillment handler that, when called with argument |responses|, performs the following substeps:
             1. Let |operations| be an empty [=list=].
             1. Let |index| be zero.
             1. For each |response| in |responses|:
@@ -2071,7 +2071,7 @@ spec: webappsec-referrer-policy; urlPrefix: https://w3c.github.io/webappsec-refe
         1. Else:
             1. Let |promise| be [=a promise resolved with=] undefined.
             1. [=map/For each=] <var ignore>cacheName</var> → |cache| of the [=relevant name to cache map=]:
-                1. Set |promise| to the result of [=transforming=] itself with a fulfillment handler that, when called with argument |response|, performs the following substeps:
+                1. Set |promise| to the result of [=upon settling|transforming=] itself with a fulfillment handler that, when called with argument |response|, performs the following substeps:
                     1. If |response| is not undefined, return |response|.
                     1. Return the result of running the algorithm specified in {{Cache/match(request, options)}} method of {{Cache}} interface with |request| and |options| as the arguments (providing |cache| as thisArgument to the `[[Call]]` internal method of {{Cache/match(request, options)}}.)
             1. Return |promise|.
@@ -2113,7 +2113,7 @@ spec: webappsec-referrer-policy; urlPrefix: https://w3c.github.io/webappsec-refe
       <dfn method for="CacheStorage"><code>delete(|cacheName|)</code></dfn> method *must* run these steps:
 
         1. Let |promise| be the result of running the algorithm specified in {{CacheStorage/has(cacheName)}} method with |cacheName|.
-        1. Return the result of [=transforming=] |promise| with a fulfillment handler that, when called with argument |cacheExists|, performs the following substeps:
+        1. Return the result of [=upon settling|transforming=] |promise| with a fulfillment handler that, when called with argument |cacheExists|, performs the following substeps:
             1. If |cacheExists| is false, then:
                 1. Return false.
             1. Let |cacheJobPromise| be [=a new promise=].
@@ -2725,7 +2725,9 @@ spec: webappsec-referrer-policy; urlPrefix: https://w3c.github.io/webappsec-refe
                   1. <a>Dispatch</a> |e| at |installingWorker|'s [=service worker/global object=].
                   1. *WaitForAsynchronousExtensions*: Run the following substeps <a>in parallel</a>:
                       1. <span id="install-settle-step">Wait until |e| is not [=ExtendableEvent/active=].</span>
-                      1. If |e|'s [=ExtendableEvent/timed out flag=] is set, or the result of [=waiting for all=] of |e|'s [=extend lifetime promises=] rejected, set |installFailed| to true.
+                      1. If |e|'s [=ExtendableEvent/timed out flag=] is set, set |installFailed| to true.
+                      1. Let |p| be the result of [=getting a promise to wait for all=] of |e|'s [=extend lifetime promises=].
+                      1. [=Upon rejection=] of |p|, set |installFailed| to true.
 
                   If |task| is discarded, set |installFailed| to true.
 

docs/v1/index.bs

@@ -1743,7 +1743,7 @@ spec: webappsec-referrer-policy; urlPrefix: https://w3c.github.io/webappsec-refe
 
         1. Let |requests| be an array containing only |request|.
         1. Let |responseArrayPromise| be the result of running the algorithm specified in {{Cache/addAll(requests)}} passing |requests| as the argument.
-        1. Return the result of [=transforming=] |responseArrayPromise| with a fulfillment handler that returns undefined.
+        1. Return the result of [=upon settling|transforming=] |responseArrayPromise| with a fulfillment handler that returns undefined.
     </section>
 
     <section algorithm="cache-addAll">
@@ -1784,8 +1784,8 @@ spec: webappsec-referrer-policy; urlPrefix: https://w3c.github.io/webappsec-refe
                     Note: The cache commit is allowed when the response's body is fully received.
 
             1. Add |responsePromise| to |responsePromises|.
-        1. Let |p| be [=waiting for all=] of |responsePromises|.
-        1. Return the result of [=transforming=] |p| with a fulfillment handler that, when called with argument |responses|, performs the following substeps:
+        1. Let |p| be the result of [=getting a promise to wait for all=] of |responsePromises|.
+        1. Return the result of [=upon settling|transforming=] |p| with a fulfillment handler that, when called with argument |responses|, performs the following substeps:
             1. Let |operations| be an empty [=list=].
             1. Let |index| be zero.
             1. For each |response| in |responses|:
@@ -1949,7 +1949,7 @@ spec: webappsec-referrer-policy; urlPrefix: https://w3c.github.io/webappsec-refe
         1. Else:
             1. Let |promise| be [=a promise resolved with=] undefined.
             1. [=map/For each=] <var ignore>cacheName</var> → |cache| of the [=relevant name to cache map=]:
-                1. Set |promise| to the result of [=transforming=] itself with a fulfillment handler that, when called with argument |response|, performs the following substeps:
+                1. Set |promise| to the result of [=upon settling|transforming=] itself with a fulfillment handler that, when called with argument |response|, performs the following substeps:
                     1. If |response| is not undefined, return |response|.
                     1. Return the result of running the algorithm specified in {{Cache/match(request, options)}} method of {{Cache}} interface with |request| and |options| as the arguments (providing |cache| as thisArgument to the `[[Call]]` internal method of {{Cache/match(request, options)}}.)
             1. Return |promise|.
@@ -1991,7 +1991,7 @@ spec: webappsec-referrer-policy; urlPrefix: https://w3c.github.io/webappsec-refe
       <dfn method for="CacheStorage"><code>delete(|cacheName|)</code></dfn> method *must* run these steps:
 
         1. Let |promise| be the result of running the algorithm specified in {{CacheStorage/has(cacheName)}} method with |cacheName|.
-        1. Return the result of [=transforming=] |promise| with a fulfillment handler that, when called with argument |cacheExists|, performs the following substeps:
+        1. Return the result of [=upon settling|transforming=] |promise| with a fulfillment handler that, when called with argument |cacheExists|, performs the following substeps:
             1. If |cacheExists| is false, then:
                 1. Return false.
             1. Let |cacheJobPromise| be [=a new promise=].
@@ -2541,7 +2541,9 @@ spec: webappsec-referrer-policy; urlPrefix: https://w3c.github.io/webappsec-refe
                   1. <a>Dispatch</a> |e| at |installingWorker|'s [=service worker/global object=].
                   1. *WaitForAsynchronousExtensions*: Run the following substeps <a>in parallel</a>:
                       1. <span id="install-settle-step">Wait until |e| is not [=ExtendableEvent/active=].</span>
-                      1. If |e|'s [=ExtendableEvent/timed out flag=] is set, or the result of [=waiting for all=] of |e|'s [=extend lifetime promises=] rejected, set |installFailed| to true.
+                      1. If |e|'s [=ExtendableEvent/timed out flag=] is set, set |installFailed| to true.
+                      1. Let |p| be the result of [=getting a promise to wait for all=] of |e|'s [=extend lifetime promises=].
+                      1. [=Upon rejection=] of |p|, set |installFailed| to true.
 
                   If |task| is discarded, set |installFailed| to true.