[webkit-changes] [WebKit/WebKit] c9c71e: REGRESSION (267943 at main): compositing and scrollin...

Alex Christensen noreply at github.com
Tue Sep 19 22:07:21 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: c9c71eb5b0451aee491cf2920e46224d456a0c5d
      https://github.com/WebKit/WebKit/commit/c9c71eb5b0451aee491cf2920e46224d456a0c5d
  Author: Alex Christensen <achristensen at apple.com>
  Date:   2023-09-19 (Tue, 19 Sep 2023)

  Changed paths:
    M Source/WebCore/page/scrolling/ScrollingStateNode.cpp
    M Source/WebKit/Shared/RemoteLayerTree/RemoteScrollingCoordinatorTransaction.cpp
    M Source/WebKit/Shared/RemoteLayerTree/RemoteScrollingCoordinatorTransaction.h
    M Source/WebKit/UIProcess/RemoteLayerTree/RemoteScrollingCoordinatorProxy.cpp
    M Source/WebKit/WebProcess/WebPage/RemoteLayerTree/RemoteScrollingCoordinator.mm

  Log Message:
  -----------
  REGRESSION (267943 at main): compositing and scrolling tests crashing
https://bugs.webkit.org/show_bug.cgi?id=261798
rdar://115746676

Unreviewed.

267943 at main had a few mistakes:
1. We used to always have a non-null ScrollingStateTree in the transaction after deserialization.
   I made that no longer the case, but this restores the non-nullness.
2. I called attachDeserializedNodes unnecessarily in the web process and the UI process.  It was
   only needed in the UI process, where we now get the value FromDeserialization::Yes.

* Source/WebCore/page/scrolling/ScrollingStateNode.cpp:
(WebCore::ScrollingStateNode::attachAfterDeserialization):
* Source/WebKit/Shared/RemoteLayerTree/RemoteScrollingCoordinatorTransaction.cpp:
(WebKit::RemoteScrollingCoordinatorTransaction::RemoteScrollingCoordinatorTransaction):
* Source/WebKit/Shared/RemoteLayerTree/RemoteScrollingCoordinatorTransaction.h:
* Source/WebKit/UIProcess/RemoteLayerTree/RemoteScrollingCoordinatorProxy.cpp:
(WebKit::RemoteScrollingCoordinatorProxy::commitScrollingTreeState):
* Source/WebKit/WebProcess/WebPage/RemoteLayerTree/RemoteScrollingCoordinator.mm:
(WebKit::RemoteScrollingCoordinator::buildTransaction):

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




More information about the webkit-changes mailing list