Correct canceling popovers step

Discovered while replacing the dontFireEvents variable with the more readable fireEvents.

Author: annevk

source

@@ -1827,7 +1827,7 @@ a.setAttribute('href', 'https://example.com/'); // change the content attribute
 
    <li><p>If <var>removedNode</var>'s <code data-x="attr-popover">popover</code> attribute is not in
    the <span data-x="attr-popover-none-state">no popover state</span>, then run the <span>hide
-   popover algorithm</span> given <var>removedNode</var>, false, true, and false.</p></li>
+   popover algorithm</span> given <var>removedNode</var>, false, false, and false.</p></li>
   </ol>
 
   <p>A <dfn id="insert-an-element-into-a-document" data-x="node is inserted into a document"
@@ -59601,7 +59601,7 @@ interface <dfn interface>HTMLDialogElement</dfn> : <span>HTMLElement</span> {
 
   <ol>
    <li><p>Run <span data-x="hide-all-popovers-until">hide all popovers until</span> given null,
-   false, and false.</p></li>
+   false, and true.</p></li>
 
    <li><p>Let <var>control</var> be null.</p></li>
 
@@ -81413,7 +81413,7 @@ dictionary <dfn dictionary>DragEventInit</dfn> : <span>MouseEventInit</span> {
 
    <li><p>If <var>oldValue</var> and <var>value</var> are in different <span
    data-x="attr-popover">states</span>, then run the <span>hide popover algorithm</span> given
-   <var>element</var>, true, false, and false.</p></li>
+   <var>element</var>, true, true, and false.</p></li>
   </ol>
 
   <dl class="domintro">
@@ -81505,7 +81505,7 @@ dictionary <dfn dictionary>DragEventInit</dfn> : <span>MouseEventInit</span> {
      ancestor</span> algorithm given <var>element</var>.</p></li>
 
      <li><p>Run <span data-x="hide-all-popovers-until">hide all popovers until</span> given
-     <var>ancestor</var>, false, and false.</p></li>
+     <var>ancestor</var>, false, and true.</p></li>
 
      <li>
       <p>If <var>originalType</var> is not equal to the value of <var>element</var>'s <code
@@ -81602,13 +81602,13 @@ dictionary <dfn dictionary>DragEventInit</dfn> : <span>MouseEventInit</span> {
   method steps are:</p>
 
   <ol>
-   <li><p>Run the <span>hide popover algorithm</span> given <span>this</span>, true, false, and
+   <li><p>Run the <span>hide popover algorithm</span> given <span>this</span>, true, true, and
    true.</p></li>
   </ol>
 
   <p>To <dfn data-x="hide popover algorithm">hide a popover</dfn> given an <span data-x="HTML
   elements">HTML element</span> <var>element</var>, a boolean <var>focusPreviousElement</var>, a
-  boolean <var>dontFireEvents</var>, and a boolean <var>throwExceptions</var>:</p>
+  boolean <var>fireEvents</var>, and a boolean <var>throwExceptions</var>:</p>
 
   <ol>
    <li>
@@ -81631,8 +81631,7 @@ dictionary <dfn dictionary>DragEventInit</dfn> : <span>MouseEventInit</span> {
 
     <ol>
      <li><p>Run <span data-x="hide-all-popovers-until">hide all popovers until</span> given
-     <var>element</var>, <var>focusPreviousElement</var>, and
-     <var>dontFireEvents</var>.</p></li>
+     <var>element</var>, <var>focusPreviousElement</var>, and <var>fireEvents</var>.</p></li>
 
      <li>
       <p>If <var>element</var> is not in <var>document</var>'s <span>auto popover list</span>:</p>
@@ -81656,7 +81655,7 @@ dictionary <dfn dictionary>DragEventInit</dfn> : <span>MouseEventInit</span> {
    <li><p>Set <var>element</var>'s <span>popover invoker</span> to null.</p></li>
 
    <li>
-    <p>If <var>dontFireEvents</var> is false:</p>
+    <p>If <var>fireEvents</var> is true:</p>
 
     <ol>
      <li><p><span data-x="concept-event-fire">Fire an event</span> named <code
@@ -81689,7 +81688,7 @@ dictionary <dfn dictionary>DragEventInit</dfn> : <span>MouseEventInit</span> {
    <li><p>Set <var>element</var>'s <span>popover visibility state</span> to <span
    data-x="popover-hidden-state">hidden</span>.</p></li>
 
-   <li><p>If <var>dontFireEvents</var> is false, then <span>queue a popover toggle event task</span>
+   <li><p>If <var>fireEvents</var> is true, then <span>queue a popover toggle event task</span>
    given <var>element</var>, "<code data-x="">open</code>", and "<code
    data-x="">closed</code>".</p></li>
 
@@ -81715,7 +81714,7 @@ dictionary <dfn dictionary>DragEventInit</dfn> : <span>MouseEventInit</span> {
   <ol>
    <li><p>If <span>this</span>'s <span>popover visibility state</span> is <span
    data-x="popover-showing-state">showing</span>, and <var>force</var> is not present or false, then
-   run the <span>hide popover algorithm</span> given <span>this</span>, true, false, and
+   run the <span>hide popover algorithm</span> given <span>this</span>, true, true, and
    true.</p></li>
 
    <li><p>Otherwise, if <var>force</var> is not present or true, then run <span>show popover</span>
@@ -81724,7 +81723,7 @@ dictionary <dfn dictionary>DragEventInit</dfn> : <span>MouseEventInit</span> {
 
   <p>To <dfn data-x="hide-all-popovers-until">hide all popovers until</dfn>, given an <span
   data-x="HTML elements">HTML element</span> <var>endpoint</var>, a boolean
-  <var>focusPreviousElement</var>, and a boolean <var>dontFireEvents</var>:</p>
+  <var>focusPreviousElement</var>, and a boolean <var>fireEvents</var>:</p>
 
   <ol>
    <li><p>Let <var>document</var> be <var>endpoint</var>'s <span>node document</span>.</p></li>
@@ -81741,7 +81740,7 @@ dictionary <dfn dictionary>DragEventInit</dfn> : <span>MouseEventInit</span> {
 
       <ol>
        <li><p>Run the <span>hide popover algorithm</span> given <var>popover</var>,
-       <var>focusPreviousElement</var>, <var>dontFireEvents</var>, and false.</p></li>
+       <var>focusPreviousElement</var>, <var>fireEvents</var>, and false.</p></li>
 
        <li><p>Set <var>popover</var> to <var>document</var>'s <span>topmost auto
        popover</span>.</p></li>
@@ -81779,8 +81778,8 @@ dictionary <dfn dictionary>DragEventInit</dfn> : <span>MouseEventInit</span> {
 
     <ol>
      <li><p>Run the <span>hide popover algorithm</span> given <var>document</var>'s <span>auto
-     popover list</span>'s last element, <var>focusPreviousElement</var>,
-     <var>dontFireEvents</var>, and false.</p></li>
+     popover list</span>'s last element, <var>focusPreviousElement</var>, <var>fireEvents</var>, and
+     false.</p></li>
     </ol>
    </li>
   </ol>
@@ -82066,7 +82065,7 @@ dictionary <dfn dictionary>DragEventInit</dfn> : <span>MouseEventInit</span> {
    <li><p>If <var>popover</var>'s <span>popover visibility state</span> is <span
    data-x="popover-showing-state">showing</span> and the result of running <span>check popover
    validity</span> given <var>popover</var> and true is true, then run the <span>hide popover
-   algorithm</span> given <var>popover</var>, true, false, and false.</p></li>
+   algorithm</span> given <var>popover</var>, true, true, and false.</p></li>
 
    <li>
     <p>Otherwise, if <var>popover</var>'s <span>popover visibility state</span> is <span
@@ -82139,11 +82138,11 @@ dictionary <dfn dictionary>DragEventInit</dfn> : <span>MouseEventInit</span> {
   a popover whose <code data-x="attr-popover">popover</code> attribute is in the <span
   data-x="attr-popover-auto-state">auto</span> state will close the popover.</p>
 
-  <p id="canceling-popovers"><strong>Canceling popovers</strong>: When <code>Document</code> has a
+  <p id="canceling-popovers"><strong>Canceling popovers</strong>: when <code>Document</code> has a
   <span>topmost auto popover</span> showing, user agents may provide a user interface that, upon
   activation, <span data-x="queue an element task">queues an element task</span> on the <span>user
   interaction task source</span> given <span>topmost auto popover</span> to run the <span>hide
-  popover algorithm</span> given the <span>topmost auto popover</span>, true, and false.</p>
+  popover algorithm</span> given the <span>topmost auto popover</span>, true, true, and false.</p>
 
   <p>To <dfn>light dismiss open popovers</dfn>, given an <code>Event</code> <var>event</var>:</p>
 
@@ -82188,7 +82187,7 @@ dictionary <dfn dictionary>DragEventInit</dfn> : <span>MouseEventInit</span> {
      <li><p>Set <var>document</var>'s <span>popover pointerdown target</span> to null.</p></li>
 
      <li><p>If <var>sameTarget</var> is true, then run <span data-x="hide-all-popovers-until">hide
-     all popovers until</span> given <var>ancestor</var>, false, and false.</p></li>
+     all popovers until</span> given <var>ancestor</var>, false, and true.</p></li>
     </ol>
    </li>
   </ol>