Re: [w3c/ServiceWorker] Update agent and realm allocation (#1508)

@jakearchibald approved this pull request.



> @@ -2853,12 +2853,11 @@ spec: webappsec-referrer-policy; urlPrefix: https://w3c.github.io/webappsec-refe
       1. Let |script| be |serviceWorker|'s [=service worker/script resource=].
       1. Assert: |script| is not null.
       1. Let |startFailed| be false.
-      1. Create a separate parallel execution environment (i.e. a separate thread or process or equivalent construct), and run the following steps in that context:
-          1. Call the JavaScript [=InitializeHostDefinedRealm|InitializeHostDefinedRealm()=] abstract operation with the following customizations:
+      1. Let |agent| be the result of [=obtain a service worker agent|obtaining a service worker agent=], and run the following steps in that context:
+          1. Let |realmExecutionContext| be the result of [=create a new JavaScript realm|creating a new JavaScript realm=] given <var>agent</var> and the following customizations:

```suggestion
          1. Let |realmExecutionContext| be the result of [=create a new JavaScript realm|creating a new JavaScript realm=] given |agent| and the following customizations:
```

>            1. Set |serviceWorker|'s [=service worker/global object=] to |workerGlobalScope|.
-          1. Let |workerEventLoop| be a newly created <a>event loop</a>.
+          1. Let |workerEventLoop| be <var>agent</var>'s <a>event loop</a>.

```suggestion
          1. Let |workerEventLoop| be |agent|'s [=event loop=].
```

Does an agent have an associated event loop?

-- 
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/1508#pullrequestreview-384007710

Received on Monday, 30 March 2020 16:12:47 UTC