-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Generate IPC serialization for WebCore::WheelEventTestMonitor::DeferReason #20161
Generate IPC serialization for WebCore::WheelEventTestMonitor::DeferReason #20161
Conversation
EWS run on previous version of this PR (hash 6a3a2ef) |
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.
6a3a2ef
to
dd99cca
Compare
EWS run on previous version of this PR (hash dd99cca) |
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.
That works.
dd99cca
to
e974d40
Compare
EWS run on current version of this PR (hash e974d40) |
β¦eason https://bugs.webkit.org/show_bug.cgi?id=264405 Reviewed by Chris Dumez. Move the WheelEventTestMonitor::DeferReason enum outside the WheelEventTestMonitor class and turn it into an enumerated scope, providing a type alias inside the WheelEventTestMonitor class and updating the uses of the enumeration values. This enables specifying IPC serialization for this enumeration, avoiding listing all the enumeration values in the EnumTraits specialization that can now be removed. The two messages in the RemoteScrollingCoordinator IPC interface are updated to handle these enumeration values as OptionSets, also requiring to adjust the deferForReason and removeDeferralForReason methods on the WheelEventTestMonitor class to work with OptionSet parameters. * Source/WebCore/page/WheelEventTestMonitor.cpp: (WebCore::WheelEventTestMonitor::deferForReason): (WebCore::WheelEventTestMonitor::removeDeferralForReason): (WebCore::operator<<): * Source/WebCore/page/WheelEventTestMonitor.h: (): Deleted. * Source/WebCore/page/scrolling/ThreadedScrollingCoordinator.cpp: (WebCore::ThreadedScrollingCoordinator::handleWheelEventForScrolling): * Source/WebCore/page/scrolling/ThreadedScrollingTree.cpp: (WebCore::ThreadedScrollingTree::scrollingTreeNodeDidScroll): * Source/WebCore/page/scrolling/mac/ScrollingTreeScrollingNodeDelegateMac.mm: (WebCore::ScrollingTreeScrollingNodeDelegateMac::handleWheelEvent): * Source/WebCore/page/scrolling/nicosia/ScrollingTreeScrollingNodeDelegateNicosia.cpp: (WebCore::ScrollingTreeScrollingNodeDelegateNicosia::handleWheelEvent): * Source/WebCore/platform/ScrollingEffectsController.cpp: (WebCore::ScrollingEffectsController::startScrollSnapAnimation): (WebCore::ScrollingEffectsController::stopScrollSnapAnimation): (WebCore::ScrollingEffectsController::scrollAnimationWillStart): (WebCore::ScrollingEffectsController::scrollAnimationDidEnd): * Source/WebCore/platform/mac/ScrollbarsControllerMac.mm: (WebCore::ScrollbarsControllerMac::didBeginScrollGesture): (WebCore::ScrollbarsControllerMac::didEndScrollGesture): (WebCore::ScrollbarsControllerMac::sendContentAreaScrolledTimerFired): (WebCore::ScrollbarsControllerMac::sendContentAreaScrolledSoon): * Source/WebCore/platform/mac/ScrollingEffectsController.mm: (WebCore::ScrollingEffectsController::willStartRubberBandAnimation): (WebCore::ScrollingEffectsController::didStopRubberBandAnimation): (WebCore::ScrollingEffectsController::scheduleDiscreteScrollSnap): (WebCore::ScrollingEffectsController::discreteSnapTransitionTimerFired): * Source/WebKit/Shared/WebCoreArgumentCoders.serialization.in: * Source/WebKit/WebProcess/WebPage/RemoteLayerTree/RemoteScrollingCoordinator.h: * Source/WebKit/WebProcess/WebPage/RemoteLayerTree/RemoteScrollingCoordinator.messages.in: * Source/WebKit/WebProcess/WebPage/RemoteLayerTree/RemoteScrollingCoordinator.mm: (WebKit::RemoteScrollingCoordinator::startDeferringScrollingTestCompletionForNode): (WebKit::RemoteScrollingCoordinator::stopDeferringScrollingTestCompletionForNode): Canonical link: https://commits.webkit.org/270514@main
e974d40
to
a6c2467
Compare
Committed 270514@main (a6c2467): https://commits.webkit.org/270514@main Reviewed commits have been landed. Closing PR #20161 and removing active labels. |
a6c2467
e974d40
π§ͺ wpe-wk2π§ͺ ios-wk2-wptπ§ͺ gtk-wk2π§ͺ mac-wk2