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 the CSSIndividualTransformPropertiesEnabled flag #20900
Remove the CSSIndividualTransformPropertiesEnabled flag #20900
Conversation
EWS run on previous version of this PR (hash cee9da2) |
cee9da2
to
52401b6
Compare
EWS run on previous version of this PR (hash 52401b6) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Non-reviewer looks good modulo binary compatibility.
- (BOOL)CSSIndividualTransformPropertiesEnabled | ||
{ | ||
return [self _boolValueForKey:WebKitCSSIndividualTransformPropertiesEnabledPreferenceKey]; | ||
} | ||
|
||
- (void)setCSSIndividualTransformPropertiesEnabled:(BOOL)flag | ||
{ | ||
[self _setBoolValue:flag forKey:WebKitCSSIndividualTransformPropertiesEnabledPreferenceKey]; | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You have to noop these and move them to the end of the file. Unless you want to deal with binary compatibility fallout as I understand it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See #20896 as an example of how bincompat can be preserved.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess you could do a codesearch to see if anyone used them. I doubt it in this case, but you never know.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we have a policy for the removal of such private preferences provided we don't find any internal clients (as is the case here)?
52401b6
to
33ace21
Compare
https://bugs.webkit.org/show_bug.cgi?id=217899 rdar://70434096 Reviewed by Dean Jackson. This flag has been on by default for years and is ripe for removal. * LayoutTests/fast/css/rotate-invalidate-if-disabled-expected.txt: Removed. * LayoutTests/fast/css/rotate-invalidate-if-disabled.html: Removed. * LayoutTests/fast/css/scale-invalidate-if-disabled-expected.txt: Removed. * LayoutTests/fast/css/scale-invalidate-if-disabled.html: Removed. * LayoutTests/fast/css/translate-invalidate-if-disabled-expected.txt: Removed. * LayoutTests/fast/css/translate-invalidate-if-disabled.html: Removed. * Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml: * Source/WebCore/css/CSSProperties.json: * Source/WebCore/css/parser/CSSParserContext.cpp: (WebCore::CSSParserContext::CSSParserContext): * Source/WebKitLegacy/mac/WebView/WebPreferenceKeysPrivate.h: * Source/WebKitLegacy/mac/WebView/WebPreferences.mm: (-[WebPreferences CSSIndividualTransformPropertiesEnabled]): Deleted. (-[WebPreferences setCSSIndividualTransformPropertiesEnabled:]): Deleted. Canonical link: https://commits.webkit.org/271133@main
33ace21
to
3635c66
Compare
Committed 271133@main (3635c66): https://commits.webkit.org/271133@main Reviewed commits have been landed. Closing PR #20900 and removing active labels. |
3635c66
33ace21
π§ͺ wpe-wk2π§ͺ ios-wk2π§ͺ api-macπ§ͺ api-wpeπ§ͺ ios-wk2-wptπ§ͺ mac-wk1π§ͺ api-iosπ§ͺ mac-wk2π§ͺ gtk-wk2π§ͺ mac-AS-debug-wk2π§ͺ api-gtkπ§ͺ jsc-armv7-tests