253117 2023-02-28 15:03:00 -0800 [UI-side compositing] Run keyboard scroll animations in the scrolling thread 2023-03-10 12:28:22 -0800 1 1 1 Unclassified WebKit Scrolling WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 simon.fraser simon.fraser richard_robinson2 simon.fraser webkit-bug-importer oldest_to_newest 1937568 0 simon.fraser 2023-02-28 15:03:00 -0800 We don't dispatch keyboard scrolling animations to the scrolling thread, and we should: 3 0x12f5acd6f WebKit::RemoteScrollingTreeMac::hasNodeWithAnimatedScrollChanged(bool) 4 0x1646dbee0 WebCore::ScrollingTree::setScrollAnimationInProgressForNode(unsigned long long, bool) 5 0x1646f7852 WebCore::ScrollingTreeScrollingNode::setScrollAnimationInProgress(bool) 6 0x1646fd406 WebCore::ThreadedScrollingTreeScrollingNodeDelegate::startAnimationCallback(WebCore::ScrollingEffectsController&) 7 0x16476fd98 WebCore::ScrollingEffectsController::startOrStopAnimationCallbacks() 8 0x16476fe3b WebCore::ScrollingEffectsController::setIsAnimatingKeyboardScrolling(bool) 9 0x16476fdea WebCore::ScrollingEffectsController::willBeginKeyboardScrolling() 10 0x164771da5 WebCore::ScrollingEffectsController::scrollAnimationWillStart(WebCore::ScrollAnimation&) 11 0x162597f5b WebCore::ScrollAnimation::didStart(WTF::MonotonicTime) 12 0x16474a98d WebCore::ScrollAnimationKeyboard::startKeyboardScroll(WebCore::KeyboardScroll const&) 13 0x16476fefc WebCore::ScrollingEffectsController::startKeyboardScroll(WebCore::KeyboardScroll const&) 14 0x1646fdcb3 WebCore::ThreadedScrollingTreeScrollingNodeDelegate::handleKeyboardScrollRequest(WebCore::RequestedKeyboardScrollData const&) 15 0x1646f6af8 WebCore::ScrollingTreeScrollingNode::handleKeyboardScrollRequest(WebCore::RequestedKeyboardScrollData const&) 16 0x1646f685a WebCore::ScrollingTreeScrollingNode::commitStateAfterChildren(WebCore::ScrollingStateNode const&) 17 0x1618fddb1 WebCore::ScrollingTreeFrameScrollingNodeMac::commitStateAfterChildren(WebCore::ScrollingStateNode const&) 18 0x1646da2b1 WebCore::ScrollingTree::updateTreeFromStateNodeRecursive(WebCore::ScrollingStateNode const*, WebCore::CommitTreeState&) 19 0x1646da27f WebCore::ScrollingTree::updateTreeFromStateNodeRecursive(WebCore::ScrollingStateNode const*, WebCore::CommitTreeState&) 20 0x1646da27f WebCore::ScrollingTree::updateTreeFromStateNodeRecursive(WebCore::ScrollingStateNode const*, WebCore::CommitTreeState&) 21 0x1646d9640 WebCore::ScrollingTree::commitTreeState(std::__1::unique_ptr<WebCore::ScrollingStateTree, std::__1::default_delete<WebCore::ScrollingStateTree>>&&) 22 0x12fcb8f1b WebKit::RemoteScrollingCoordinatorProxy::commitScrollingTreeState(WebKit::RemoteScrollingCoordinatorTransaction const&) 1937569 1 webkit-bug-importer 2023-02-28 15:03:31 -0800 <rdar://problem/106053716> 1937573 2 simon.fraser 2023-02-28 15:15:36 -0800 Need to do something similar to what ThreadedScrollingTree::scrollingTreeNodeRequestsScroll() does. 1939971 3 richard_robinson2 2023-03-08 22:48:31 -0800 Pull request: https://github.com/WebKit/WebKit/pull/11284 1940477 4 ews-feeder 2023-03-10 12:28:20 -0800 Committed 261518@main (b1e1c93fe134): <https://commits.webkit.org/261518@main> Reviewed commits have been landed. Closing PR #11284 and removing active labels.