RESOLVED FIXED243565
AX: AccessibilityObject::visibleCharacterRange sometimes returns content from the previous viewport boundary 
https://bugs.webkit.org/show_bug.cgi?id=243565
Summary AX: AccessibilityObject::visibleCharacterRange sometimes returns content from...
Tyler Wilcock
Reported 2022-08-04 16:03:41 PDT
AccessibilityObject::visibleCharacterRange sometimes returns content from the previous viewport boundary. This is a regression from https://bugs.webkit.org/show_bug.cgi?id=243286.
Attachments
Patch (7.24 KB, patch)
2022-08-04 17:05 PDT, Tyler Wilcock 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2022-08-04 16:03:53 PDT
<rdar://problem/98153603>
Tyler Wilcock
Comment 2 2022-08-04 17:05:13 PDT
Created attachment 461410 [details] Patch
Tyler Wilcock
Comment 3 2022-08-04 17:09:56 PDT
rdar://96884540
chris fleizach
Comment 4 2022-08-12 09:01:54 PDT
Comment on attachment 461410 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=461410&action=review > Source/WebCore/accessibility/AccessibilityObject.cpp:848 > + if (lineStartBoundaryPoint && lineStartBoundaryPoint->container.ptr() == startBoundary.container.ptr()) { can you not do equality check on a RetainPtr ?
Tyler Wilcock
Comment 5 2022-08-12 10:05:46 PDT
(In reply to chris fleizach from comment #4) > Comment on attachment 461410 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=461410&action=review > > > Source/WebCore/accessibility/AccessibilityObject.cpp:848 > > + if (lineStartBoundaryPoint && lineStartBoundaryPoint->container.ptr() == startBoundary.container.ptr()) { > > can you not do equality check on a RetainPtr ? These are Ref<Node>s, and unfortunately Ref's don't currently have an operator== implementation.
EWS
Comment 6 2022-08-12 11:15:02 PDT
Committed 253385@main (2559a63f0d9c): <https://commits.webkit.org/253385@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 461410 [details].
Note You need to log in before you can comment on or make changes to this bug.