Address some comments

Author: lukewarlow

source

@@ -8798,77 +8798,80 @@ a.setAttribute('href', 'https://example.com/'); // change the content attribute
 
   <h4>Using reflect via IDL extended attributes</h4>
 
-  <p>To enable improved usage of <span data-x="reflect">reflection</span>, we introduce the <dfn
-  extended-attribute data-lt="Reflect" data-x="xattr-Reflect"><code>[Reflect]</code></dfn>, <dfn
-  extended-attribute data-lt="ReflectSetter"
-  data-x="xattr-ReflectSetter"><code>[ReflectSetter]</code></dfn>, <dfn extended-attribute
-  data-lt="ReflectURL" data-x="xattr-ReflectURL"><code>[ReflectURL]</code></dfn>, <dfn
-  extended-attribute data-lt="ReflectNonNegative"
+  <p>To enable improved usage of <span data-x="reflect">reflection</span>, we introduce multiple new
+  IDL <span data-x="extended attribute">extended attributes</span>. <dfn extended-attribute
+  data-lt="Reflect" data-x="xattr-Reflect"><code>[Reflect]</code></dfn>, <dfn extended-attribute
+  data-lt="ReflectSetter" data-x="xattr-ReflectSetter"><code>[ReflectSetter]</code></dfn>, <dfn
+  extended-attribute data-lt="ReflectURL" data-x="xattr-ReflectURL"><code>[ReflectURL]</code></dfn>,
+  <dfn extended-attribute data-lt="ReflectNonNegative"
   data-x="xattr-ReflectNonNegative"><code>[ReflectNonNegative]</code></dfn>, <dfn extended-attribute
-  data-lt="ReflectPositive" data-x="xattr-ReflectPositive"><code>[ReflectPositive]</code></dfn>,
+  data-lt="ReflectPositive" data-x="xattr-ReflectPositive"><code>[ReflectPositive]</code></dfn>, and
   <dfn extended-attribute data-lt="ReflectPositiveWithFallback"
-  data-x="xattr-ReflectPositiveWithFallback"><code>[ReflectPositiveWithFallback]</code></dfn>, <dfn
-  extended-attribute data-lt="ReflectRange"
-  data-x="xattr-ReflectRange"><code>[ReflectRange]</code></dfn>, and <dfn extended-attribute
-  data-lt="ReflectDefault" data-x="xattr-ReflectDefault"><code>[ReflectDefault]</code></dfn> IDL
-  <span data-x="extended attribute">extended attributes</span>.</p>
+  data-x="xattr-ReflectPositiveWithFallback"><code>[ReflectPositiveWithFallback]</code></dfn> all
+  trigger <span data-x="reflect">reflection</span>. These must either take no arguments, or take a
+  string; they must not appear on anything other than an interface member attribute; and only one of
+  these can be used at a time.</p>
 
-  <p>The <code data-x="xattr-Reflect">[Reflect]</code>, <code
-  data-x="xattr-ReflectSetter">[ReflectSetter]</code>, <code
-  data-x="xattr-ReflectURL">[ReflectURL]</code>, <code
-  data-x="xattr-ReflectNonNegative">[ReflectNonNegative]</code>, <code
-  data-x="xattr-ReflectPositive">[ReflectPositive]</code>, <code
-  data-x="xattr-ReflectPositiveWithFallback">[ReflectPositiveWithFallback]</code> <span>extended
-  attribute</span>s must either take no arguments, or take a string. They must not appear on
-  anything other than an interface member attribute.</p>
+  <p>For one of these primary reflection <span data-x="extended attribute">extended
+  attributes</span>, its <span>reflected content attribute name</span> is the string value it takes,
+  if one is provided, or the IDL attribute name otherwise.</p>
 
   <p>IDL attributes with the <code data-x="xattr-Reflect">[Reflect]</code> <span>extended
-  attribute</span> must <span>reflect</span> the content attribute of the name provided by <code
-  data-x="xattr-Reflect">[Reflect]</code> or the IDL attribute name if none is provided.</p>
+  attribute</span> must <span>reflect</span> <code data-x="xattr-Reflect">[Reflect]</code>'s
+  <span>reflected content attribute name</span>.</p>
 
   <p>IDL attributes with the <code data-x="xattr-ReflectSetter">[ReflectSetter]</code>
-  <span>extended attribute</span> on setting must <span>reflect</span> the content attribute of the
-  name provided by <code data-x="xattr-ReflectSetter">[ReflectSetter]</code> or the IDL attribute
-  name if none is provided.</p>
+  <span>extended attribute</span> on setting must <span>reflect</span> <code
+  data-x="xattr-ReflectSetter">[ReflectSetter]</code>'s <span>reflected content attribute
+  name</span>.</p>
+
+  <p>The <code data-x="xattr-ReflectURL">[ReflectURL]</code> <span>extended attribute</span> must
+  only appear on attributes with a type of <code data-x="idl-USVString">USVString</code>.</p>
 
   <p>IDL attributes with the <code data-x="xattr-ReflectURL">[ReflectURL]</code> <span>extended
-  attribute</span> must <span>reflect</span>, <span data-x="treated as a URL">as a URL</span>, the
-  content attribute of the name provided by <code data-x="xattr-ReflectURL">[ReflectURL]</code> or
-  the IDL attribute name if none is provided.</p>
+  attribute</span> must <span>reflect</span>, <span data-x="treated as a URL">as a URL</span>, <code
+  data-x="xattr-ReflectURL">[ReflectURL]</code>'s <span>reflected content attribute name</span>.</p>
 
   <p>The <code data-x="xattr-ReflectNonNegative">[ReflectNonNegative]</code> <span>extended
-  attribute</span> must only appear on interface member attributes with a type of <code
+  attribute</span> must only appear on attributes with a type of <code
   data-x="idl-long">long</code>.</p>
 
   <p>IDL attributes with the <code data-x="xattr-ReflectNonNegative">[ReflectNonNegative]</code>
   <span>extended attribute</span> must <span>reflect</span>, <span>limited to only non-negative
-  numbers</span>, the content attribute of the name provided by <code
-  data-x="xattr-ReflectNonNegative">[ReflectNonNegative]</code> or the IDL attribute name if none is
-  provided.</p>
+  numbers</span>, <code data-x="xattr-ReflectNonNegative">[ReflectNonNegative]</code>'s
+  <span>reflected content attribute name</span>.</p>
 
   <p>The <code data-x="xattr-ReflectPositive">[ReflectPositive]</code> and <code
-  data-x="xattr-ReflectPositiveWithFallback">[ReflectPositiveWithFallback]</code> <span>extended
-  attribute</span>s must only appear on interface member attributes with a type of <code
-  data-x="idl-double">double</code> or <code data-x="idl-unsigned-long">unsigned long</code>.</p>
+  data-x="xattr-ReflectPositiveWithFallback">[ReflectPositiveWithFallback]</code> <span
+  data-x="extended attribute">extended attributes</span> must only appear on attributes with a type
+  of <code data-x="idl-double">double</code> or <code data-x="idl-unsigned-long">unsigned
+  long</code>.</p>
 
   <p>IDL attributes with the <code data-x="xattr-ReflectPositive">[ReflectPositive]</code>
   <span>extended attribute</span> must <span>reflect</span>, <span>limited to only positive
-  numbers</span>, the content attribute of the name provided by <code
-  data-x="xattr-ReflectPositive">[ReflectPositive]</code> or the IDL attribute name if none is
-  provided.</p>
+  numbers</span>, <code data-x="xattr-ReflectPositive">[ReflectPositive]</code>'s <span>reflected
+  content attribute name</span>.</p>
 
   <p>IDL attributes with the <code
   data-x="xattr-ReflectPositiveWithFallback">[ReflectPositiveWithFallback]</code> <span>extended
   attribute</span> must <span>reflect</span>, <span>limited to only positive numbers with
-  fallback</span>, the content attribute of the name provided by <code
-  data-x="xattr-ReflectPositiveWithFallback">[ReflectPositiveWithFallback]</code> or the IDL
-  attribute name if none is provided.</p>
+  fallback</span>, <code
+  data-x="xattr-ReflectPositiveWithFallback">[ReflectPositiveWithFallback]</code>'s <span>reflected
+  content attribute name</span>.</p>
+
+  <hr>
+
+  <p>To supplement the above <span data-x="extended attribute">extended attributes</span> we also
+  introduce <dfn extended-attribute data-lt="ReflectRange"
+  data-x="xattr-ReflectRange"><code>[ReflectRange]</code></dfn>, and <dfn extended-attribute
+  data-lt="ReflectDefault" data-x="xattr-ReflectDefault"><code>[ReflectDefault]</code></dfn>. These
+  augment how <span data-x="reflect">reflection</span> works and also must only appear on interface
+  member attributes.</p>
 
   <p>The <code data-x="xattr-ReflectRange">[ReflectRange]</code> <span>extended attribute</span>
-  must take an integer list limited to two values. It must not appear on anything other than an
-  interface member attribute with a type of <code data-x="idl-unsigned-long">unsigned long</code>.
-  Additionally, it must also only appear alongside <code
-  data-x="xattr-Reflect">[Reflect]</code>.</p>
+  must take an integer list limited to two values. It must only be used on attributes with a type of
+  <code data-x="idl-unsigned-long">unsigned long</code>. Additionally, it must also only appear
+  alongside <code data-x="xattr-Reflect">[Reflect]</code>.</p>
 
   <p>IDL attributes with the <code data-x="xattr-ReflectRange">[ReflectRange]</code> <span>extended
   attribute</span> are <span>clamped to the range</span> [<var>clampedMin</var>,
@@ -8877,18 +8880,18 @@ a.setAttribute('href', 'https://example.com/'); // change the content attribute
   data-x="xattr-ReflectRange">[ReflectRange]</code>.</p>
 
   <p>The <code data-x="xattr-ReflectDefault">[ReflectDefault]</code> <span>extended attribute</span>
-  must not appear on anything other than an interface member attribute with a type of <code
-  data-x="idl-double">double</code>, <code data-x="idl-long">long</code>, or <code
-  data-x="idl-unsigned-long">unsigned long</code>. When used on an attribute of type <code
-  data-x="idl-double">double</code>, it must take a decimal, otherwise it must take an integer.
-  Additionally, it must also only appear alongside <code data-x="xattr-Reflect">[Reflect]</code>,
-  <code data-x="xattr-ReflectNonNegative">[ReflectNonNegative]</code>, <code
+  must only be used on attributes with a type of <code data-x="idl-double">double</code>, <code
+  data-x="idl-long">long</code>, or <code data-x="idl-unsigned-long">unsigned long</code>. When used
+  on an attribute of type <code data-x="idl-double">double</code>, it must take a decimal; otherwise
+  it must take an integer. Additionally, it must also only appear alongside <code
+  data-x="xattr-Reflect">[Reflect]</code>, <code
+  data-x="xattr-ReflectNonNegative">[ReflectNonNegative]</code>, <code
   data-x="xattr-ReflectPositive">[ReflectPositive]</code>, or <code
   data-x="xattr-ReflectPositiveWithFallback">[ReflectPositiveWithFallback]</code>.</p>
 
   <p>IDL attributes with the <code data-x="xattr-ReflectDefault">[ReflectDefault]</code>
-  <span>extended attribute</span> have a <span>default value</span> provided by <code
-  data-x="xattr-ReflectDefault">[ReflectDefault]</code>.</p>
+  <span>extended attribute</span> have a <span>default value</span> provided by the argument
+  provided to <code data-x="xattr-ReflectDefault">[ReflectDefault]</code>.</p>
 
   <h4>Using reflect in specifications</h4>
 
@@ -20308,8 +20311,9 @@ interface <dfn interface>HTMLQuoteElement</dfn> : <span>HTMLElement</span> {
   [<span>HTMLConstructor</span>] constructor();
 
   [<span>CEReactions</span>, <span data-x="xattr-ReflectURL">ReflectURL</span>] attribute USVString <dfn attribute for="HTMLQuoteElement" data-x="dom-quote-cite">cite</dfn>;
-};</code></pre> <p class="note">The <code>HTMLQuoteElement</code> interface is also used by the
-<code>q</code> element.</p>
+};</code></pre>
+    <p class="note">The <code>HTMLQuoteElement</code> interface is also used by the <code>q</code>
+    element.</p>
    </dd>
    <dd w-dev>Uses <code>HTMLQuoteElement</code>.</dd>
   </dl>
@@ -20620,7 +20624,7 @@ interface <dfn interface>HTMLOListElement</dfn> : <span>HTMLElement</span> {
 
   <div w-nodev>
 
-  <p class="note">The <span data-x="xattr-ReflectDefault">[ReflectDefault]</span> means that the
+  <p class="note">Due to <span data-x="xattr-ReflectDefault">[ReflectDefault]</span> the
   <code data-x="dom-ol-start">start</code> IDL attribute does not necessarily match the list's <span
   data-x="concept-ol-start">starting value</span>, in cases where the <code
   data-x="attr-ol-start">start</code> content attribute is omitted and the <code
@@ -29056,8 +29060,8 @@ interface <dfn interface>HTMLSourceElement</dfn> : <span>HTMLElement</span> {
   [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute USVString <dfn attribute for="HTMLSourceElement" data-x="dom-source-srcset">srcset</dfn>;
   [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute DOMString <dfn attribute for="HTMLSourceElement" data-x="dom-source-sizes">sizes</dfn>;
   [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute DOMString <dfn attribute for="HTMLSourceElement" data-x="dom-source-media">media</dfn>;
-  [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute unsigned long <dfn attribute for="HTMLIFrameElement,HTMLEmbedElement,HTMLObjectElement,HTMLSourceElement,HTMLVideoElement" data-x="dom-dim-width">width</dfn>;
-  [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute unsigned long <dfn attribute for="HTMLIFrameElement,HTMLEmbedElement,HTMLObjectElement,HTMLSourceElement,HTMLVideoElement" data-x="dom-dim-height">height</dfn>;
+  [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute unsigned long <dfn attribute for="HTMLSourceElement" data-x="dom-source-width">width</dfn>;
+  [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute unsigned long <dfn attribute for="HTMLSourceElement" data-x="dom-source-height">height</dfn>;
 };</code></pre>
    </dd>
    <dd w-dev>Uses <code>HTMLSourceElement</code>.</dd>
@@ -33383,8 +33387,8 @@ interface <dfn interface>HTMLIFrameElement</dfn> : <span>HTMLElement</span> {
   [SameObject, PutForwards=<span data-x="dom-DOMTokenList-value">value</span>, <span data-x="xattr-Reflect">Reflect</span>] readonly attribute <span>DOMTokenList</span> <dfn attribute for="HTMLIFrameElement" data-x="dom-iframe-sandbox">sandbox</dfn>;
   [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute DOMString <dfn attribute for="HTMLIFrameElement" data-x="dom-iframe-allow">allow</dfn>;
   [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>="<span data-x="attr-iframe-allowfullscreen">allowfullscreen</span>"] attribute boolean <dfn attribute for="HTMLIFrameElement" data-x="dom-iframe-allowFullscreen">allowFullscreen</dfn>;
-  [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute DOMString <span data-x="dom-dim-width">width</span>;
-  [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute DOMString <span data-x="dom-dim-height">height</span>;
+  [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute DOMString <dfn attribute for="HTMLIFrameElement" data-x="dom-dim-width">width</dfn>;
+  [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute DOMString <dfn attribute for="HTMLIFrameElement" data-x="dom-dim-height">height</dfn>;
   [<span>CEReactions</span>] attribute DOMString <span data-x="dom-iframe-referrerPolicy">referrerPolicy</span>;
   [<span>CEReactions</span>] attribute DOMString <span data-x="dom-iframe-loading">loading</span>;
   readonly attribute <span>Document</span>? <span data-x="dom-iframe-contentDocument">contentDocument</span>;
@@ -34269,8 +34273,8 @@ interface <dfn interface>HTMLEmbedElement</dfn> : <span>HTMLElement</span> {
 
   [<span>CEReactions</span>, <span data-x="xattr-ReflectURL">ReflectURL</span>] attribute USVString <dfn attribute for="HTMLEmbedElement" data-x="dom-embed-src">src</dfn>;
   [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute DOMString <dfn attribute for="HTMLEmbedElement" data-x="dom-embed-type">type</dfn>;
-  [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute DOMString <span data-x="dom-dim-width">width</span>;
-  [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute DOMString <span data-x="dom-dim-height">height</span>;
+  [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute DOMString <dfn attribute for="HTMLEmbedElement" data-x="dom-embed-width">width</dfn>;
+  [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute DOMString <dfn attribute for="HTMLEmbedElement" data-x="dom-embed-height">height</dfn>;
   <span>Document</span>? <span data-x="dom-media-getSVGDocument">getSVGDocument</span>();
 
   // <a href="#HTMLEmbedElement-partial">also has obsolete members</a>
@@ -34539,8 +34543,8 @@ interface <dfn interface>HTMLObjectElement</dfn> : <span>HTMLElement</span> {
   [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute DOMString <dfn attribute for="HTMLObjectElement" data-x="dom-object-type">type</dfn>;
   [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute DOMString <dfn attribute for="HTMLObjectElement" data-x="dom-object-name">name</dfn>;
   readonly attribute <span>HTMLFormElement</span>? <span data-x="dom-fae-form">form</span>;
-  [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute DOMString <span data-x="dom-dim-width">width</span>;
-  [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute DOMString <span data-x="dom-dim-height">height</span>;
+  [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute DOMString <dfn attribute for="HTMLObjectElement" data-x="dom-object-width">width</dfn>;
+  [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute DOMString <dfn attribute for="HTMLObjectElement" data-x="dom-object-height">height</dfn>;
   readonly attribute <span>Document</span>? <span data-x="dom-object-contentDocument">contentDocument</span>;
   readonly attribute <span>WindowProxy</span>? <span data-x="dom-object-contentWindow">contentWindow</span>;
   <span>Document</span>? <span data-x="dom-media-getSVGDocument">getSVGDocument</span>();
@@ -35060,8 +35064,8 @@ interface <dfn interface>HTMLObjectElement</dfn> : <span>HTMLElement</span> {
 interface <dfn interface>HTMLVideoElement</dfn> : <span>HTMLMediaElement</span> {
   [<span>HTMLConstructor</span>] constructor();
 
-  [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute unsigned long <span data-x="dom-dim-width">width</span>;
-  [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute unsigned long <span data-x="dom-dim-height">height</span>;
+  [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute unsigned long <dfn attribute for="HTMLVideoElement" data-x="dom-video-width">width</dfn>;
+  [<span>CEReactions</span>, <span data-x="xattr-Reflect">Reflect</span>] attribute unsigned long <dfn attribute for="HTMLVideoElement" data-x="dom-video-height">height</dfn>;
   readonly attribute unsigned long <span data-x="dom-video-videoWidth">videoWidth</span>;
   readonly attribute unsigned long <span data-x="dom-video-videoHeight">videoHeight</span>;
   [<span>CEReactions</span>, <span data-x="xattr-ReflectURL">ReflectURL</span>] attribute USVString <dfn attribute for="HTMLVideoElement" data-x="dom-video-poster">poster</dfn>;