Skip to content

Crash under RenderLayerCompositor::updateSynchronousScrollingNodes()#47075

Merged
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
smfr:eng/Crash-under-RenderLayerCompositor-updateSynchronousScrollingNodes
Jun 24, 2025
Merged

Crash under RenderLayerCompositor::updateSynchronousScrollingNodes()#47075
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
smfr:eng/Crash-under-RenderLayerCompositor-updateSynchronousScrollingNodes

Conversation

@smfr
Copy link
Copy Markdown
Contributor

@smfr smfr commented Jun 23, 2025

@smfr smfr self-assigned this Jun 23, 2025
@smfr smfr added the Scrolling Bugs related to main thread and off-main thread scrolling label Jun 23, 2025
@smfr smfr added the merge-queue Applied to send a pull request to merge-queue label Jun 24, 2025
https://bugs.webkit.org/show_bug.cgi?id=294855
rdar://75139287

Reviewed by Alan Baradlay.

Crash data suggest that rootScrollingNodeID can be an unset optional here, so check it.

* Source/WebCore/rendering/RenderLayerCompositor.cpp:
(WebCore::RenderLayerCompositor::updateSynchronousScrollingNodes):

Canonical link: https://commits.webkit.org/296573@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/Crash-under-RenderLayerCompositor-updateSynchronousScrollingNodes branch from 19ad0a2 to 1e994c6 Compare June 24, 2025 16:45
@webkit-commit-queue
Copy link
Copy Markdown
Collaborator

Committed 296573@main (1e994c6): https://commits.webkit.org/296573@main

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

@webkit-commit-queue webkit-commit-queue merged commit 1e994c6 into WebKit:main Jun 24, 2025
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Jun 24, 2025
@smfr smfr deleted the eng/Crash-under-RenderLayerCompositor-updateSynchronousScrollingNodes branch October 26, 2025 16:39
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

Development

Successfully merging this pull request may close these issues.

4 participants