Bug 257045 – AX: Schedule the accessibility regions paint before building the isolated tree.

• Status: RESOLVED
• Resolution: FIXED
• Component: Accessibility

[Andres Gonzalez]

This is important during the creation of the isolated tree in order to cache RelativeFrames as the isolated objects are created.

[Radar WebKit Bug Importer]

<rdar://problem/109577190>

[Andres Gonzalez]

Created attachment 466425 [details]
Patch

[Tyler Wilcock]

Comment on attachment 466425 [details]
Patch

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

> Source/WebCore/accessibility/isolatedtree/AXIsolatedObject.cpp:177
> +    else if (object.isScrollView() || object.isWebArea() || object.isScrollbar()) {

Scrollbars currently aren't handled in PaintPhase::Accessibility, so by caching this here we will never update it, which probably isn't right.

[Andres Gonzalez]

Created attachment 466447 [details]
Patch

[Andres Gonzalez]

(In reply to Tyler Wilcock from comment #3)
> Comment on attachment 466425 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=466425&action=review
> 
> > Source/WebCore/accessibility/isolatedtree/AXIsolatedObject.cpp:177
> > +    else if (object.isScrollView() || object.isWebArea() || object.isScrollbar()) {
> 
> Scrollbars currently aren't handled in PaintPhase::Accessibility, so by
> caching this here we will never update it, which probably isn't right.

Removed that change and filed rdar://109655440 to handle scrollbars in PaintPhase::Accessibility.

[EWS]

Committed 264320@main (1e29a8fa8f79): <https://commits.webkit.org/264320@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 466447 [details].

[Tyler Wilcock]

(In reply to Andres Gonzalez from comment #5)
> (In reply to Tyler Wilcock from comment #3)
> > Comment on attachment 466425 [details]
> > Patch
> > 
> > View in context:
> > https://bugs.webkit.org/attachment.cgi?id=466425&action=review
> > 
> > > Source/WebCore/accessibility/isolatedtree/AXIsolatedObject.cpp:177
> > > +    else if (object.isScrollView() || object.isWebArea() || object.isScrollbar()) {
> > 
> > Scrollbars currently aren't handled in PaintPhase::Accessibility, so by
> > caching this here we will never update it, which probably isn't right.
> 
> Removed that change and filed rdar://109655440 to handle scrollbars in
> PaintPhase::Accessibility.
The patch you landed is still caching the frame for object.isScrollbar(), so I think that's not right.