[webkit-reviews] review granted: [Bug 224617] AX: inverse role synonyms "presentation" and "none" to make "none" the primary : [Attachment 427078] patch

[bugzilla-daemon at webkit.org]

Devin Rousso <drousso at apple.com> has granted chris fleizach
<cfleizach at apple.com>'s request for review:
Bug 224617: AX: inverse role synonyms "presentation" and "none" to make "none"
the primary
https://bugs.webkit.org/show_bug.cgi?id=224617

Attachment 427078: patch

https://bugs.webkit.org/attachment.cgi?id=427078&action=review




--- Comment #9 from Devin Rousso <drousso at apple.com> ---
Comment on attachment 427078
  --> https://bugs.webkit.org/attachment.cgi?id=427078
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=427078&action=review

rs=me

> Source/WebCore/accessibility/AccessibilityObject.cpp:2182
> +    // http://webkit.org/b/224617 - prefer "none" as the canonical role to
be used for presentationals (instead of "presentation"

Rather than have a link to the bugzilla, can you explain inline why this is the
way it is (ideally with a link to a spec if possible)?