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

Remove CSSCustomPropertiesAndValuesEnabled preference #26018

Conversation

annevk
Copy link
Contributor

@annevk annevk commented Mar 17, 2024

014919f

Remove CSSCustomPropertiesAndValuesEnabled preference
https://bugs.webkit.org/show_bug.cgi?id=271126

Reviewed by Tim Nguyen.

It's been stable over a year.

* LayoutTests/css-custom-properties-api/crash.html:
* LayoutTests/css-custom-properties-api/cycles.html:
* LayoutTests/css-custom-properties-api/inherits.html:
* LayoutTests/css-custom-properties-api/initialValue.html:
* LayoutTests/css-custom-properties-api/initialValueJS.html:
* LayoutTests/css-custom-properties-api/inline.html:
* LayoutTests/css-custom-properties-api/length.html:
* LayoutTests/css-custom-properties-api/length2.html:
* LayoutTests/css-custom-properties-api/registerProperty.html:
* LayoutTests/fast/css-custom-paint/properties.html:
* Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml:
* Source/WebCore/css/CSSProperties.json:
* Source/WebCore/css/DOMCSSCustomPropertyDescriptor.idl:
* Source/WebCore/css/DOMCSSNamespace+CSSPropertiesandValues.idl:
* Source/WebCore/css/parser/CSSParserImpl.cpp:
(WebCore::CSSParserImpl::consumePropertyRule):
* Source/WebKitLegacy/mac/WebView/WebPreferenceKeysPrivate.h:
* Source/WebKitLegacy/mac/WebView/WebPreferences.mm:
(-[WebPreferences CSSCustomPropertiesAndValuesEnabled]):
(-[WebPreferences setCSSCustomPropertiesAndValuesEnabled:]):
* Source/WebKitLegacy/mac/WebView/WebPreferencesPrivate.h:

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

bc1b014

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

@annevk annevk self-assigned this Mar 17, 2024
@annevk annevk added the WebCore Misc. For miscellaneous bugs in the WebCore framework (and not JavaScriptCore or WebKit). label Mar 17, 2024
LayoutTests/css-custom-properties-api/crash.html Outdated Show resolved Hide resolved
@annevk annevk force-pushed the eng/Remove-CSSCustomPropertiesAndValuesEnabled-preference branch from a45f750 to bc1b014 Compare March 18, 2024 08:16
@annevk annevk added the merge-queue Applied to send a pull request to merge-queue label Mar 18, 2024
https://bugs.webkit.org/show_bug.cgi?id=271126

Reviewed by Tim Nguyen.

It's been stable over a year.

* LayoutTests/css-custom-properties-api/crash.html:
* LayoutTests/css-custom-properties-api/cycles.html:
* LayoutTests/css-custom-properties-api/inherits.html:
* LayoutTests/css-custom-properties-api/initialValue.html:
* LayoutTests/css-custom-properties-api/initialValueJS.html:
* LayoutTests/css-custom-properties-api/inline.html:
* LayoutTests/css-custom-properties-api/length.html:
* LayoutTests/css-custom-properties-api/length2.html:
* LayoutTests/css-custom-properties-api/registerProperty.html:
* LayoutTests/fast/css-custom-paint/properties.html:
* Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml:
* Source/WebCore/css/CSSProperties.json:
* Source/WebCore/css/DOMCSSCustomPropertyDescriptor.idl:
* Source/WebCore/css/DOMCSSNamespace+CSSPropertiesandValues.idl:
* Source/WebCore/css/parser/CSSParserImpl.cpp:
(WebCore::CSSParserImpl::consumePropertyRule):
* Source/WebKitLegacy/mac/WebView/WebPreferenceKeysPrivate.h:
* Source/WebKitLegacy/mac/WebView/WebPreferences.mm:
(-[WebPreferences CSSCustomPropertiesAndValuesEnabled]):
(-[WebPreferences setCSSCustomPropertiesAndValuesEnabled:]):
* Source/WebKitLegacy/mac/WebView/WebPreferencesPrivate.h:

Canonical link: https://commits.webkit.org/276260@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/Remove-CSSCustomPropertiesAndValuesEnabled-preference branch from bc1b014 to 014919f Compare March 18, 2024 09:37
@webkit-commit-queue
Copy link
Collaborator

Committed 276260@main (014919f): https://commits.webkit.org/276260@main

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

@webkit-commit-queue webkit-commit-queue merged commit 014919f into WebKit:main Mar 18, 2024
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Mar 18, 2024
@annevk annevk deleted the eng/Remove-CSSCustomPropertiesAndValuesEnabled-preference branch March 18, 2024 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WebCore Misc. For miscellaneous bugs in the WebCore framework (and not JavaScriptCore or WebKit).
Projects
None yet
4 participants