Cleanup Path2D constructor

This reverts f635643 which added a constructor that did not get implemented. (Verified for Chrome, Firefox, and Safari by inspecting IDL.)

It also changes the constructor signature slightly to more idiomatic IDL and makes various editorial improvements.

Author: annevk

source

@@ -59915,10 +59915,7 @@ interface <dfn>ImageData</dfn> {
   readonly attribute <span data-x="idl-Uint8ClampedArray">Uint8ClampedArray</span> <span data-x="dom-imagedata-data">data</span>;
 };
 
-[<span data-x="dom-Path2D">Constructor</span>,
- <span data-x="dom-Path2D-copy">Constructor</span>(<span>Path2D</span> path),
- <span data-x="dom-Path2D-merge">Constructor</span>(sequence&lt;<span>Path2D</span>&gt; paths, optional <span>CanvasFillRule</span> fillRule = "<span data-x="dom-context-2d-fillRule-nonzero">nonzero</span>"),
- <span data-x="dom-Path2D-withdata">Constructor</span>(DOMString d),
+[<span data-x="dom-Path2D">Constructor</span>(optional (<span>Path2D</span> or DOMString) path),
  Exposed=(Window,Worker)]
 interface <dfn>Path2D</dfn> {
   void <span data-x="dom-Path2D-addPath">addPath</span>(<span>Path2D</span> path, optional <span>DOMMatrix2DInit</span> transform);
@@ -61784,143 +61781,51 @@ try {
   add text to paths.</p>
 
   <dl class="domintro">
-
    <dt><var>path</var> = new <code subdfn data-x="dom-Path2D">Path2D</code>()</dt>
+   <dd><p>Creates a new empty <code>Path2D</code> object.</p></dd>
 
+   <dt><var>path</var> = new <code data-x="dom-Path2D">Path2D</code>(<var>path</var>)</dt>
    <dd>
+    <p>When <var>path</var> is a <code>Path2D</code> object, returns a copy.</p>
 
-    <p>Creates a new empty <code>Path2D</code> object.</p>
-
-   </dd>
-
-   <dt><var>path</var> = new <code subdfn data-x="dom-Path2D-copy">Path2D</code>(<var>path</var>)</dt>
-
-   <dd>
-
-    <p>Creates a new <code>Path2D</code> object that is a copy of the argument.</p>
-
-   </dd>
-
-   <dt><var>path</var> = new <code subdfn data-x="dom-Path2D-merge">Path2D</code>(<var>paths</var> [, <var>fillRule</var> ] )</dt>
-
-   <dd>
-
-    <p>Creates a new <code>Path2D</code> object that describes a path that outlines the given <var>paths</var>, using the given fill rule.</p>
-
-   </dd>
-
-   <dt><var>path</var> = new <code subdfn data-x="dom-Path2D-withdata">Path2D</code>(<var>d</var>)</dt>
-
-   <dd>
-
-    <p>Creates a new path with the path described by the argument, interpreted as SVG path data. <ref spec=SVG></p>
-
+    <p>When <var>path</var> is a string, creates the path described by the argument, interpreted as
+    SVG path data. <ref spec=SVG>
    </dd>
 
    <dt><var>path</var> . <code subdfn data-x="dom-Path2D-addPath">addPath</code>(<var>path</var> [, <var>transform</var> ] )</dt>
-
-   <dd>
-
-    <p>Adds to the path the path given by the argument.</p>
-
-   </dd>
-
+   <dd><p>Adds to the path the path given by the argument.</p></dd>
   </dl>
 
   <div w-nodev>
 
-  <p>The <dfn><code data-x="dom-Path2D">Path2D()</code></dfn> constructor, when invoked, must return a
-  newly created <code>Path2D</code> object.</p>
-
-  <hr>
-
-  <p>The <dfn><code data-x="dom-Path2D-copy">Path2D(<var>path</var>)</code></dfn>
-  constructor, when invoked, must return a newly created <code>Path2D</code> object, to which the
-  subpaths of the argument are added. (In other words, it returns a copy of the argument.)</p>
-
-  <hr>
-
-  <p>The <dfn><code data-x="dom-Path2D-merge">Path2D(<var>paths</var>,
-  <var>fillRule</var>)</code></dfn> constructor, when invoked, must run these steps:</p>
+  <p>The <dfn><code data-x="dom-Path2D">Path2D(<var>path</var>)</code></dfn> constructor, when
+  invoked, must run these steps:</p>
 
   <ol>
+   <li><p>Let <var>output</var> be a new <code>Path2D</code> object.</p></li>
 
-   <li>
+   <li><p>If <var>path</var> is not given, then return <var>output</var>.</p></li>
 
-    <p>Run the appropriate step from the following list, based on the constructor's second
-    argument:</p>
-
-    <dl class="switch">
-
-     <dt>If it is "nonzero"</dt>
-
-     <dd>
-
-      <p>Let <var>merged path</var> be a <span data-x="concept-path">path</span> that
-      consists of a set of non-overlapping subpaths that exactly outline the points from which, in
-      any of the <span data-x="concept-path">paths</span> provided in the constructor's first
-      argument, the number of times a half-infinite straight line drawn from that point crosses a
-      subpath going in one direction is not equal to the number of times it crosses a subpath going
-      in the other direction.</p>
-
-     </dd>
-
-     <dt>If it is "evenodd"</dt>
-
-     <dd>
-
-      <p>Let <var>merged path</var> be a <span data-x="concept-path">path</span> that
-      consists of a set of non-overlapping subpaths that exactly outline the points from which, in
-      any of the <span data-x="concept-path">paths</span> provided in the constructor's first
-      argument, the number of times a half-infinite straight line drawn from that point crosses that
-      path is odd.</p>
-
-     </dd>
-
-    </dl>
-
-    <p>The subpaths in <var>merged path</var> must be oriented such that for any point,
-    the number of times a half-infinite straight line drawn from that point crosses a subpath is
-    even if and only if the number of times a half-infinite straight line drawn from that same point
-    crosses a subpath going in one direction is equal to the number of times it crosses a subpath
-    going in the other direction.</p>
-
-   </li>
-
-   <li><p>Add all the subpaths in <var>merged path</var> to the <code>Path2D</code>
-   object.</p></li>
-
-   <li><p>Set the <code>Path2D</code> object's <span>need new subpath</span> flag.</p></li>
-
-  </ol>
-
-  <hr>
-
-  <p>The <dfn><code data-x="dom-Path2D-withdata">Path2D(<var>d</var>)</code></dfn>
-  constructor, when invoked, must run these steps:</p>
-
-  <ol>
+   <li><p>If <var>path</var> is a <code>Path2D</code> object, then add all subpaths of
+   <var>path</var> to <var>output</var> and return <var>output</var>. (In other words, it returns a
+   copy of the argument.)</p></li>
 
    <li>
-
-    <p>Parse and interpret the <var>d</var> argument according to the SVG specification's
-    rules for path data, thus obtaining an SVG path. <ref spec=SVG></p>
+    <p>Let <var>svgPath</var> be the result of parsing and interpreting <var>path</var> according to
+    the SVG specification's rules for path data. <ref spec=SVG></p>
 
     <p class="note">The resulting path could be empty. SVG defines error handling rules for parsing
     and applying path data.</p>
-
    </li>
 
-   <li><p>Let (<var>x</var>, <var>y</var>) be the last point in the SVG
-   path.</p></li>
+   <li><p>Let (<var>x</var>, <var>y</var>) be the last point in <var>svgPath</var>.</p></li>
 
-   <li><p>Create a new <code>Path2D</code> object and add all the subpaths in the SVG path, if any,
-   to that <code>Path2D</code> object. </p>
+   <li><p>Add all the subpaths, if any, from <var>svgPath</var> to <var>output</var>.</p></li>
 
-   <li><p>Create a new subpath in the <code>Path2D</code> object with (<var>x</var>, <var>y</var>) as the only point in the subpath.</p></li>
-
-   <li><p>Return the <code>Path2D</code> object as the constructed object.</p></li>
+   <li><p>Create a new subpath in <var>output</var> with (<var>x</var>, <var>y</var>) as the only
+   point in the subpath.</p></li>
 
+   <li><p>Return <var>output</var>.</p></li>
   </ol>
 
   <hr>