Bug 1882430 - Ensure isUnlabeledUrlBarOption is checked before failin…

…g the label check in AccessibilityUtils. r=Jamie

Adding the check in one more place that is failing on Linux.

This should resolve the failing of `browser/components/urlbar/tests/browser/browser_acknowledgeFeedbackAndDismissal.js` test with a11y-checks jobs, thus removing the temporary `fail-if` notation from its test manifest.

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

browser/components/urlbar/tests/browser/browser.toml

@@ -71,7 +71,6 @@ skip-if = [
 ]
 
 ["browser_acknowledgeFeedbackAndDismissal.js"]
-fail-if = ["a11y_checks"] # Bug 1882430 while the general patch is under review
 
 ["browser_action_searchengine.js"]
 skip-if = [

testing/mochitest/tests/SimpleTest/AccessibilityUtils.js

@@ -951,7 +951,11 @@ this.AccessibilityUtils = (function () {
     if (gEnv.labelRule && !name) {
       // The URL and Search Bar comboboxes are purposefully unlabeled,
       // since they include labeled inputs that are receiving focus.
-      if (isUnlabeledUrlBarCombobox(DOMNode)) {
+      // Or the Accessible died because the DOM node was removed or hidden.
+      if (
+        isUnlabeledUrlBarCombobox(DOMNode) ||
+        isUnlabeledUrlBarOption(DOMNode)
+      ) {
         return;
       }
       a11yFail("Interactive elements must be labeled", accessible);