Ignore DOM state when hiding popovers if needed

Fixes whatwg#9161
Fixes whatwg#9367
Makes obsolete whatwg/dom#1185

This PR prevents the hide popover algorithm from returning early when
the popover attribute is removed or when the element with the popover
attribute is removed from the document.

The fireEvents parameter is used as an indicator that either the element
is being removed or that the attribute is being removed, and when it is
false, the calls to check popover validity are replaced with a check to
simply see if the popover is already hidden.

This patch also makes removal of the popover attribute stop firing
events in order to signal to the hide popover algorithm that checks for
the popover attribute should be ignored.

Author: josepharhar

source

@@ -82409,9 +82409,11 @@ dictionary <dfn dictionary>DragEventInit</dfn> : <span>MouseEventInit</span> {
    <li><p>If <var>localName</var> is not <code data-x="attr-popover">popover</code>, then
    return.</p></li>
 
+   <li><p>Let <var>fireEvents</var> be false if <var>value</var> is null, otherwise true.</p></li>
+
    <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, true, and false.</p></li>
+   <var>element</var>, true, <var>fireEvents</var>, and false.</p></li>
   </ol>
 
   <dl class="domintro">
@@ -82613,8 +82615,17 @@ dictionary <dfn dictionary>DragEventInit</dfn> : <span>MouseEventInit</span> {
   boolean <var>fireEvents</var>, and a boolean <var>throwExceptions</var>:</p>
 
   <ol>
-   <li><p>If the result of running <span>check popover validity</span> given <var>element</var>,
-   true, <var>throwExceptions</var>, and null is false, then return.</p></li>
+   <li>
+    <p>If <var>fireEvents</var> is true:</p>
+
+    <ol>
+     <li><p>If the result of running <span>check popover validity</span> given <var>element</var>,
+     true, <var>throwExceptions</var>, and null is false, then return.</p></li>
+    </ol>
+   </li>
+
+   <li><p>Otherwise, if <var>element</var>'s <span>popover visibility state</span> is <span
+   data-x="popover-hidden-state">hidden</span>, then return.</p></li>
 
    <li><p>Let <var>document</var> be <var>element</var>'s <span>node document</span>.</p></li>
 
@@ -82643,15 +82654,24 @@ dictionary <dfn dictionary>DragEventInit</dfn> : <span>MouseEventInit</span> {
      <var>element</var>, <var>focusPreviousElement</var>, and <var>fireEvents</var>.</p></li>
 
      <li>
-      <p>If the result of running <span>check popover validity</span> given <var>element</var>,
-      true, and <var>throwExceptions</var> is false, then run <var>cleanupHidingFlag</var> and
-      return.</p>
+      <p>If <var>fireEvents</var> is true:</p>
 
-      <p class="note"><span>Check popover validity</span> is called again because running <span
-      data-x="hide-all-popovers-until">hide all popovers until</span> could have disconnected
-      <var>element</var> or changed its <code data-x="attr-popover">popover</code> attribute.</p>
+      <ol>
+       <li>
+        <p>If the result of running <span>check popover validity</span> given <var>element</var>,
+        true, and <var>throwExceptions</var> is false, then run <var>cleanupHidingFlag</var> and
+        return.</p>
+
+        <p class="note"><span>Check popover validity</span> is called again because running <span
+        data-x="hide-all-popovers-until">hide all popovers until</span> could have disconnected
+        <var>element</var> or changed its <code data-x="attr-popover">popover</code> attribute.</p>
+       </li>
+      </ol>
      </li>
 
+     <li><p>Otherwise, if <var>element</var>'s <span>popover visibility state</span> is <span
+     data-x="popover-hidden-state">hidden</span>, then return.</p></li>
+
      <li><p>If the last item in <var>document</var>'s <span>auto popover list</span> is
      <var>element</var>, then <span>break</span>.</p></li>
     </ol>