[webkit-changes] [WebKit/WebKit] c9a2af: [Site Isolation][iOS] http/tests/site-isolation/dr...

[Sihui]

  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: c9a2af523d0d33a839871df19690e9bb78aa819a
      https://github.com/WebKit/WebKit/commit/c9a2af523d0d33a839871df19690e9bb78aa819a
  Author: Sihui Liu <sihui_liu at apple.com>
  Date:   2024-06-06 (Thu, 06 Jun 2024)

  Changed paths:
    M Source/WebKit/UIProcess/WebFrameProxy.cpp
    M Source/WebKit/UIProcess/WebFrameProxy.h
    M Source/WebKit/UIProcess/WebPageProxy.cpp
    M Source/WebKit/WebProcess/WebPage/WebPage.cpp

  Log Message:
  -----------
  [Site Isolation][iOS] http/tests/site-isolation/draw-after-cross-origin-navigation-between-two-remote-frames.html is failing
https://bugs.webkit.org/show_bug.cgi?id=275065
rdar://129169433

Reviewed by Alex Christensen and Brady Eidson.

In the test, the iframe is first loaded in the mainframe process and switched to a subframe process on navigation. Then
the iframe is switched to another subframe process on second navigation. In the second navigation, the frame's size is
not propogated to the new subframe process, so the iframe does not render correctly. To fix that, we make UI process
record frame size and send it to new hosting web process when frame load is commited. On web process side, it needs to
update both exposedContentRect and unobscuredContentRect of the FrameView for rendering.

* Source/WebKit/UIProcess/WebFrameProxy.cpp:
(WebKit::WebFrameProxy::commitProvisionalFrame):
* Source/WebKit/UIProcess/WebFrameProxy.h:
(WebKit::WebFrameProxy::setRemoteFrameSize):
* Source/WebKit/UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::updateRemoteFrameSize):
* Source/WebKit/WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::updateFrameSize):

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



To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications