Bug 144896

Summary: AX: [GTK] Defer to WebCore Accessibility for table exposure
Product: WebKit Reporter: Joanmarie Diggs <jdiggs>
Component: AccessibilityAssignee: Joanmarie Diggs <jdiggs>
Status: RESOLVED FIXED    
Severity: Normal CC: aboxhall, apinheiro, cfleizach, commit-queue, dmazzoni, jcraig, mario, samuel_white, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch none

Joanmarie Diggs
Reported 2015-05-11 21:32:55 PDT
WebKitGtk exposes all tables (data and layout) as tables, overriding WebCore Accessibility's default behavior in order to do so. The main motivation behind this platform difference was to make it possible for Orca to be able to provide consistent user experiences between Gecko and WebKitGtk. Given improved table identification heuristics in both engines, Orca now treats tables exposed with layout-guess:true as non-tables. Since Orca is deferring to the decision of WebKit, there doesn't seem to be much benefit in continuing to override WebCore Accessibility's default behavior in WebKitGtk.
Attachments
Patch (12.98 KB, patch)
2015-05-11 22:00 PDT, Joanmarie Diggs 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2015-05-11 21:33:22 PDT
<rdar://problem/20911379>
Joanmarie Diggs
Comment 2 2015-05-11 22:00:09 PDT
Created attachment 252935 [details] Patch
Joanmarie Diggs
Comment 3 2015-05-11 22:10:54 PDT
BTW, I just opened bug 144898 for the EFL port. I'm pretty sure they just inherited the GTK port's expectation wrt table exposure. But I could be wrong, and I cannot build WebKitEfl at the moment even if I were sure I was right. So we need one of those developers to chime in.
WebKit Commit Bot
Comment 4 2015-05-12 05:03:16 PDT
Comment on attachment 252935 [details] Patch Clearing flags on attachment: 252935 Committed r184199: <http://trac.webkit.org/changeset/184199>
WebKit Commit Bot
Comment 5 2015-05-12 05:03:20 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.