Commit the scrolling tree from the main thread
Created attachment 397572 [details] Patch
<rdar://problem/62374855>
Created attachment 397573 [details] Patch
Comment on attachment 397573 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=397573&action=review > Source/WebCore/page/scrolling/ScrollingCoordinator.h:71 > +using PlatformDisplayID = uint32_t; I think this name would work just as well without the word "platform" in it. > Source/WebCore/page/scrolling/ScrollingTree.h:57 > +#if ENABLE(SCROLLING_THREAD) > +using PlatformDisplayID = uint32_t; > +#endif Same comment as above that I don’t think the word "platform" is helpful in this name. Is there some way to avoid defining this type separately in multiple headers? Please add a blank line before this since it doesn’t fit great into the paragraph of forward-declared classes. > Source/WebCore/page/scrolling/mac/ScrollingCoordinatorMac.mm:108 > + auto stateTree = scrollingStateTree()->commit(LayerRepresentation::PlatformLayerRepresentation); > + scrollingTree()->commitTreeState(WTFMove(stateTree)); Would this read better as a single line? > Source/WebCore/page/scrolling/nicosia/ScrollingCoordinatorNicosia.cpp:98 > + auto stateTree = scrollingStateTree()->commit(LayerRepresentation::PlatformLayerRepresentation); > + scrollingTree()->commitTreeState(WTFMove(stateTree)); Would this read better as a single line?
Oops, I didn't mean to include the windowScreenDidChange changes in this page (webkit-patch upload strikes again)
Created attachment 397578 [details] Patch
Created attachment 397587 [details] Patch
Created attachment 397591 [details] Patch
Committed r260716: <https://trac.webkit.org/changeset/260716> All reviewed patches have been landed. Closing bug and clearing flags on attachment 397591 [details].