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 Legacy Dashboard Support #12554
Remove Legacy Dashboard Support #12554
Conversation
EWS run on previous version of this PR (hash 4a59794) |
4a59794
to
9868bb9
Compare
@cdumez Do you think this is acceptable? |
EWS run on current version of this PR (hash 9868bb9) |
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.
Gtk failures look unrelated.
https://bugs.webkit.org/show_bug.cgi?id=255204 Reviewed by Alexey Proskuryakov. The Legacy Dashboard itself has not been used for many years. We can remove it without issue now, especially since nothing is dependent on the files themselves anymore. * Source/WebCore/platform/mac/PasteboardMac.mm: (WebCore::Pasteboard::setDragImage): * Source/WebKit/Configurations/WebKit.xcconfig: * Source/WebKit/MigratedHeaders-input.xcfilelist: * Source/WebKit/MigratedHeaders-output.xcfilelist: * Source/WebKit/Modules/OSX_Private.modulemap: * Source/WebKitLegacy/PlatformMac.cmake: * Source/WebKitLegacy/WebKitLegacy.xcodeproj/project.pbxproj: * Source/WebKitLegacy/mac/Configurations/WebKitLegacy.xcconfig: * Source/WebKitLegacy/mac/WebKit.order: * Source/WebKitLegacy/mac/WebView/WebDashboardRegion.h: Removed. * Source/WebKitLegacy/mac/WebView/WebDashboardRegion.mm: Removed. * Source/WebKitLegacy/mac/WebView/WebUIDelegatePrivate.h: * Source/WebKitLegacy/mac/WebView/WebView.mm: (-[WebView _addScrollerDashboardRegions:]): Deleted. (-[WebView _dashboardRegions]): Deleted. (-[WebView _setDashboardBehavior:to:]): Deleted. (-[WebView _dashboardBehavior:]): Deleted. * Source/WebKitLegacy/mac/WebView/WebViewData.h: * Source/WebKitLegacy/mac/WebView/WebViewPrivate.h: Canonical link: https://commits.webkit.org/262798@main
9868bb9
to
e432547
Compare
Committed 262798@main (e432547): https://commits.webkit.org/262798@main Reviewed commits have been landed. Closing PR #12554 and removing active labels. |
e432547
9868bb9
π§ͺ ios-wk2