<html>
    <head>
      <base href="https://bugs.webkit.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - AX: AOM: Implement relation type properties"
   href="https://bugs.webkit.org/show_bug.cgi?id=179500#c5">Comment # 5</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - AX: AOM: Implement relation type properties"
   href="https://bugs.webkit.org/show_bug.cgi?id=179500">bug 179500</a>
              from <span class="vcard"><a class="email" href="mailto:n_wang@apple.com" title="Nan Wang <n_wang@apple.com>"> <span class="fn">Nan Wang</span></a>
</span></b>
        <pre>Comment on <span class=""><a href="attachment.cgi?id=328064&action=diff" name="attach_328064" title="patch">attachment 328064</a> <a href="attachment.cgi?id=328064&action=edit" title="patch">[details]</a></span>
patch

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

<span class="quote">>> Source/WebCore/accessibility/AccessibilityObject.cpp:2194
>> +            return elements.first();

> What happens when elements contain more than one elements?
> Should that ever happen? If so, is it really correct to return nullptr in that case?
> Where is this behavior spec'ed?</span >

I implemented this as a convenient function for relation types that only contain one AccessibleNode. Maybe we should add an assert here to make sure it only contains one element.
The more than one elements case should be addressed in another bug. In that case we could have elementsValueForProperty(property)
<a href="https://wicg.github.io/aom/spec/#accessiblenodelist-interface">https://wicg.github.io/aom/spec/#accessiblenodelist-interface</a>

<span class="quote">>> Source/WebCore/accessibility/AccessibleNode.cpp:340
>> +    if (accessibleNodes.size() == 1)

> Is it possible for accessibleNodes to have more than one items? If so, is it really correct to return nullptr.
> If that's not possible, we should assert that accessibleNodes.size() is either 0 or 1,
> and this check should instead be replaced by if (accessibleNodes.size()).</span >

Yes I think an assert here is more appropriate. This is to cover the properties that only should contain one AccessibleNode.</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>