[webkit-reviews] review granted: [Bug 267477] AX: Add AccessibilityThreadTextApisEnabled flag and hook it up to several APIs : [Attachment 469387] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jan 16 11:41:10 PST 2024

chris fleizach <cfleizach at apple.com> has granted  review:
Bug 267477: AX: Add AccessibilityThreadTextApisEnabled flag and hook it up to
several APIs

Attachment 469387: Patch


--- Comment #5 from chris fleizach <cfleizach at apple.com> ---
Comment on attachment 469387
  --> https://bugs.webkit.org/attachment.cgi?id=469387

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

>>> +FAIL: textElement.textMarkerRangeLength(textMarkerRange) !== 43, was 39
>> Should we be committing these with the expected PASS value, rather than
FAIL? I see you have a FIXME below, but I wonder if it's better to get a
failure when running the test, rather than a false positive.
> That's a good question. I chose to commit FAIL to try to keep our post-commit
bot results clean...but thinking about it more, we're the only ones watching
that, and it seems pretty logical to just ignore failures in the
ax-thread-text-apis/ folder. Curious to hear what others think, but I think
committing the expected PASS (and letting the test fail) instead could make

Is the strategy to fix this before enabling feature? if so, probably ok to
commit expected results (PASS) and then when it gets enabled and not skipped,
we ensure it passes. otherwise easy to forget a task like this

More information about the webkit-reviews mailing list