Bug 1823060 - Popover: when the attribute is null or empty, it shouldn't match popover='auto'. r=emilio

Differential Revision: https://phabricator.services.mozilla.com/D172897

dom/html/nsGenericHTMLElement.cpp

@@ -187,11 +187,10 @@ static const nsAttrValue::EnumTable kDirTable[] = {
 
 static const nsAttrValue::EnumTable kPopoverTable[] = {
     {"auto", PopoverState::Auto},
-    {"", PopoverState::Auto},
     {"manual", PopoverState::Manual},
     {nullptr, 0}};
 
-static const nsAttrValue::EnumTable* kPopoverTableDefault = &kPopoverTable[2];
+static const nsAttrValue::EnumTable* kPopoverTableDefault = &kPopoverTable[1];
 
 void nsGenericHTMLElement::AddToNameTable(nsAtom* aName) {
   MOZ_ASSERT(HasName(), "Node doesn't have name?");

testing/web-platform/meta/html/semantics/popovers/popover-appearance.html.ini

@@ -1,2 +0,0 @@
-[popover-appearance.html]
-  expected: FAIL

testing/web-platform/meta/html/semantics/popovers/popover-types-with-hints.tentative.html.ini

@@ -1,7 +1,4 @@
 [popover-types-with-hints.tentative.html]
-  [manuals do not close popovers]
-    expected: FAIL
-
   [autos close hints but not manuals]
     expected: FAIL
 
@@ -16,6 +13,3 @@
 
   [If you: a) show a popover=auto (call it D), then b) show a descendent popover=hint of D (call it T), then c) hide D, then T should be hidden. (A popover=hint can be nested inside a popover=auto)]
     expected: FAIL
-
-  [If you: a) show a popover=auto (call it D), then b) show a non-descendent popover=hint of D (call it T), then c) hide D, then T should be left showing. (Non-nested popover=hint can stay open when unrelated popover=autos are hidden)]
-    expected: FAIL

testing/web-platform/meta/html/semantics/popovers/popover-types.html.ini

@@ -1,3 +0,0 @@
-[popover-types.html]
-  [manuals do not close popovers]
-    expected: FAIL