Address review feedback

Author: lutien

index.bs

@@ -1553,7 +1553,7 @@ explicitly test that scripts running in sandboxes work in all implementations.
 
 ## Sandbox Realms ## {#sandbox-realm}
 
-Each sandbox is a unique ECMAScript [=Realm=]. However the sandbox realm
+Each sandbox is a unique ECMAScript [=/Realm=]. However the sandbox realm
 provides access to platform objects in an existing {{Window}} realm via
 {{SandboxProxy}} objects.
 
@@ -7739,7 +7739,7 @@ a <code>io.Stream</code> and [=stream settings=].
 
 <dfn>Stream settings</dfn> is a [=struct=] with
 an [=struct/item=] named <dfn for="stream settings">stream</dfn>, which is a [=readable stream=],
-an [=struct/item=] named <dfn for="stream settings">navigable</dfn>, which is a <code>browsingContext.BrowsingContext</code> or null.
+an [=struct/item=] named <dfn for="stream settings">realm</dfn>, which is a <code>script.Realm</code> or null.
 
 <div class="algorithm">
 
@@ -7824,16 +7824,15 @@ The [=remote end steps=] given |session| and |command parameters| are:
 
 1. Let |stream| be |stream settings|[[=stream settings/stream=]].
 
-1. Let |navigable id| be |stream settings|[[=stream settings/navigable=]].
+1. Let |realm id| be |stream settings|[[=stream settings/realm=]].
 
 1. If |command parameters| [=map/contains=] "<code>reason</code>", let |reason|
    be |command parameters|["<code>reason</code>"]. Otherwise let |reason| be
    undefined.
 
-1. If |navigable id| is not null:
+1. If |realm id| is not null:
 
-     1. Let |realm| be the result of [=trying=] to [=get a realm from a navigable=]
-        with |navigable id| and null.
+     1. Let |realm| be the result of [=trying=] to [=get a realm=] with |realm id|.
 
      1. Let |environment settings| be the [=environment settings object=] whose
         [=realm execution context=]'s Realm component is |realm|.
@@ -7843,23 +7842,21 @@ The [=remote end steps=] given |session| and |command parameters| are:
      1. Let |promise| be the result of [=ReadableStream/cancel=] |stream| with
         |reason|.
 
+     1. [=Clean up after running script=] with |environment settings|.
+
      1. [=React=] to |promise|:
 
       1. If |promise| was rejected, then:
 
-         1. [=Clean up after running script=] with |environment settings|.
-
          1. Return [=error=] with [=error code=] [=stream closed=].
 
       1. If |promise| was fulfilled, then:
 
-         1. [=Clean up after running script=] with |environment settings|.
-
          1. Return [=success=] with data null.
 
 1. Otherwise, return error with [=error code=] [=unsupported operation=].
 
-   TODO: Add support for streams that don't have an associated navigable.
+   TODO: Add support for streams that don't have an associated realm.
 
 </div>
 
@@ -7899,21 +7896,33 @@ The [=remote end steps=] given |session| and |command parameters| are:
 
 1. Let |stream| be |stream settings|[[=stream settings/stream=]].
 
-1. Let |navigable id| be |stream settings|[[=stream settings/navigable=]].
+1. Let |realm id| be |stream settings|[[=stream settings/realm=]].
 
-1. If |navigable id| is not null:
+1. If |realm id| is not null:
 
-   1. Let |realm| be the result of [=trying=] to [=get a realm from a navigable=]
-      with |navigable id| and null.
+   1. Let |realm| be the result of [=trying=] to [=get a realm=] with |realm id|.
 
    1. Let |environment settings| be the [=environment settings object=] whose
       [=realm execution context=]'s Realm component is |realm|.
 
    1. [=Prepare to run script=] with |environment settings|.
 
-   1. Let |reader| be the result of [=ReadableStream/getting a reader=] for |stream|.
+   1. Let |reader| be the result of [=trying=] to [=ReadableStream/getting a reader=] for |stream|.
 
-   1. Let |read result| be the result of calling {{ReadableStreamDefaultReader/read()}} for |reader|.
+   1. Let |promise| be [=a new promise=].
+
+   1. Let |read request| be a new [=read request=] with the following [=struct/items=]:
+
+     <dl>
+        <dt>[=read request/chunk steps=], given |chunk|
+        <dd>Resolve |promise| with «[ "value" → |chunk|, "done" → false ]».
+        <dt>[=read request/close steps=]
+        <dd>Resolve promise with «[ "value" → undefined, "done" → true ]».
+        <dt>[=read request/error steps=], given |e|
+        <dd>Reject |promise| with |e|.
+     </dl>
+
+   1. Let |read request| be the result of [=ReadableStreamDefaultReader/read a chunk=] from |reader| given |read request|.
 
    1. If |read result|.\[[Type]] is <code>normal</code>.:
 
@@ -7949,7 +7958,7 @@ The [=remote end steps=] given |session| and |command parameters| are:
 
 1. Otherwise, return [=error=] with [=error code=] [=unsupported operation=].
 
-   TODO: Add support for streams that don't have an associated navigable.
+   TODO: Add support for streams that don't have an associated realm.
 
 </div>
 
@@ -11568,9 +11577,9 @@ script.Handle = text;
 </pre>
 
 The <code>script.Handle</code> type represents a handle to an object owned by
-the ECMAScript runtime. The handle is only valid in a specific [=Realm=].
+the ECMAScript runtime. The handle is only valid in a specific [=/Realm=].
 
-Each ECMAScript [=Realm=] has a corresponding <dfn>handle object map</dfn>. This is a
+Each ECMAScript [=/Realm=] has a corresponding <dfn>handle object map</dfn>. This is a
 strong [=/map=] from handle ids to their corresponding objects.
 
 #### The script.InternalId Type #### {#type-script-InternalId}
@@ -11806,7 +11815,7 @@ on realm creation.
 script.Realm = text;
 </pre>
 
-Each [=realm=] has an associated <dfn export>realm id</dfn>, which is a string
+Each [=/realm=] has an associated <dfn export>realm id</dfn>, which is a string
 uniquely identifying that realm. This is implicitly set when the realm is
 created.
 
@@ -11822,10 +11831,10 @@ To <dfn>get a realm</dfn> given |realm id|:
 
 1. If |realm id| is null, return [=success=] with data null.
 
-1. If there is no [=realm=] with [=realm id|id=] |realm id| return
+1. If there is no [=/realm=] with [=realm id|id=] |realm id| return
    [=error=] with [=error code=] [=no such frame=]
 
-1. Let |realm| be the [=realm=] with [=realm id|id=] |realm id|.
+1. Let |realm| be the [=/realm=] with [=realm id|id=] |realm id|.
 
 1. Return [=success=] with data |realm|
 
@@ -12254,7 +12263,7 @@ script.RemoteObjectReference = {
 
 The <code><dfn>script.RemoteReference</dfn></code> type is either a
 <code>script.RemoteObjectReference</code> representing a remote reference to an
-existing ECMAScript object in [=handle object map=] in the given [=Realm=], or
+existing ECMAScript object in [=handle object map=] in the given [=/Realm=], or
 is a <code>script.SharedReference</code> representing a reference to a [=node=].
 
 Issue: handle "stale object reference" case.
@@ -12897,16 +12906,14 @@ an |ownership type|, a |serialization internal map|, a |realm| and a |session|:
     1. Let |stream realm| be |value|'s [=relevant settings object=]'s
        [=realm execution context=]'s Realm component.
 
-    1. Let |navigable| be the result of [=get the navigable=] with |stream realm|.
-
-    1. If |navigable| is null, let |navigable id| be null. Otherwise, let
-       |navigable id| be the [=navigable id=] for |navigable|.
+    1. If |stream realm| is null, let |realm id| be null. Otherwise, let
+       |realm id| be the [=realm id=] for |stream realm|.
 
     1. Let |stream id| be the string representation of a [[!RFC9562|UUID]].
 
     1. Let |stream settings| be a [=stream settings=] struct with the
        [=stream settings/stream=] item set to |value| and the
-       [=stream settings/navigable=] item set to |navigable id|.
+       [=stream settings/realm=] item set to |realm id|.
 
     1. Set |session|'s [=readable stream settings map=][|stream id|] to
        |stream settings|.
@@ -13766,7 +13773,7 @@ The [=remote end steps=] given |session| and |command parameters| are:
 #### The script.getRealms Command ####  {#command-script-getRealms}
 
 The <dfn export for=commands>script.getRealms</dfn> command returns a list of
-all realms, optionally filtered to [=realms=] of a specific type, or to the
+all realms, optionally filtered to [=/realms=] of a specific type, or to the
 realm associated with a [=/navigable=]'s [=active document=].
 
 <dl>
@@ -14097,6 +14104,13 @@ Define the following [=unloading document cleanup steps=] with |document|:
 
 1. Let |realm id| be the [=realm id=] for |realm|.
 
+1. For each |active session| in [=active sessions=]:
+
+   1. For each |stream setting| in [=active sessions=]'s [=stream settings=]:
+
+      1. If |stream setting|[[=stream settings/realm=]] equal |realm id|
+         [=map/remove=] |stream setting|.
+
 1. Let |params| be a [=/map=] matching the
    <code>script.RealmDestroyedParameters</code> production, with the
    <code>realm</code> field set to |realm id|.
@@ -14122,6 +14136,13 @@ worker=] algorithm:
 
 1. Let |realm id| be the [=realm id=] for |realm|.
 
+1. For each |active session| in [=active sessions=]:
+
+   1. For each |stream setting| in [=active sessions=]'s [=stream settings=]:
+
+      1. If |stream setting|[[=stream settings/realm=]] equal |realm id|
+         [=map/remove=] |stream setting|.
+
 1. Let |params| be a [=/map=] matching the <code>script.RealmDestroyedParameters</code>
    production, with the <code>realm</code> field set of |realm id|.