WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
245301
Complete id-ref reflection implementation for ARIAAttributes
https://bugs.webkit.org/show_bug.cgi?id=245301
Summary
Complete id-ref reflection implementation for ARIAAttributes
Ryosuke Niwa
Reported
2022-09-16 14:05:06 PDT
The implementation of id-ref reflection is incomplete. We need to finish this work before we can ship this feature.
Attachments
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2022-09-16 14:05:21 PDT
<
rdar://problem/100042185
>
Manuel Rego Casasnovas
Comment 2
2022-09-19 00:56:33 PDT
Thanks for filling the issues. (In reply to Ryosuke Niwa from
comment #0
)
> The implementation of id-ref reflection is incomplete. > We need to finish this work before we can ship this feature.
Should we switch off AriaReflectionForElementReferencesEnabled flag until these issues are fixed?
Ryosuke Niwa
Comment 3
2022-09-20 01:28:48 PDT
(In reply to Manuel Rego Casasnovas from
comment #2
)
> Thanks for filling the issues. > > (In reply to Ryosuke Niwa from
comment #0
) > > The implementation of id-ref reflection is incomplete. > > We need to finish this work before we can ship this feature. > > Should we switch off AriaReflectionForElementReferencesEnabled flag until > these issues are fixed?
I'd recommend that given the primary motivation of element reflection was making ARIA work with shadow DOM.
Ryosuke Niwa
Comment 4
2022-09-23 20:58:52 PDT
Pull request:
https://github.com/WebKit/WebKit/pull/4660
EWS
Comment 5
2022-09-23 21:01:44 PDT
Test gardening commit
254818@main
(9dc671ed1a7b): <
https://commits.webkit.org/254818@main
> Reviewed commits have been landed. Closing PR #4660 and removing active labels.
EWS
Comment 6
2022-10-10 11:34:04 PDT
Committed
255358@main
(3de868265f67): <
https://commits.webkit.org/255358@main
> Reviewed commits have been landed. Closing PR #5179 and removing active labels.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug