Re: [w3c/ServiceWorker] Avoid array manipulation in parallel (#1243)

annevk commented on this pull request.



>                  1. If |client| is an [=environment settings object=], then:
-                    1. If |client| is not a <a>secure context</a>, continue to the next iteration of the loop.
-                1. Else:
-                    1. If |client|’s <a>creation URL</a> is not a <a>potentially trustworthy URL</a>, continue to the next iteration of the loop.
-                1. If |options|.{{ClientQueryOptions/includeUncontrolled}} is false, then:
-                    1. If |client|'s <a>active service worker</a> is not the associated [=ServiceWorkerGlobalScope/service worker=], continue to the next iteration of the loop.
-                1. If |options|.{{ClientQueryOptions/includeReserved}} is false, then:
-                    1. If |client|'s [=environment/execution ready flag=] is unset, continue to the next iteration of the loop.
+                    1. If |client| is not a [=secure context=], [=continue=].
+                1. Otherwise:

We should define conditionals more clearly I suppose, raise it? I suspect we'd go with "Otherwise" given the overwhelming precedent, but who knows.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/w3c/ServiceWorker/pull/1243#discussion_r155844156

Received on Friday, 8 December 2017 18:36:30 UTC