Editorial: clarify specification around scripts

In particular:

* Give script's settings object an ID and data-x.
* Note that a script's fetch options can be null.
* Correct miscount in definition of 'script'. This was introduced in 3d45584 (#4898).
* Fix typos in HostLoadImportedModule.

Fixes #9867 and closes #9888.

source

@@ -103908,11 +103908,13 @@ document.querySelector("button").addEventListener("click", bound);
 
   <h5 id="script-structs"><span id="definitions-2"></span>Scripts</h5>
 
-  <p>A <dfn data-x="concept-script" export>script</dfn> is one of three possible <span
-  data-x="struct">structs</span>. All scripts have:</p>
+  <p>A <dfn data-x="concept-script" export>script</dfn> is one of two possible <span
+  data-x="struct">structs</span> (namely, a <span>classic script</span> or a
+  <span>module script</span>). All scripts have:</p>
 
   <dl>
-   <dt>A <dfn for="script" export>settings object</dfn></dt>
+   <dt>A <dfn for="script" export data-x="concept-script-settings-object"
+   id="settings-object">settings object</dfn></dt>
 
    <dd><p>An <span>environment settings object</span>, containing various settings that are shared
    with other <span data-x="concept-script">scripts</span> in the same context.</p></dd>
@@ -103968,8 +103970,8 @@ document.querySelector("button").addEventListener("click", bound);
    <dt><dfn for="script" export
    data-x="concept-script-script-fetch-options">Fetch options</dfn></dt>
 
-   <dd>A <span>script fetch options</span>, containing various options related to fetching this
-   script or <span data-x="module script">module scripts</span> that it imports.</dd>
+   <dd>Null or a <span>script fetch options</span>, containing various options related to fetching
+   this script or <span data-x="module script">module scripts</span> that it imports.</dd>
 
    <dt>A <dfn for="script" export data-x="concept-script-base-url">base
    URL</dfn></dt>
@@ -104984,7 +104986,8 @@ document.querySelector("button").addEventListener("click", bound);
    <li><p>Let <var>script</var> be a new <span>classic script</span> that this algorithm will
    subsequently initialize.</p></li>
 
-   <li><p>Set <var>script</var>'s <span>settings object</span> to <var>settings</var>.</p></li>
+   <li><p>Set <var>script</var>'s <span data-x="concept-script-settings-object">settings
+   object</span> to <var>settings</var>.</p></li>
 
    <li><p>Set <var>script</var>'s <span data-x="concept-script-base-url">base URL</span> to
    <var>baseURL</var>.</p></li>
@@ -105036,7 +105039,8 @@ document.querySelector("button").addEventListener("click", bound);
    <li><p>Let <var>script</var> be a new <span>module script</span> that this algorithm will
    subsequently initialize.</p></li>
 
-   <li><p>Set <var>script</var>'s <span>settings object</span> to <var>settings</var>.</p></li>
+   <li><p>Set <var>script</var>'s <span data-x="concept-script-settings-object">settings
+   object</span> to <var>settings</var>.</p></li>
 
    <li><p>Set <var>script</var>'s <span data-x="concept-script-base-url">base URL</span> to
    <var>baseURL</var>.</p></li>
@@ -105144,7 +105148,8 @@ document.querySelector("button").addEventListener("click", bound);
    <li><p>Let <var>script</var> be a new <span>module script</span> that this algorithm will
    subsequently initialize.</p></li>
 
-   <li><p>Set <var>script</var>'s <span>settings object</span> to <var>settings</var>.</p></li>
+   <li><p>Set <var>script</var>'s <span data-x="concept-script-settings-object">settings
+   object</span> to <var>settings</var>.</p></li>
 
    <li><p>Set <var>script</var>'s <span data-x="concept-script-base-url">base URL</span> and
    <span data-x="concept-script-script-fetch-options">fetch options</span> to null.</p></li>
@@ -105183,7 +105188,8 @@ document.querySelector("button").addEventListener("click", bound);
    <li><p>Let <var>script</var> be a new <span>module script</span> that this algorithm will
    subsequently initialize.</p></li>
 
-   <li><p>Set <var>script</var>'s <span>settings object</span> to <var>settings</var>.</p></li>
+   <li><p>Set <var>script</var>'s <span data-x="concept-script-settings-object">settings
+   object</span> to <var>settings</var>.</p></li>
 
    <li><p>Set <var>script</var>'s <span data-x="concept-script-base-url">base URL</span> and
    <span data-x="concept-script-script-fetch-options">fetch options</span> to null.</p></li>
@@ -105269,7 +105275,8 @@ document.querySelector("button").addEventListener("click", bound);
   and an optional boolean <var>rethrow errors</var> (default false):</p>
 
   <ol>
-   <li><p>Let <var>settings</var> be the <span>settings object</span> of <var>script</var>.</p></li>
+   <li><p>Let <var>settings</var> be the <span data-x="concept-script-settings-object">settings
+   object</span> of <var>script</var>.</p></li>
 
    <li><p><span>Check if we can run script</span> with <var>settings</var>. If this returns "do
    not run" then return <span>NormalCompletion</span>(empty).</p></li>
@@ -105350,7 +105357,8 @@ document.querySelector("button").addEventListener("click", bound);
   and an optional boolean <var>preventErrorReporting</var> (default false):</p>
 
   <ol>
-   <li><p>Let <var>settings</var> be the <span>settings object</span> of <var>script</var>.</p></li>
+   <li><p>Let <var>settings</var> be the <span data-x="concept-script-settings-object">settings
+   object</span> of <var>script</var>.</p></li>
 
    <li><p><span>Check if we can run script</span> with <var>settings</var>. If this returns "do not
    run", then return <span>a promise resolved with</span> with undefined.</p></li>
@@ -105583,9 +105591,9 @@ document.querySelector("button").addEventListener("click", bound);
   data-x="concept-script">script</span>, with the problematic position (line number and column
   number) in the resource containing the script, using the <span
   data-x="concept-settings-object-global">global object</span> specified by the script's
-  <span>settings object</span> as the target. If the error is still <i
-  data-x="concept-error-nothandled">not handled</i> after this, then the error may be reported to a
-  developer console.</p>
+  <span data-x="concept-script-settings-object">settings object</span> as the target. If the error
+  is still <i data-x="concept-error-nothandled">not handled</i> after this, then the error may be
+  reported to a developer console.</p>
 
   <p class="XXX">The existence of both <span data-x="report the error">report an error</span> and
   <span data-x="report the exception">report an exception</span> is confusing, and both algorithms
@@ -105830,8 +105838,8 @@ dictionary <dfn dictionary>PromiseRejectionEventInit</dfn> : <span>EventInit</sp
     <p>If <var>referringScript</var> is not null, then:</p>
 
     <ol>
-     <li><p>Set <var>settingsObject</var> to <var>referringScript</var>'s <span>settings
-     object</span>.</p></li>
+     <li><p>Set <var>settingsObject</var> to <var>referringScript</var>'s
+     <span data-x="concept-script-settings-object">settings object</span>.</p></li>
 
      <li><p>Set <var>baseURL</var> to <var>referringScript</var>'s <span
      data-x="concept-script-base-url">base URL</span>.</p></li>
@@ -106603,7 +106611,8 @@ dictionary <dfn dictionary>PromiseRejectionEventInit</dfn> : <span>EventInit</sp
    <li><p>Let <var>settings object</var> be the <span>current settings object</span>.</p></li>
 
    <li><p>If <var>script</var> is not null, then set <var>settings object</var> to
-   <var>script</var>'s <span>settings object</span>.</p></li>
+   <var>script</var>'s <span data-x="concept-script-settings-object">settings
+   object</span>.</p></li>
 
    <li>
     <p>If <var>operation</var> is "<code data-x="">reject</code>", then:</p>
@@ -106879,8 +106888,9 @@ dictionary <dfn dictionary>PromiseRejectionEventInit</dfn> : <span>EventInit</sp
    <li>
     <p>If <var>active script</var> is not null, set <var>script execution context</var> to a new
     <span>JavaScript execution context</span>, with its Function field set to null, its Realm field
-    set to <var>active script</var>'s <span>settings object</span>'s <span data-x="environment
-    settings object's realm">realm</span>, and its ScriptOrModule set to <var>active script</var>'s
+    set to <var>active script</var>'s <span data-x="concept-script-settings-object">settings
+    object</span>'s <span data-x="environment settings object's realm">realm</span>, and its
+    ScriptOrModule set to <var>active script</var>'s
     <span data-x="concept-script-record">record</span>.</p>
 
     <p class="note">As seen below, this is used in order to propagate the current <span>active
@@ -107167,8 +107177,8 @@ import "https://example.com/foo/../module2.mjs";</code></pre>
     <ol>
      <li><p>Set <var>referencingScript</var> to <var>referrer</var>.[[HostDefined]].</p>
 
-     <li><p>Set <var>settingsObject</var> to <var>referencingScript</var>'s <span>settings
-     object</span>.</p></li>
+     <li><p>Set <var>settingsObject</var> to <var>referencingScript</var>'s <span
+     data-x="concept-script-settings-object">settings object</span>.</p></li>
 
      <li><p>Set <var>fetchOptions</var> to the new <span>descendant script fetch options</span> for
      <var>referencingScript</var>'s <span data-x="concept-script-script-fetch-options">fetch
@@ -107228,14 +107238,14 @@ import "https://example.com/foo/../module2.mjs";</code></pre>
     <ol>
      <li><p>Set <var>destination</var> to <var>loadState</var>.[[Destination]].</p></li>
 
-     <li><p>Set <var>fetchClient</var> <var>loadState</var>.[[FetchClient]].</p></li>
+     <li><p>Set <var>fetchClient</var> to <var>loadState</var>.[[FetchClient]].</p></li>
     </ol>
    </li>
 
    <li>
     <p><span>Fetch a single imported module script</span> given <var>url</var>,
     <var>fetchClient</var>, <var>destination</var>, <var>fetchOptions</var>,
-    <var>settingsObject</var> <var>fetchReferrer</var>, <var>moduleRequest</var>, and
+    <var>settingsObject</var>, <var>fetchReferrer</var>, <var>moduleRequest</var>, and
     <var>onSingleFetchComplete</var> as defined below. If <var>loadState</var> is not undefined and
     <var>loadState</var>.[[PerformFetch]] is not null, pass <var>loadState</var>.[[PerformFetch]]
     along as well.</p>
@@ -107267,8 +107277,8 @@ import "https://example.com/foo/../module2.mjs";</code></pre>
      </li>
 
      <li><p>Otherwise, set <var>completion</var> to <span>Completion Record</span> { [[Type]]:
-     normal, [[Value]]: <var>result</var>'s <span data-x="concept-script-record">record</span>,
-     [[Target]]: empty }.</p></li>
+     normal, [[Value]]: <var>moduleScript</var>'s
+     <span data-x="concept-script-record">record</span>, [[Target]]: empty }.</p></li>
 
      <li><p>Perform <span>FinishLoadingImportedModule</span>(<var>referrer</var>,
      <var>moduleRequest</var>, <var>payload</var>, <var>completion</var>).</p></li>
@@ -107544,7 +107554,7 @@ import "https://example.com/foo/../module2.mjs";</code></pre>
    the browsing context's <span>active document</span>.</p></li>
 
    <li><p>If the task is being queued by or for a <span data-x="concept-script">script</span>, then
-   return the script's <span>settings object</span>'s <span
+   return the script's <span data-x="concept-script-settings-object">settings object</span>'s <span
    data-x="concept-settings-object-global">global object</span>'s <span
    data-x="concept-document-window">associated <code>Document</code></span>.</p></li>
 
@@ -116630,9 +116640,9 @@ interface <dfn interface>SharedWorker</dfn> : <span>EventTarget</span> {
      <li>
       <p>If <var>worker global scope</var> is not null, but the user agent has been
       configured to disallow communication between the worker represented by the <var>worker global
-      scope</var> and the <span data-x="concept-script">scripts</span> whose <span data-x="settings
-      object">settings object</span> is <var>outside settings</var>, then set <var>worker global
-      scope</var> to null.</p>
+      scope</var> and the <span data-x="concept-script">scripts</span> whose <span
+      data-x="concept-script-settings-object">settings object</span> is <var>outside settings</var>,
+      then set <var>worker global scope</var> to null.</p>
 
       <p class="note">For example, a user agent could have a development mode that isolates a
       particular <span>top-level traversable</span> from all other pages, and scripts in that
@@ -132924,10 +132934,11 @@ if (s = prompt('What is your name?')) {
    <li>None of the elements in the <code>Document</code> are in any of the following namespaces: <span>HTML namespace</span>, <span>SVG namespace</span>, <span>MathML namespace</span>
    <li>The <code>Document</code> has no <span>focusable area</span> (e.g. from XLink) other than the <span>viewport</span>.
    <li>The <code>Document</code> has no <span data-x="hyperlink">hyperlinks</span> (e.g. from XLink).
-   <li>There exists no <span data-x="concept-script">script</span> whose <span>settings
-   object</span>'s <span data-x="concept-settings-object-global">global object</span> is a
-   <code>Window</code> object with this <code>Document</code> as its <span
-   data-x="concept-document-window">associated <code>Document</code></span>.
+   <li>There exists no <span data-x="concept-script">script</span> whose <span
+   data-x="concept-script-settings-object">settings object</span>'s <span
+   data-x="concept-settings-object-global">global object</span> is a <code>Window</code> object with
+   this <code>Document</code> as its <span data-x="concept-document-window">associated
+   <code>Document</code></span>.
    <li>None of the elements in the <code>Document</code> have any registered event listeners.
   </ul>