Fix url and src in NotRestoredReasons

url was previously listed as a DOMString, but it should be a USVString. This change is not observable since it always contained a serialized URL anyway.

src was previously listed as containing the literal value of the src="" content attribute. However, the intended behavior was to contain the parsed-then-serialized value of the src IDL attribute, i.e., the result after content/IDL attribute reflection. Fix the spec to reflect that, which allows also changing it to USVString.

Finally, also account for the fact that sometimes content attributes can be missing, in which case these properties need to return the empty string. (Not null, which is reserved for the top level or non-iframe child cases.)

Closes whatwg#10895.

Author: domenic

source

@@ -96936,10 +96936,10 @@ interface <dfn interface>NotRestoredReasonDetails</dfn> {
 
 [Exposed=Window]
 interface <dfn interface>NotRestoredReasons</dfn> {
-  readonly attribute DOMString? <span data-x="dom-not-restored-reasons-src">src</span>;
+  readonly attribute USVString? <span data-x="dom-not-restored-reasons-src">src</span>;
   readonly attribute DOMString? <span data-x="dom-not-restored-reasons-id">id</span>;
   readonly attribute DOMString? <span data-x="dom-not-restored-reasons-name">name</span>;
-  readonly attribute DOMString? <span data-x="dom-not-restored-reasons-url">url</span>;
+  readonly attribute USVString? <span data-x="dom-not-restored-reasons-url">url</span>;
   readonly attribute FrozenArray&lt;<span>NotRestoredReasonDetails</span>&gt;? <span data-x="dom-not-restored-reasons-reasons">reasons</span>;
   readonly attribute FrozenArray&lt;<span>NotRestoredReasons</span>&gt;? <span data-x="dom-not-restored-reasons-children">children</span>;
   [Default] object toJSON();
@@ -97383,7 +97383,8 @@ interface <dfn interface>NotRestoredReasons</dfn> {
   the following <span data-x="struct item">items</span>:</p>
 
   <ul>
-   <li><p><dfn data-x="nrr-src">src</dfn>, a string or null, initially null.</p></li>
+   <li><p><dfn data-x="nrr-src">src</dfn>, a <span>scalar value string</span> or null, initially
+   null.</p></li>
 
    <li><p><dfn data-x="nrr-id">id</dfn>, a string or null, initially null.</p></li>
 
@@ -104738,22 +104739,38 @@ location.href = '#foo';</code></pre>
    <li><p>Set <var>notRestoredReasonsForDocument</var>'s <span data-x="nrr-url">URL</span> to
    <var>document</var>'s <span data-x="concept-document-url">URL</span>.</p></li>
 
+   <li><p>Let <var>container</var> be <var>document</var>'s <span>node navigable</span>'s <span
+   data-x="nav-container">container</span>.</p></li>
+
    <li>
-    <p>If <var>document</var>'s <span>node navigable</span>'s <span
-    data-x="nav-container">container</span> is an <code>iframe</code> element, then:</p>
+    <p>If <var>container</var> is an <code>iframe</code> element:</p>
 
     <ol>
+     <li><p>Let <var>src</var> be the empty string.</p></li>
+
+     <li>
+      <p>If <var>container</var> has a <code data-x="dom-iframe-src">src</code> attribute:</p>
+
+      <ol>
+       <li><p>Let <var>src</var> be the result of <span>encoding-parsing-and-serializing a
+       URL</span> given <var>container</var>'s <code data-x="attr-iframe-src">src</code> attribute's
+       value, relative to <var>container</var>'s <span>node document</span>.</p></li>
+
+       <li><p>If <var>src</var> is failure, then set <var>src</var> to <var>container</var>'s <code
+       data-x="attr-iframe-src">src</code> attribute's value.</p></li>
+      </ol>
+     </li>
+
      <li><p>Set <var>notRestoredReasonsForDocument</var>'s <span data-x="nrr-src">src</span> to
-     the value of <var>document</var>'s <span>node navigable</span>'s <span
-     data-x="nav-container">container</span>'s <code data-x="attr-iframe-src">src</code> attribute.</p></li>
+     <var>src</var>.</p></li>
 
      <li><p>Set <var>notRestoredReasonsForDocument</var>'s <span data-x="nrr-id">id</span> to
-     the value of <var>document</var>'s <span>node navigable</span>'s <span
-     data-x="nav-container">container</span>'s <code data-x="attr-id">id</code> attribute.</p></li>
+     <var>container</var>'s <code data-x="attr-id">id</code> attribute's value, or the empty string
+     if it has no such attribute.</p></li>
 
-     <li><p>Set <var>notRestoredReasonsForDocument</var>'s <span data-x="nrr-name">name</span>
-     to the value of <var>document</var>'s <span>node navigable</span>'s <span
-     data-x="nav-container">container</span>'s <code data-x="attr-iframe-name">name</code> attribute.</p></li>
+     <li><p>Set <var>notRestoredReasonsForDocument</var>'s <span data-x="nrr-name">name</span> to
+     <var>container</var>'s <code data-x="attr-iframe-name">name</code> attribute's value, or the
+     empty string if it has no such attribute.</p></li>
     </ol>
    </li>