Fix setting a window settings object's URL

Fixes #2455 by passing the creation URL to "set up a window
environment's settings object", instead of having it attempt to grab the
creation URL from the not-yet-created Document object.

The later comments in #2455 discuss some followup work we could do to
clarify the Document/Window relationship, but that is left for a future
commit.

This also removes the XXX box mentioned in #2688, since Documents are
nowadays created after Windows in the spec.

Author: domfarolino

source

@@ -77250,13 +77250,8 @@ popup4.close();</code></pre></div>
   steps:</p>
 
   <ol>
-   <li>
-    <p>Let <var>window</var> be <var>document</var>'s <span>relevant global object</span>.</p>
-
-    <p class="XXX">Per this standard <var>document</var> can be created before <var>window</var>,
-    which does not make much sense. See <a href="https://github.com/whatwg/html/issues/2688">issue
-    #2688</a>.</p>
-   </li>
+   <li><p>Let <var>window</var> be <var>document</var>'s <span>relevant global
+   object</span>.</p></li>
 
    <li><p>Set <var>browsingContext</var>'s <span>active window</span> to <var>window</var>.</p></li>
 
@@ -77371,8 +77366,9 @@ popup4.close();</code></pre></div>
    otherwise <var>embedder</var>'s <span>relevant settings object</span>'s <span>top-level
    origin</span>.</p></li>
 
-   <li><p><span>Set up a window environment settings object</span> with <var>realm execution
-   context</var>, null, <var>topLevelCreationURL</var>, and <var>topLevelOrigin</var>.</p></li>
+   <li><p><span>Set up a window environment settings object</span> with <code>about:blank</code>,
+   <var>realm execution context</var>, null, <var>topLevelCreationURL</var>, and
+   <var>topLevelOrigin</var>.</p></li>
 
    <li><p>Let <var>coop</var> be a new <span>cross-origin opener policy</span>.</p></li>
 
@@ -77392,6 +77388,11 @@ popup4.close();</code></pre></div>
    opener policy</span> is <var>coop</var>, and which is <span>ready for post-load
    tasks</span>.</p></li>
 
+   <li><p>Assert: <var>document</var>'s <span data-x="concept-document-url">URL</span> and
+   <var>document</var>'s <span>relevant settings object</span>'s <span
+   data-x="concept-environment-creation-url">creation URL</span> are
+   <code>about:blank</code>.</p></li>
+
    <li><p>Ensure that <var>document</var> has a single child <code>html</code> node, which itself
    has two empty child nodes: a <code>head</code> element, and a <code>body</code> element.</p></li>
 
@@ -79531,10 +79532,11 @@ interface <dfn>BarProp</dfn> {
 
   <h4>Script settings for <code>Window</code> objects</h4>
 
-  <p>To <dfn>set up a window environment settings object</dfn>, given a <span>JavaScript execution
-  context</span> <var>execution context</var>, null or an <span>environment</span>
-  <var>reservedEnvironment</var>, a <span>URL</span> <var>topLevelCreationURL</var>, and an
-  <span>origin</span> <var>topLevelOrigin</var>, run these steps:</p>
+  <p>To <dfn>set up a window environment settings object</dfn>, given a <span>URL</span>
+  <var>creationURL</var>, a <span>JavaScript execution context</span> <var>execution context</var>,
+  null or an <span>environment</span> <var>reservedEnvironment</var>, a <span>URL</span>
+  <var>topLevelCreationURL</var>, and an <span>origin</span> <var>topLevelOrigin</var>, run these
+  steps:</p>
 
   <ol>
    <li><p>Let <var>realm</var> be the value of <var>execution context</var>'s Realm
@@ -79543,10 +79545,6 @@ interface <dfn>BarProp</dfn> {
    <li><p>Let <var>window</var> be <var>realm</var>'s <span data-x="concept-realm-global">global
    object</span>.</p></li>
 
-   <li><p>Let <var>url</var> be a copy of the <span data-x="concept-document-url">URL</span> of
-   <var>window</var>'s <span data-x="concept-document-window">associated
-   <code>Document</code></span>.</p></li>
-
    <li>
     <p>Let <var>settings object</var> be a new <span>environment settings object</span> whose
     algorithms are defined as follows:</p>
@@ -79653,9 +79651,9 @@ interface <dfn>BarProp</dfn> {
    data-x="concept-environment-active-service-worker">active service worker</span> to null.</p></li>
 
    <li><p>Set <var>settings object</var>'s <span data-x="concept-environment-creation-url">creation
-   URL</span> to <var>url</var>, <var>settings object</var>'s <span>top-level creation URL</span> to
-   <var>topLevelCreationURL</var>, and <var>settings object</var>'s <span>top-level origin</span> to
-   <var>topLevelOrigin</var>.</p></li>
+   URL</span> to <var>creationURL</var>, <var>settings object</var>'s <span>top-level creation
+   URL</span> to <var>topLevelCreationURL</var>, and <var>settings object</var>'s <span>top-level
+   origin</span> to <var>topLevelOrigin</var>.</p></li>
 
    <li><p>Set <var>realm</var>'s [[HostDefined]] field to <var>settings object</var>.</p></li>
   </ol>
@@ -85004,8 +85002,8 @@ interface <dfn>Location</dfn> { // but see also <a href="#the-location-interface
       </ol>
      </li>
 
-     <li><p><span>Set up a window environment settings object</span> with <var>realm execution
-     context</var>, <var>navigationParams</var>'s <span
+     <li><p><span>Set up a window environment settings object</span> with <var>creationURL</var>,
+     <var>realm execution context</var>, <var>navigationParams</var>'s <span
      data-x="navigation-params-reserved-environment">reserved environment</span>,
      <var>topLevelCreationURL</var>, and <var>topLevelOrigin</var>.</p></li>
     </ol>