Change DOMQuad#bounds to non-live getBounds()

zcorpan · svgeesus · commit 4f477e55edc8 · 2025-12-02T10:49:32.000-05:00
Fixes https://lists.w3.org/Archives/Public/public-fx/2015OctDec/0056.html.
diff --git a/geometry/Overview.bs b/geometry/Overview.bs
@@ -21,6 +21,7 @@ Abstract: This specification provides basic geometric interfaces to represent po
 !Issues list: <a href="https://www.w3.org/Bugs/Public/buglist.cgi?product=FXTF&amp;component=Geometry&amp;resolution=---">Bugzilla</a> (<a href="https://www.w3.org/Bugs/Public/enter_bug.cgi?product=FXTF&amp;component=Geometry">file a bug</a>)
 Test Suite: http://test.csswg.org/suites/geometry-1_dev/nightly-unstable/
 Use <i> Autolinks: yes
+Ignored Vars: f, Point object, m44, e, m42, quad1, m41, stringifier, point object, m12, c, transformPoint, m21, d, m22, quad2
 </pre>
 
 <pre class=anchors>
@@ -59,7 +60,7 @@ A 2D or a 3D <dfn>point</dfn> can be represented by the following WebIDL interfa
              optional unrestricted double z = 0, optional unrestricted double w = 1),
  Exposed=(Window,Worker)]
 interface DOMPointReadOnly {
-    [NewObject] static DOMPointReadOnly fromPoint(optional DOMPointInit point);
+    [NewObject] static DOMPointReadOnly fromPoint(optional DOMPointInit other);
 
     readonly attribute unrestricted double x;
     readonly attribute unrestricted double y;
@@ -104,8 +105,8 @@ The <dfn dfn-type=constructor dfn-for=DOMPointReadOnly><code>DOMPointReadOnly(<v
   <li>Return <var>point</var>.</li>
 </ol>
 
-The <dfn dfn-for=DOMPointReadOnly id=dom-dompointreadonly-frompoint><code>fromPoint(<var>other</var>)</code></dfn> static method on {{DOMPointReadOnly}} and
-the <dfn dfn-for=DOMPoint id=dom-dompoint-frompoint><code>fromPoint(<var>other</var>)</code></dfn> static method on {{DOMPoint}} must run the following steps:
+The <dfn method dfn-for=DOMPointReadOnly id=dom-dompointreadonly-frompoint><code>fromPoint(<var>other</var>)</code></dfn> static method on {{DOMPointReadOnly}} and
+the <dfn method dfn-for=DOMPoint id=dom-dompoint-frompoint><code>fromPoint(<var>other</var>)</code></dfn> static method on {{DOMPoint}} must run the following steps:
 
 <ol class=algorithm>
   <li>Let <var>point</var> be a new {{DOMPointReadOnly}} or {{DOMPoint}} as appropriate.
@@ -271,7 +272,7 @@ interface DOMQuad {
     [SameObject] readonly attribute DOMPoint p2;
     [SameObject] readonly attribute DOMPoint p3;
     [SameObject] readonly attribute DOMPoint p4;
-    [SameObject] readonly attribute DOMRectReadOnly bounds;
+    [NewObject] DOMRect getBounds();
 };
 
 dictionary DOMQuadInit {
@@ -289,7 +290,7 @@ The <dfn dfn-type=constructor dfn-for=DOMQuad><code>DOMQuad(<var>p1</var>, <var>
   <li>Let <var>point 2</var> be a new <a interface>DOMPoint</a> object with its attributes set to the values of the namesake dictionary members in <var>p2</var>.
   <li>Let <var>point 3</var> be a new <a interface>DOMPoint</a> object with its attributes set to the values of the namesake dictionary members in <var>p3</var>.
   <li>Let <var>point 4</var> be a new <a interface>DOMPoint</a> object with its attributes set to the values of the namesake dictionary members in <var>p4</var>.
-  <li><p>Return a new <a interface>DOMQuad</a> with <a attribute for=DOMQuad>p1</a> set to <var>point 1</var>, <a attribute for=DOMQuad>p2</a> set to <var>point 2</var>, <a attribute for=DOMQuad>p3</a> set to <var>point 3</var> and <a attribute for=DOMQuad>p4</a> set to <var>point 4</var>, and let the <a>associated bounding rectangle</a> be <var>bounds</var>.
+  <li><p>Return a new <a interface>DOMQuad</a> with <a attribute for=DOMQuad>p1</a> set to <var>point 1</var>, <a attribute for=DOMQuad>p2</a> set to <var>point 2</var>, <a attribute for=DOMQuad>p3</a> set to <var>point 3</var> and <a attribute for=DOMQuad>p4</a> set to <var>point 4</var>.
 </ol>
 
 Note: It is possible to pass <a interface>DOMPoint</a>/<a interface>DOMPointReadOnly</a> arguments as well. The passed arguments will be transformed to the correct object type internally following the WebIDL rules [[!WEBIDL]].
@@ -302,7 +303,7 @@ The <dfn method lt=fromRect() dfn-for=DOMQuad><code>fromRect(<var>other</var>)</
   <li>Let <var>point 2</var> be a new <a interface>DOMPoint</a> object with <a attribute for=DOMPoint>x</a> set to <var>x</var> + <var>width</var>, <a attribute for=DOMPoint>y</a> set to <var>y</var>, <a attribute for=DOMPoint>z</a> set to zero and <a attribute for=DOMPoint>w</a> set to one.
   <li>Let <var>point 3</var> be a new <a interface>DOMPoint</a> object with <a attribute for=DOMPoint>x</a> set to <var>x</var> + <var>width</var>, <a attribute for=DOMPoint>y</a> set to <var>y</var> + <var>height</var>, <a attribute for=DOMPoint>z</a> set to zero and <a attribute for=DOMPoint>w</a> set to one.
   <li>Let <var>point 4</var> be a new <a interface>DOMPoint</a> object with <a attribute for=DOMPoint>x</a> set to <var>x</var>, <a attribute for=DOMPoint>y</a> set to <var>y</var> + <var>height</var>, <a attribute for=DOMPoint>z</a> set to zero and <a attribute for=DOMPoint>w</a> set to one.
-  <li><p>Return a new <a interface>DOMQuad</a> with <a attribute for=DOMQuad>p1</a> set to <var>point 1</var>, <a attribute for=DOMQuad>p2</a> set to <var>point 2</var>, <a attribute for=DOMQuad>p3</a> set to <var>point 3</var> and <a attribute for=DOMQuad>p4</a> set to <var>point 4</var>, and let the <a>associated bounding rectangle</a> be <var>bounds</var>.
+  <li><p>Return a new <a interface>DOMQuad</a> with <a attribute for=DOMQuad>p1</a> set to <var>point 1</var>, <a attribute for=DOMQuad>p2</a> set to <var>point 2</var>, <a attribute for=DOMQuad>p3</a> set to <var>point 3</var> and <a attribute for=DOMQuad>p4</a> set to <var>point 4</var>.
 </ol>
 
 The <dfn method lt=fromQuad() dfn-for=DOMQuad><code>fromQuad(<var>other</var>)</code></dfn> static method on {{DOMQuad}} must run the following steps:
@@ -325,20 +326,18 @@ The <dfn method lt=fromQuad() dfn-for=DOMQuad><code>fromQuad(<var>other</var>)</
  The <dfn>p3</dfn> attribute must return a <a interface>DOMPoint</a> that represents <a attribute>p3</a> of the quadrilateral it was invoked on. The author can modify the returned <a interface>DOMPoint</a> object, which directly affects the quadrilateral.
 
  The <dfn>p4</dfn> attribute must return a <a interface>DOMPoint</a> that represents <a attribute>p4</a> of the quadrilateral it was invoked on. The author can modify the returned <a interface>DOMPoint</a> object, which directly affects the quadrilateral.
-
- The <dfn>bounds</dfn> attribute must return the <a>associated bounding rectangle</a> of the quadrilateral it was invoked on.
 </div>
 
-<a interface>DOMQuad</a> objects have an <dfn for=quadlirateral>associated bounding rectangle</dfn> set to a <a interface>DOMRectReadOnly</a> object when created. <a interface>DOMQuad</a> objects must have read and write access to the internal member variables of the <a>associated bounding rectangle</a>. The <a>associated bounding rectangle</a> is live; whenever the objects for <a attribute>p1</a>, <a attribute>p2</a>, <a attribute>p3</a> or <a attribute>p4</a> change, the <a>associated bounding rectangle</a> must update its internal member variable values as appropriate to describe the new smallest bounding box of the four points.
+The <dfn method for=DOMQuad>getBounds()</dfn> method, when invoked, must run the following algorithm:
 
-The <a>associated bounding rectangle</a> is computed as follows:
 <ol class='algorithm'>
-  <li>Let <var>bounds</var> be a <a interface>DOMRectReadOnly</a> object.</li>
+  <li>Let <var>bounds</var> be a <a interface>DOMRect</a> object.</li>
   <li>Let <var>left</var> be the minimum of <a attribute>p1</a>.x, <a attribute>p2</a>.x, <a attribute>p3</a>.x and <a attribute>p4</a>.x.</li>
   <li>Let <var>top</var> be the minimum of <a attribute>p1</a>.y, <a attribute>p2</a>.y, <a attribute>p3</a>.y and <a attribute>p4</a>.y.</li>
   <li>Let <var>right</var> be the maximum of <a attribute>p1</a>.x, <a attribute>p2</a>.x, <a attribute>p3</a>.x and <a attribute>p4</a>.x.</li>
   <li>Let <var>bottom</var> be the maximum of <a attribute>p1</a>.y, <a attribute>p2</a>.y, <a attribute>p3</a>.y and <a attribute>p4</a>.y.</li>
   <li>Let <a for=rectangle>x coordinate</a> of <var>bounds</var> be <var>left</var>, <a for=rectangle>y coordinate</a> of <var>bounds</var> be <var>top</var>, <a for=rectangle>width dimension</a> of <var>bounds</var> be <var>right</var> - <var>left</var> and <a for=rectangle>height dimension</a> of <var>bounds</var> be <var>bottom</var> - <var>top</var>.</li>
+  <li>Return <var>bounds</var>.
 </ol>
 
 <div class='example'>
@@ -363,7 +362,7 @@ var quad2 = DOMQuad.fromRect(rect);</code></pre>
     <circle cx="210" cy="150" r="3" fill="rgb(204, 51, 51)"/>
     <circle cx="10" cy="180" r="3" fill="rgb(204, 51, 51)"/>
   </svg>
-  <p class="capture">An irregular quadrilateral represented by a <a interface>DOMQuad</a>. The four red colored circles represent the <a interface>DOMPoint</a> attributes <a attribute>p1</a> to <a attribute>p4</a>. The dashed rectangle represents the <a>associated bounding rectangle</a> <a attribute>bounds</a> of the <a interface>DOMQuad</a>.</p>
+  <p class="capture">An irregular quadrilateral represented by a <a interface>DOMQuad</a>. The four red colored circles represent the <a interface>DOMPoint</a> attributes <a attribute>p1</a> to <a attribute>p4</a>. The dashed rectangle represents the bounding rectangle returned by the {{DOMQuad/getBounds()}} method of the <a interface>DOMQuad</a>.</p>
 </div>
 </div>
 
@@ -673,8 +672,8 @@ the <dfn method lt=fromMatrix() dfn-for=DOMMatrix id=dom-dommatrix-frommatrix><c
   </li>
 </ol>
 
-The <dfn method lt=fromString() dfn-for=DOMMatrixReadOnly id=dom-dommatrixreadonly-fromstring><code>fromString(<var>transformList</var>)</code></dfn> static method on {{DOMMatrixReadOnly}} and
-the <dfn method lt=fromString() dfn-for=DOMMatrix id=dom-dommatrix-fromstring><code>fromString(<var>transformList</var>)</code></dfn> static method on {{DOMMatrix}} must follow these steps:
+The <dfn method dfn-for=DOMMatrixReadOnly id=dom-dommatrixreadonly-fromstring><code>fromString(<var>transformList</var>)</code></dfn> static method on {{DOMMatrixReadOnly}} and
+the <dfn method dfn-for=DOMMatrix id=dom-dommatrix-fromstring><code>fromString(<var>transformList</var>)</code></dfn> static method on {{DOMMatrix}} must follow these steps:
 
 <ol>
   <li>Parse <var>transformList</var> by following the syntax description in “<a href=http://www.w3.org/TR/css-transforms-1/#svg-syntax>Syntax of the SVG ‘transform’ attribute</a>” [[!CSS3-TRANSFORMS]] to a <<transform-list>>. If parsing is not successful or any <<transform-function>> has <<length>> values without <a spec='css-values'>absolute length</a> units<!--For WD: <a spec='css-values-3'>absolute length units</a>-->, throw a {{SyntaxError}} exception.</li>
@@ -691,15 +690,15 @@ the <dfn method lt=fromString() dfn-for=DOMMatrix id=dom-dommatrix-fromstring><c
   <li>
     <dl class=switch>
       <dt>If <var>2dTransform</var> is set to <code>true</code></dt>
-      <dd>Return the result of invoking <a>create a 2d matrix</a> of type {{DOMMatrixReadOnly}} or {{DOMMatrix}} as appropriate, with a sequence of numbers, the values being the elements <var>m11</var>, <var>m12</var>, <var>21</var>, <var>m22</var>, <var>m41</var> and <var>m42</var> of <var>matrix</var> of <var>matrix</var>.</dd>
+      <dd>Return the result of invoking <a>create a 2d matrix</a> of type {{DOMMatrixReadOnly}} or {{DOMMatrix}} as appropriate, with a sequence of numbers, the values being the elements <var>m11</var>, <var>m12</var>, <var>m21</var>, <var>m22</var>, <var>m41</var> and <var>m42</var> of <var>matrix</var> of <var>matrix</var>.</dd>
       <dt>If <var>2dTransform</var> is set to <code>false</code></dt>
       <dd>Return the result of invoking <a>create a 3d matrix</a> of type {{DOMMatrixReadOnly}} or {{DOMMatrix}} as appropriate, with a sequence of numbers, the values being the 16 elements of <var>matrix</var>.</dd>
     </dl>
   </li>
 </ol>
 
-The <dfn method lt=fromFloat32Array() dfn-for=DOMMatrixReadOnly id=dom-dommatrixreadonly-fromfloat32array><code>fromFloat32Array(<var>array32</var>)</code></dfn> static method on {{DOMMatrixReadOnly}} and
-the <dfn method lt=fromFloat32Array() dfn-for=DOMMatrix id=dom-dommatrix-fromfloat32array><code>fromFloat32Array(<var>array32</var>)</code></dfn> static method on {{DOMMatrix}} must follow these steps:
+The <dfn method dfn-for=DOMMatrixReadOnly id=dom-dommatrixreadonly-fromfloat32array><code>fromFloat32Array(<var>array32</var>)</code></dfn> static method on {{DOMMatrixReadOnly}} and
+the <dfn method dfn-for=DOMMatrix id=dom-dommatrix-fromfloat32array><code>fromFloat32Array(<var>array32</var>)</code></dfn> static method on {{DOMMatrix}} must follow these steps:
 
 <dl class=switch>
   <dt>If <var>array32</var> has 6 elements</dt>
@@ -710,8 +709,8 @@ the <dfn method lt=fromFloat32Array() dfn-for=DOMMatrix id=dom-dommatrix-fromflo
   <dd>Throw a <code>TypeError</code> exception.</dd>
 </dl>
 
-The <dfn method lt=fromFloat64Array() dfn-for=DOMMatrixReadOnly id=dom-dommatrixreadonly-fromfloat64array><code>fromFloat64Array(<var>array64</var>)</code></dfn> static method on {{DOMMatrixReadOnly}} and
-the <dfn method lt=fromFloat64Array() dfn-for=DOMMatrix id=dom-dommatrix-fromfloat64array><code>fromFloat64Array(<var>array64</var>)</code></dfn> static method on {{DOMMatrix}} must follow these steps:
+The <dfn method dfn-for=DOMMatrixReadOnly id=dom-dommatrixreadonly-fromfloat64array><code>fromFloat64Array(<var>array64</var>)</code></dfn> static method on {{DOMMatrixReadOnly}} and
+the <dfn method dfn-for=DOMMatrix id=dom-dommatrix-fromfloat64array><code>fromFloat64Array(<var>array64</var>)</code></dfn> static method on {{DOMMatrix}} must follow these steps:
 
 <dl class=switch>
   <dt>If <var>array64</var> has 6 elements</dt>
@@ -1123,6 +1122,7 @@ The following changes were made since the <a href="http://www.w3.org/TR/2014/WD-
 
 <ul>
   <li>Exposed <a interface>DOMPointReadOnly</a>, <a interface>DOMPoint</a>, <a interface>DOMRectReadOnly</a>, <a interface>DOMRect</a>, <a interface>DOMQuad</a>, <a interface>DOMMatrixReadOnly</a> and <a interface>DOMMatrix</a> to <code>Window</code> and <code>Worker</code>. Defined cloning of the interfaces.</li>
+  <li>The live <code>bounds</code> attribute on {{DOMQuad}} was replaced with a non-live {{DOMQuad/getBounds()}} method. The "associated bounding rectangle" concept was also removed.
 </ul>
 
 The following changes were made since the <a href="http://www.w3.org/TR/2014/WD-geometry-1-20140626/">26 June 2014 Last Call Public Working Draft</a>.
