Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AX: Remove AccessibilityObjectModelEnabled setting #8295

Merged
merged 1 commit into from Jan 7, 2023

Conversation

nt1m
Copy link
Member

@nt1m nt1m commented Jan 6, 2023

94dae17

AX: Remove AccessibilityObjectModelEnabled setting
https://bugs.webkit.org/show_bug.cgi?id=250193
rdar://103943537

Reviewed by Chris Fleizach.

The setting does nothing. Also remove SPIs which aren't used internally.

* Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml:
* Source/WebCore/page/DeprecatedGlobalSettings.h:
(WebCore::DeprecatedGlobalSettings::setAccessibilityObjectModelEnabled): Deleted.
(WebCore::DeprecatedGlobalSettings::accessibilityObjectModelEnabled): Deleted.
* Source/WebKit/UIProcess/API/C/WKPreferences.cpp:
(WKPreferencesSetAccessibilityObjectModelEnabled): Deleted.
(WKPreferencesGetAccessibilityObjectModelEnabled): Deleted.
* Source/WebKit/UIProcess/API/C/WKPreferencesRefPrivate.h:
* Source/WebKitLegacy/mac/WebView/WebPreferenceKeysPrivate.h:
* Source/WebKitLegacy/mac/WebView/WebPreferences.mm:
(-[WebPreferences accessibilityObjectModelEnabled]): Deleted.
(-[WebPreferences setAccessibilityObjectModelEnabled:]): Deleted.
* Source/WebKitLegacy/mac/WebView/WebPreferencesPrivate.h:

Canonical link: https://commits.webkit.org/258571@main

ae4ad4d

Misc iOS, tvOS & watchOS macOS Linux Windows
βœ… πŸ§ͺ style βœ… πŸ›  ios βœ… πŸ›  mac βœ… πŸ›  wpe   πŸ›  πŸ§ͺ win
βœ… πŸ§ͺ bindings βœ… πŸ›  ios-sim βœ… πŸ›  mac-AS-debug   πŸ›  gtk   πŸ›  wincairo
βœ… πŸ§ͺ webkitperl   πŸ§ͺ ios-wk2   πŸ§ͺ api-mac   πŸ§ͺ gtk-wk2
  πŸ§ͺ api-ios βœ… πŸ§ͺ mac-wk1   πŸ§ͺ api-gtk
  πŸ›  πŸ§ͺ jsc   πŸ›  tv   πŸ§ͺ mac-wk2 βœ… πŸ›  jsc-armv7
βœ… πŸ›  πŸ§ͺ jsc-arm64 βœ… πŸ›  tv-sim   πŸ§ͺ mac-AS-debug-wk2 βœ… πŸ§ͺ jsc-armv7-tests
βœ… πŸ›  watch βœ… πŸ§ͺ mac-wk2-stress βœ… πŸ›  jsc-mips
βœ… πŸ›  πŸ§ͺ merge   πŸ›  watch-sim βœ… πŸ§ͺ jsc-mips-tests

@nt1m nt1m requested a review from cdumez as a code owner January 6, 2023 09:41
@nt1m nt1m self-assigned this Jan 6, 2023
@nt1m nt1m added the Accessibility For bugs related to accessibility. label Jan 6, 2023
@nt1m nt1m added merge-queue Applied to send a pull request to merge-queue unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing and removed merge-queue Applied to send a pull request to merge-queue unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing labels Jan 6, 2023
@nt1m nt1m added the merge-queue Applied to send a pull request to merge-queue label Jan 6, 2023
https://bugs.webkit.org/show_bug.cgi?id=250193
rdar://103943537

Reviewed by Chris Fleizach.

The setting does nothing. Also remove SPIs which aren't used internally.

* Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml:
* Source/WebCore/page/DeprecatedGlobalSettings.h:
(WebCore::DeprecatedGlobalSettings::setAccessibilityObjectModelEnabled): Deleted.
(WebCore::DeprecatedGlobalSettings::accessibilityObjectModelEnabled): Deleted.
* Source/WebKit/UIProcess/API/C/WKPreferences.cpp:
(WKPreferencesSetAccessibilityObjectModelEnabled): Deleted.
(WKPreferencesGetAccessibilityObjectModelEnabled): Deleted.
* Source/WebKit/UIProcess/API/C/WKPreferencesRefPrivate.h:
* Source/WebKitLegacy/mac/WebView/WebPreferenceKeysPrivate.h:
* Source/WebKitLegacy/mac/WebView/WebPreferences.mm:
(-[WebPreferences accessibilityObjectModelEnabled]): Deleted.
(-[WebPreferences setAccessibilityObjectModelEnabled:]): Deleted.
* Source/WebKitLegacy/mac/WebView/WebPreferencesPrivate.h:

Canonical link: https://commits.webkit.org/258571@main
@webkit-commit-queue
Copy link
Collaborator

Committed 258571@main (94dae17): https://commits.webkit.org/258571@main

Reviewed commits have been landed. Closing PR #8295 and removing active labels.

@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Jan 7, 2023
@nt1m nt1m deleted the eng/ax-om-setting branch January 7, 2023 00:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accessibility For bugs related to accessibility.
Projects
None yet
4 participants