From ad3b725aaddfd18b1d7f8a784c5063741a6757b1 Mon Sep 17 00:00:00 2001 From: Luke Warlow Date: Thu, 31 Jul 2025 16:44:37 +0100 Subject: [PATCH] Imply [Reflect] when [ReflectRange] or [ReflectDefault] are used. This allows reducing noise in the IDL for the common case that no argument needs passing to [Reflect]. --- source | 38 ++++++++++++++++++++++---------------- 1 file changed, 22 insertions(+), 16 deletions(-) diff --git a/source b/source index a89f18aad87..191d133dee5 100644 --- a/source +++ b/source @@ -8864,17 +8864,20 @@ a.setAttribute('href', 'https://example.com/'); // change the content attribute
-

To supplement the above extended attributes we also - introduce To supplement the above extended attributes [ReflectRange], and [ReflectDefault]. These - augment how reflection works and also must only appear on interface - member attributes.

+ data-lt="ReflectDefault" data-x="xattr-ReflectDefault">[ReflectDefault] can be + used. These augment how reflection works and also must only appear + on interface member attributes.

The [ReflectRange] extended attribute must take an integer list limited to two values. It must only be used on attributes with a type of - unsigned long. Additionally, it must also only appear - alongside [Reflect].

+ unsigned long. Additionally, it must not appear alongside + any primary reflection extended attribute except [Reflect]. If there is no primary reflection extended attribute then, [Reflect] with no argument is implied.

IDL attributes with the [ReflectRange] extended attribute are clamped to the range [clampedMin, @@ -8886,11 +8889,14 @@ a.setAttribute('href', 'https://example.com/'); // change the content attribute must only be used on attributes with a type of double, long, or unsigned long. When used on an attribute of type double, it must take a decimal; otherwise - it must take an integer. Additionally, it must also only appear alongside reflection extended attribute except [Reflect], [ReflectNonNegative], [ReflectPositive], or [ReflectPositiveWithFallback].

+ data-x="xattr-ReflectPositiveWithFallback">[ReflectPositiveWithFallback]. If there is no + primary reflection extended attribute then, [Reflect] with no argument is implied.

IDL attributes with the [ReflectDefault] extended attribute have a default value provided by the argument @@ -20524,7 +20530,7 @@ interface HTMLOListElement : HTMLElement { [HTMLConstructor] constructor(); [CEReactions, Reflect] attribute boolean reversed; - [CEReactions, Reflect, ReflectDefault=1] attribute long start; + [CEReactions, ReflectDefault=1] attribute long start; [CEReactions, Reflect] attribute DOMString type; // also has obsolete members @@ -43429,7 +43435,7 @@ the cell that corresponds to the values of the two dice. interface HTMLTableColElement : HTMLElement { [HTMLConstructor] constructor(); - [CEReactions, Reflect, ReflectDefault=1, ReflectRange=(1, 1000)] attribute unsigned long span; + [CEReactions, ReflectDefault=1, ReflectRange=(1, 1000)] attribute unsigned long span; // also has obsolete members }; @@ -43931,8 +43937,8 @@ interface HTMLTableRowElement : HTMLElement { interface HTMLTableCellElement : HTMLElement { [HTMLConstructor] constructor(); - [CEReactions, Reflect, ReflectDefault=1, ReflectRange=(1, 1000)] attribute unsigned long colSpan; - [CEReactions, Reflect, ReflectDefault=1, ReflectRange=(0, 65534)] attribute unsigned long rowSpan; + [CEReactions, ReflectDefault=1, ReflectRange=(1, 1000)] attribute unsigned long colSpan; + [CEReactions, ReflectDefault=1, ReflectRange=(0, 65534)] attribute unsigned long rowSpan; [CEReactions, Reflect] attribute DOMString headers; readonly attribute long cellIndex; @@ -53906,7 +53912,7 @@ interface HTMLSelectElement : HTMLElement { [CEReactions, Reflect] attribute boolean multiple; [CEReactions, Reflect] attribute DOMString name; [CEReactions, Reflect] attribute boolean required; - [CEReactions, Reflect, ReflectDefault=0] attribute unsigned long size; + [CEReactions, ReflectDefault=0] attribute unsigned long size; @@ -140835,8 +140841,8 @@ interface HTMLMarqueeElement : HTMLElement { [CEReactions, Reflect] attribute DOMString height; [CEReactions, Reflect] attribute unsigned long hspace; [CEReactions] attribute long loop; - [CEReactions, Reflect, ReflectDefault=6] attribute unsigned long scrollAmount; - [CEReactions, Reflect, ReflectDefault=85] attribute unsigned long scrollDelay; + [CEReactions, ReflectDefault=6] attribute unsigned long scrollAmount; + [CEReactions, ReflectDefault=85] attribute unsigned long scrollDelay; [CEReactions, Reflect] attribute boolean trueSpeed; [CEReactions, Reflect] attribute unsigned long vspace; [CEReactions, Reflect] attribute DOMString width;