<html>
    <head>
      <base href="https://bugs.webkit.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Enhance support of aria-haspopup per ARIA 1.1 specification."
   href="https://bugs.webkit.org/show_bug.cgi?id=199216#c9">Comment # 9</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Enhance support of aria-haspopup per ARIA 1.1 specification."
   href="https://bugs.webkit.org/show_bug.cgi?id=199216">bug 199216</a>
              from <span class="vcard"><a class="email" href="mailto:andresg_22@apple.com" title="Andres Gonzalez <andresg_22@apple.com>"> <span class="fn">Andres Gonzalez</span></a>
</span></b>
        <pre>(In reply to chris fleizach from <a href="show_bug.cgi?id=199216#c2">comment #2</a>)
<span class="quote">> Comment on <span class="bz_obsolete"><a href="attachment.cgi?id=372892&action=diff" name="attach_372892" title="Patch">attachment 372892</a> <a href="attachment.cgi?id=372892&action=edit" title="Patch">[details]</a></span>
> Patch

> View in context:
> <a href="https://bugs.webkit.org/attachment.cgi?id=372892&action=review">https://bugs.webkit.org/attachment.cgi?id=372892&action=review</a>

> > Source/WebCore/accessibility/AccessibilityObject.cpp:2767
> > +    if (equalLettersIgnoringASCIICase(hasPopup, "dialog"))

> should we make a static HashSet of valid values and then just check based on
> passing in the popupAttr.lowercaseString() ?</span >

Thanks for the suggestion. Using HashSet in latest patch. Doesn't buy us much in terms of performance, but code is more elegant and compact. Still have to handle a couple of cases separately. Was looking for an equivalent to boost::hana strings in WTF, but couldn't find it. boost::hana would be ideal for cases like this because the hana strings are as efficient as literals but the set also have the searching capabilities like contains.

Also fixed the DumpRenderTree build and test in latest upload.</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>