[webkit-changes] [WebKit/WebKit] f40a2f: [UI-side compositing] alert() causes entire page c...
Simon Fraser
noreply at github.com
Thu Jan 12 13:42:05 PST 2023
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: f40a2f336eaee267fcd56a029035b3a459e4dd8c
https://github.com/WebKit/WebKit/commit/f40a2f336eaee267fcd56a029035b3a459e4dd8c
Author: Simon Fraser <simon.fraser at apple.com>
Date: 2023-01-12 (Thu, 12 Jan 2023)
Changed paths:
M Source/WebKit/UIProcess/Cocoa/WebPageProxyCocoa.mm
M Source/WebKit/UIProcess/PageClient.h
M Source/WebKit/UIProcess/WebPageProxy.cpp
M Source/WebKit/UIProcess/WebPageProxy.h
M Source/WebKit/UIProcess/ios/WebPageProxyIOS.mm
M Source/WebKit/UIProcess/mac/PageClientImplMac.h
M Source/WebKit/UIProcess/mac/PageClientImplMac.mm
Log Message:
-----------
[UI-side compositing] alert() causes entire page contents to disappear
https://bugs.webkit.org/show_bug.cgi?id=250501
rdar://104139106
Reviewed by Chris Dumez.
When a JavaScript alert/prompt/confirm dialog is shown, `WebPageProxy::makeViewBlankIfUnpaintedSinceLastLoadCommit()`
is called (see 236143 at main). This consults `m_hasUpdatedRenderingAfterDidCommitLoad`, but that was always false
with UI-side compositing on macOS, because it relied on code in `WebPageProxy::didCommitLayerTree()` which
was iOS only.
So make `WebPageProxy::didCommitLayerTree()` shared between macOS and iOS, and add some PageClient stubs for
macOS.
* Source/WebKit/UIProcess/Cocoa/WebPageProxyCocoa.mm:
(WebKit::exceedsRenderTreeSizeSizeThreshold):
(WebKit::WebPageProxy::didCommitLayerTree):
(WebKit::WebPageProxy::layerTreeCommitComplete):
* Source/WebKit/UIProcess/PageClient.h:
* Source/WebKit/UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::didCommitLoadForFrame):
(WebKit::WebPageProxy::resetState):
* Source/WebKit/UIProcess/WebPageProxy.h:
* Source/WebKit/UIProcess/ios/WebPageProxyIOS.mm:
(WebKit::exceedsRenderTreeSizeSizeThreshold): Deleted.
(WebKit::WebPageProxy::didCommitLayerTree): Deleted.
(WebKit::WebPageProxy::layerTreeCommitComplete): Deleted.
* Source/WebKit/UIProcess/mac/PageClientImplMac.h:
* Source/WebKit/UIProcess/mac/PageClientImplMac.mm:
(WebKit::PageClientImpl::didCommitLayerTree):
(WebKit::PageClientImpl::layerTreeCommitComplete):
Canonical link: https://commits.webkit.org/258849@main
More information about the webkit-changes
mailing list