Title: [293346] branches/safari-613.2.7.1-branch/Source/WebCore
- Revision
- 293346
- Author
- [email protected]
- Date
- 2022-04-25 13:52:11 -0700 (Mon, 25 Apr 2022)
Log Message
Cherry-pick r293286. rdar://problem/92198272
Crash under AsyncScrollingCoordinator::scrollingThreadAddedPendingUpdate()
https://bugs.webkit.org/show_bug.cgi?id=239683
<rdar://92198272>
Reviewed by Alan Bujtas.
Crash data suggest that m_page can be null in
AsyncScrollingCoordinator::scheduleRenderingUpdate(), which does seem possible because
scheduleRenderingUpdate() is a dispatch from the scrolling thread, and
ScrollingCoordinator::pageDestroyed() may have run before it runs.
* page/scrolling/AsyncScrollingCoordinator.cpp:
(WebCore::AsyncScrollingCoordinator::scheduleRenderingUpdate):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@293286 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Modified Paths
Diff
Modified: branches/safari-613.2.7.1-branch/Source/WebCore/ChangeLog (293345 => 293346)
--- branches/safari-613.2.7.1-branch/Source/WebCore/ChangeLog 2022-04-25 20:49:06 UTC (rev 293345)
+++ branches/safari-613.2.7.1-branch/Source/WebCore/ChangeLog 2022-04-25 20:52:11 UTC (rev 293346)
@@ -1,3 +1,40 @@
+2022-04-25 Alan Coon <[email protected]>
+
+ Cherry-pick r293286. rdar://problem/92198272
+
+ Crash under AsyncScrollingCoordinator::scrollingThreadAddedPendingUpdate()
+ https://bugs.webkit.org/show_bug.cgi?id=239683
+ <rdar://92198272>
+
+ Reviewed by Alan Bujtas.
+
+ Crash data suggest that m_page can be null in
+ AsyncScrollingCoordinator::scheduleRenderingUpdate(), which does seem possible because
+ scheduleRenderingUpdate() is a dispatch from the scrolling thread, and
+ ScrollingCoordinator::pageDestroyed() may have run before it runs.
+
+ * page/scrolling/AsyncScrollingCoordinator.cpp:
+ (WebCore::AsyncScrollingCoordinator::scheduleRenderingUpdate):
+
+
+ git-svn-id: https://svn.webkit.org/repository/webkit/trunk@293286 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+ 2022-04-22 Simon Fraser <[email protected]>
+
+ Crash under AsyncScrollingCoordinator::scrollingThreadAddedPendingUpdate()
+ https://bugs.webkit.org/show_bug.cgi?id=239683
+ <rdar://92198272>
+
+ Reviewed by Alan Bujtas.
+
+ Crash data suggest that m_page can be null in
+ AsyncScrollingCoordinator::scheduleRenderingUpdate(), which does seem possible because
+ scheduleRenderingUpdate() is a dispatch from the scrolling thread, and
+ ScrollingCoordinator::pageDestroyed() may have run before it runs.
+
+ * page/scrolling/AsyncScrollingCoordinator.cpp:
+ (WebCore::AsyncScrollingCoordinator::scheduleRenderingUpdate):
+
2022-04-22 Russell Epstein <[email protected]>
Cherry-pick r292274. rdar://problem/80544133
Modified: branches/safari-613.2.7.1-branch/Source/WebCore/page/scrolling/AsyncScrollingCoordinator.cpp (293345 => 293346)
--- branches/safari-613.2.7.1-branch/Source/WebCore/page/scrolling/AsyncScrollingCoordinator.cpp 2022-04-25 20:49:06 UTC (rev 293345)
+++ branches/safari-613.2.7.1-branch/Source/WebCore/page/scrolling/AsyncScrollingCoordinator.cpp 2022-04-25 20:52:11 UTC (rev 293346)
@@ -373,7 +373,8 @@
void AsyncScrollingCoordinator::scheduleRenderingUpdate()
{
- m_page->scheduleRenderingUpdate(RenderingUpdateStep::ScrollingTreeUpdate);
+ if (m_page)
+ m_page->scheduleRenderingUpdate(RenderingUpdateStep::ScrollingTreeUpdate);
}
FrameView* AsyncScrollingCoordinator::frameViewForScrollingNode(ScrollingNodeID scrollingNodeID) const
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
