Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 6a9b49bfd3e696f05eb9a30b95522dc22919a64a
      
https://github.com/WebKit/WebKit/commit/6a9b49bfd3e696f05eb9a30b95522dc22919a64a
  Author: Alex Christensen <[email protected]>
  Date:   2024-01-25 (Thu, 25 Jan 2024)

  Changed paths:
    M LayoutTests/platform/mac-site-isolation/TestExpectations
    M Source/WebCore/dom/Document.cpp
    M Source/WebCore/dom/Document.h
    M Source/WebCore/page/Location.cpp
    M Source/WebCore/page/RemoteDOMWindow.cpp
    M Source/WebKit/WebProcess/WebPage/WebPage.cpp

  Log Message:
  -----------
  Fix 
html/browsers/browsing-the-web/navigating-across-documents/child-navigates-parent-cross-origin.window.html
 wpt test with site isolation on
https://bugs.webkit.org/show_bug.cgi?id=268093
rdar://121608952

Reviewed by Charlie Wolfe.

Two things were needed to get this test passing.

1. The test removed iframes from the DOM around when they had attempted 
navigations
in other processes, and then we received IPC about a frame that didn't exist 
any more.
This is fine.  I removed the assertions.  If the frame is gone there's nothing 
to do.

2. The test verified that Document::canNavigate checks were in place.  I needed 
to
make that function take a Frame instead of a LocalFrame and everything tested 
in that
wpt test passed.

* LayoutTests/platform/mac-site-isolation/TestExpectations:
* Source/WebCore/dom/Document.cpp:
(WebCore::canAccessAncestor):
(WebCore::printNavigationErrorMessage):
(WebCore::Document::canNavigate):
(WebCore::Document::canNavigateInternal):
(WebCore::Document::isNavigationBlockedByThirdPartyIFrameRedirectBlocking):
* Source/WebCore/dom/Document.h:
* Source/WebCore/page/Location.cpp:
(WebCore::Location::replace):
(WebCore::Location::setLocation):
* Source/WebCore/page/RemoteDOMWindow.cpp:
(WebCore::RemoteDOMWindow::setLocation):
* Source/WebKit/WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::didCommitLoadInAnotherProcess):
(WebKit::WebPage::didFinishLoadInAnotherProcess):

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


_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to