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

[UI-side compositing] Assertion under RemoteLayerTreeEventDispatcher::displayLink() running tests #10824

Conversation

smfr
Copy link
Contributor

@smfr smfr commented Feb 28, 2023

f249228

[UI-side compositing] Assertion under RemoteLayerTreeEventDispatcher::displayLink() running tests
https://bugs.webkit.org/show_bug.cgi?id=253112
rdar://106051454

Reviewed by Tim Horton.

RemoteScrollingCoordinatorProxy owns a RemoteLayerTreeEventDispatcher which needs to get back to
the DrawingAreaProxy to cancel a DisplayLink on cleanup. This means that the RemoteScrollingCoordinatorProxy
needs to be destroyed before the DrawingAreaProxy, so ensure that in WebPageProxy.

* Source/WebKit/UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::setDrawingArea):
(WebKit::WebPageProxy::resetState): Call setDrawingArea(nullptr) so we run the same cleanup code.

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

d8ac764

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

@smfr smfr requested a review from cdumez as a code owner February 28, 2023 22:26
@smfr smfr self-assigned this Feb 28, 2023
@smfr smfr added the Scrolling Bugs related to main thread and off-main thread scrolling label Feb 28, 2023
@smfr smfr added the merge-queue Applied to send a pull request to merge-queue label Feb 28, 2023
…:displayLink() running tests

https://bugs.webkit.org/show_bug.cgi?id=253112
rdar://106051454

Reviewed by Tim Horton.

RemoteScrollingCoordinatorProxy owns a RemoteLayerTreeEventDispatcher which needs to get back to
the DrawingAreaProxy to cancel a DisplayLink on cleanup. This means that the RemoteScrollingCoordinatorProxy
needs to be destroyed before the DrawingAreaProxy, so ensure that in WebPageProxy.

* Source/WebKit/UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::setDrawingArea):
(WebKit::WebPageProxy::resetState): Call setDrawingArea(nullptr) so we run the same cleanup code.

Canonical link: https://commits.webkit.org/260975@main
@webkit-early-warning-system webkit-early-warning-system force-pushed the eng/UI-side-compositing-Assertion-under-RemoteLayerTreeEventDispatcherdisplayLink-running-tests branch from d8ac764 to f249228 Compare March 1, 2023 00:21
@webkit-commit-queue
Copy link
Collaborator

Committed 260975@main (f249228): https://commits.webkit.org/260975@main

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

@webkit-early-warning-system webkit-early-warning-system merged commit f249228 into WebKit:main Mar 1, 2023
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Mar 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scrolling Bugs related to main thread and off-main thread scrolling
Projects
None yet
4 participants