Bug 215000

Summary: Add the ability of comparing the accessibility tree with isolated tree mode on and off.
Product: WebKit Reporter: Andres Gonzalez <andresg_22>
Component: New BugsAssignee: Andres Gonzalez <andresg_22>
Status: RESOLVED FIXED    
Severity: Normal CC: aboxhall, apinheiro, cfleizach, dmazzoni, ews-watchlist, jcraig, jdiggs, samuel_white, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch none

Description Andres Gonzalez 2020-07-30 18:14:42 PDT 
Add the ability of comparing the accessibility tree with isolated tree mode on and off.
Comment 1 Andres Gonzalez 2020-07-30 18:25:19 PDT 
Created attachment 405650 [details]
Patch
Comment 2 chris fleizach 2020-07-30 23:40:33 PDT 
Comment on attachment 405650 [details]
Patch

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

> LayoutTests/accessibility/mac/isolated-tree-mode-on-off.html:20
> +    if (window.accessibilityController) {

side-but related topic- can we also add something to ensure that the isolated tree objects are returned? maybe we can return an attribute from the WebObjectWrapper that exists only when our backing object is an isolated tree object
Comment 3 Andres Gonzalez 2020-07-31 08:50:47 PDT 
Created attachment 405696 [details]
Patch
Comment 4 chris fleizach 2020-07-31 10:09:11 PDT 
lots of build fails
Comment 5 Andres Gonzalez 2020-07-31 10:59:06 PDT 
Created attachment 405708 [details]
Patch
Comment 6 Andres Gonzalez 2020-07-31 11:22:13 PDT 
Created attachment 405712 [details]
Patch
Comment 7 Andres Gonzalez 2020-07-31 12:03:32 PDT 
Created attachment 405719 [details]
Patch
Comment 8 Andres Gonzalez 2020-07-31 12:54:16 PDT 
Created attachment 405724 [details]
Patch
Comment 9 Andres Gonzalez 2020-07-31 13:48:45 PDT 
Created attachment 405734 [details]
Patch
Comment 10 Andres Gonzalez 2020-08-01 11:08:32 PDT 
Created attachment 405787 [details]
Patch
Comment 11 chris fleizach 2020-08-01 11:20:07 PDT 
Comment on attachment 405787 [details]
Patch

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

> Source/WebCore/accessibility/mac/WebAccessibilityObjectWrapperBase.mm:379
> +    return backingObject ? backingObject->isAXIsolatedObjectInstance() : NO;

This could be

return backingObject && backingObject->.....
Comment 12 Andres Gonzalez 2020-08-02 10:31:31 PDT 
Created attachment 405811 [details]
Patch
Comment 13 Andres Gonzalez 2020-08-03 09:20:21 PDT 
Created attachment 405837 [details]
Patch
Comment 14 chris fleizach 2020-08-03 09:25:53 PDT 
Comment on attachment 405837 [details]
Patch

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

> Source/WebCore/accessibility/mac/WebAccessibilityObjectWrapperBase.mm:379
> +    return backingObject ? backingObject->isAXIsolatedObjectInstance() : NO;

this could just be

return backingObject && backingObject->isAXIsolatedObjectInstance();
Comment 15 Andres Gonzalez 2020-08-03 09:34:17 PDT 
Created attachment 405840 [details]
Patch
Comment 16 EWS 2020-08-03 16:15:30 PDT 
Tools/Scripts/svn-apply failed to apply attachment 405840 [details] to trunk.
Please resolve the conflicts and upload a new patch.
Comment 17 Andres Gonzalez 2020-08-03 17:29:20 PDT 
Created attachment 405895 [details]
Patch
Comment 18 EWS 2020-08-04 09:56:38 PDT 
Committed r265248: <https://trac.webkit.org/changeset/265248>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 405895 [details].
Comment 19 Radar WebKit Bug Importer 2020-08-04 09:57:17 PDT 
<rdar://problem/66528080>