Summary: | AX: Accessibility needs to know which part of the content view is visible on iOS | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Nan Wang <n_wang> | ||||||
Component: | Accessibility | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | aboxhall, apinheiro, cfleizach, commit-queue, dmazzoni, ews-watchlist, jcraig, jdiggs, samuel_white, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Nan Wang
2018-04-27 13:06:10 PDT
Created attachment 339017 [details]
patch
Comment on attachment 339017 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=339017&action=review > Source/WebCore/accessibility/ios/WebAccessibilityObjectWrapperIOS.mm:1676 > + return CGRectZero; Does this account for objects in an iframe and various states of scroll offset and embedded iframes (In reply to chris fleizach from comment #3) > Comment on attachment 339017 [details] > patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=339017&action=review > > > Source/WebCore/accessibility/ios/WebAccessibilityObjectWrapperIOS.mm:1676 > > + return CGRectZero; > > Does this account for objects in an iframe and various states of scroll > offset and embedded iframes I think so, this is per frameview. Ok probably good to test that Created attachment 339021 [details]
patch
added iframe test case
Comment on attachment 339021 [details] patch Clearing flags on attachment: 339021 Committed r231113: <https://trac.webkit.org/changeset/231113> All reviewed patches have been landed. Closing bug. |