Diff
Modified: trunk/Source/WebCore/ChangeLog (244330 => 244331)
--- trunk/Source/WebCore/ChangeLog 2019-04-16 15:58:59 UTC (rev 244330)
+++ trunk/Source/WebCore/ChangeLog 2019-04-16 16:28:28 UTC (rev 244331)
@@ -1,3 +1,18 @@
+2019-04-16 Commit Queue <[email protected]>
+
+ Unreviewed, rolling out r244321.
+ https://bugs.webkit.org/show_bug.cgi?id=196968
+
+ Causing all WK2 Debug builds to exit early after Assertion
+ failures. (Requested by ShawnRoberts on #webkit).
+
+ Reverted changeset:
+
+ "URL set by document.open() is not communicated to the
+ UIProcess"
+ https://bugs.webkit.org/show_bug.cgi?id=196941
+ https://trac.webkit.org/changeset/244321
+
2019-04-16 Caitlin Potter <[email protected]>
[JSC] Filter DontEnum properties in ProxyObject::getOwnPropertyNames()
Modified: trunk/Source/WebCore/loader/FrameLoader.cpp (244330 => 244331)
--- trunk/Source/WebCore/loader/FrameLoader.cpp 2019-04-16 15:58:59 UTC (rev 244330)
+++ trunk/Source/WebCore/loader/FrameLoader.cpp 2019-04-16 16:28:28 UTC (rev 244331)
@@ -592,8 +592,6 @@
// Calling document.open counts as committing the first real document load.
if (!m_stateMachine.committedFirstRealDocumentLoad())
m_stateMachine.advanceTo(FrameLoaderStateMachine::DisplayingInitialEmptyDocumentPostCommit);
-
- m_client.dispatchDidExplicitOpen(m_frame.document() ? m_frame.document()->url() : URL());
// Prevent window.open(url) -- eg window.open("about:blank") -- from blowing away results
// from a subsequent window.document.open / window.document.write call.
Modified: trunk/Source/WebCore/loader/FrameLoaderClient.h (244330 => 244331)
--- trunk/Source/WebCore/loader/FrameLoaderClient.h 2019-04-16 15:58:59 UTC (rev 244330)
+++ trunk/Source/WebCore/loader/FrameLoaderClient.h 2019-04-16 16:28:28 UTC (rev 244331)
@@ -181,7 +181,6 @@
virtual void dispatchDidFailLoad(const ResourceError&) = 0;
virtual void dispatchDidFinishDocumentLoad() = 0;
virtual void dispatchDidFinishLoad() = 0;
- virtual void dispatchDidExplicitOpen(const URL&) { }
#if ENABLE(DATA_DETECTION)
virtual void dispatchDidFinishDataDetection(NSArray *detectionResults) = 0;
#endif
Modified: trunk/Source/WebKit/ChangeLog (244330 => 244331)
--- trunk/Source/WebKit/ChangeLog 2019-04-16 15:58:59 UTC (rev 244330)
+++ trunk/Source/WebKit/ChangeLog 2019-04-16 16:28:28 UTC (rev 244331)
@@ -1,3 +1,18 @@
+2019-04-16 Commit Queue <[email protected]>
+
+ Unreviewed, rolling out r244321.
+ https://bugs.webkit.org/show_bug.cgi?id=196968
+
+ Causing all WK2 Debug builds to exit early after Assertion
+ failures. (Requested by ShawnRoberts on #webkit).
+
+ Reverted changeset:
+
+ "URL set by document.open() is not communicated to the
+ UIProcess"
+ https://bugs.webkit.org/show_bug.cgi?id=196941
+ https://trac.webkit.org/changeset/244321
+
2019-04-15 Joseph Pecoraro <[email protected]>
Web Inspector: CRASH when reopening tab with docked inspector on crashed page
Modified: trunk/Source/WebKit/UIProcess/PageLoadState.cpp (244330 => 244331)
--- trunk/Source/WebKit/UIProcess/PageLoadState.cpp 2019-04-16 15:58:59 UTC (rev 244330)
+++ trunk/Source/WebKit/UIProcess/PageLoadState.cpp 2019-04-16 16:28:28 UTC (rev 244331)
@@ -246,15 +246,6 @@
m_uncommittedState.pendingAPIRequestURL = String();
}
-void PageLoadState::didExplicitOpen(const Transaction::Token& token, const String& url)
-{
- ASSERT_UNUSED(token, &token.m_pageLoadState == this);
-
- m_uncommittedState.state = State::Finished;
- m_uncommittedState.url = ""
- m_uncommittedState.provisionalURL = String();
-}
-
void PageLoadState::didStartProvisionalLoad(const Transaction::Token& token, const String& url, const String& unreachableURL)
{
ASSERT_UNUSED(token, &token.m_pageLoadState == this);
Modified: trunk/Source/WebKit/UIProcess/PageLoadState.h (244330 => 244331)
--- trunk/Source/WebKit/UIProcess/PageLoadState.h 2019-04-16 15:58:59 UTC (rev 244330)
+++ trunk/Source/WebKit/UIProcess/PageLoadState.h 2019-04-16 16:28:28 UTC (rev 244331)
@@ -143,7 +143,6 @@
void clearPendingAPIRequestURL(const Transaction::Token&);
void didStartProvisionalLoad(const Transaction::Token&, const String& url, const String& unreachableURL);
- void didExplicitOpen(const Transaction::Token&, const String& url);
void didReceiveServerRedirectForProvisionalLoad(const Transaction::Token&, const String& url);
void didFailProvisionalLoad(const Transaction::Token&);
Modified: trunk/Source/WebKit/UIProcess/WebPageProxy.cpp (244330 => 244331)
--- trunk/Source/WebKit/UIProcess/WebPageProxy.cpp 2019-04-16 15:58:59 UTC (rev 244330)
+++ trunk/Source/WebKit/UIProcess/WebPageProxy.cpp 2019-04-16 16:28:28 UTC (rev 244331)
@@ -3922,23 +3922,6 @@
m_navigationClient->didStartProvisionalNavigation(*this, navigation.get(), process->transformHandlesToObjects(userData.object()).get());
}
-void WebPageProxy::didExplicitOpenForFrame(uint64_t frameID, URL&& url)
-{
- auto* frame = m_process->webFrame(frameID);
- MESSAGE_CHECK(m_process, frame);
- MESSAGE_CHECK_URL(m_process, url);
-
- auto transaction = m_pageLoadState.transaction();
-
- if (frame->isMainFrame())
- m_pageLoadState.didExplicitOpen(transaction, url);
-
- m_hasCommittedAnyProvisionalLoads = true;
- m_process->didCommitProvisionalLoad();
-
- m_pageLoadState.commitChanges();
-}
-
void WebPageProxy::didReceiveServerRedirectForProvisionalLoadForFrame(uint64_t frameID, uint64_t navigationID, ResourceRequest&& request, const UserData& userData)
{
didReceiveServerRedirectForProvisionalLoadForFrameShared(m_process.copyRef(), frameID, navigationID, WTFMove(request), userData);
Modified: trunk/Source/WebKit/UIProcess/WebPageProxy.h (244330 => 244331)
--- trunk/Source/WebKit/UIProcess/WebPageProxy.h 2019-04-16 15:58:59 UTC (rev 244330)
+++ trunk/Source/WebKit/UIProcess/WebPageProxy.h 2019-04-16 16:28:28 UTC (rev 244331)
@@ -1591,7 +1591,6 @@
void didFailLoadForFrame(uint64_t frameID, uint64_t navigationID, const WebCore::ResourceError&, const UserData&);
void didSameDocumentNavigationForFrame(uint64_t frameID, uint64_t navigationID, uint32_t sameDocumentNavigationType, URL&&, const UserData&);
void didChangeMainDocument(uint64_t frameID);
- void didExplicitOpenForFrame(uint64_t frameID, URL&&);
void didReceiveTitleForFrame(uint64_t frameID, const String&, const UserData&);
void didFirstLayoutForFrame(uint64_t frameID, const UserData&);
Modified: trunk/Source/WebKit/UIProcess/WebPageProxy.messages.in (244330 => 244331)
--- trunk/Source/WebKit/UIProcess/WebPageProxy.messages.in 2019-04-16 15:58:59 UTC (rev 244330)
+++ trunk/Source/WebKit/UIProcess/WebPageProxy.messages.in 2019-04-16 16:28:28 UTC (rev 244331)
@@ -140,8 +140,7 @@
DidRunInsecureContentForFrame(uint64_t frameID, WebKit::UserData userData)
DidDetectXSSForFrame(uint64_t frameID, WebKit::UserData userData)
DidSameDocumentNavigationForFrame(uint64_t frameID, uint64_t navigationID, uint32_t type, URL url, WebKit::UserData userData)
- DidChangeMainDocument(uint64_t frameID)
- DidExplicitOpenForFrame(uint64_t frameID, URL url)
+ DidChangeMainDocument(uint64_t frameID);
DidDestroyNavigation(uint64_t navigationID)
HasInsecureContent() -> (enum:bool WebCore::HasInsecureContent hasInsecureContent) Synchronous
Modified: trunk/Source/WebKit/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp (244330 => 244331)
--- trunk/Source/WebKit/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp 2019-04-16 15:58:59 UTC (rev 244330)
+++ trunk/Source/WebKit/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp 2019-04-16 16:28:28 UTC (rev 244331)
@@ -458,16 +458,6 @@
notImplemented();
}
-void WebFrameLoaderClient::dispatchDidExplicitOpen(const URL& url)
-{
- auto* webPage = m_frame->page();
- if (!webPage)
- return;
-
- // Notify the UIProcess.
- webPage->send(Messages::WebPageProxy::DidExplicitOpenForFrame(m_frame->frameID(), url));
-}
-
void WebFrameLoaderClient::dispatchDidStartProvisionalLoad()
{
WebPage* webPage = m_frame->page();
Modified: trunk/Source/WebKit/WebProcess/WebCoreSupport/WebFrameLoaderClient.h (244330 => 244331)
--- trunk/Source/WebKit/WebProcess/WebCoreSupport/WebFrameLoaderClient.h 2019-04-16 15:58:59 UTC (rev 244330)
+++ trunk/Source/WebKit/WebProcess/WebCoreSupport/WebFrameLoaderClient.h 2019-04-16 16:28:28 UTC (rev 244331)
@@ -116,7 +116,6 @@
void dispatchDidFailLoad(const WebCore::ResourceError&) final;
void dispatchDidFinishDocumentLoad() final;
void dispatchDidFinishLoad() final;
- void dispatchDidExplicitOpen(const URL&) final;
void dispatchDidReachLayoutMilestone(OptionSet<WebCore::LayoutMilestone>) final;
void dispatchDidLayout() final;
Modified: trunk/Tools/ChangeLog (244330 => 244331)
--- trunk/Tools/ChangeLog 2019-04-16 15:58:59 UTC (rev 244330)
+++ trunk/Tools/ChangeLog 2019-04-16 16:28:28 UTC (rev 244331)
@@ -1,3 +1,18 @@
+2019-04-16 Commit Queue <[email protected]>
+
+ Unreviewed, rolling out r244321.
+ https://bugs.webkit.org/show_bug.cgi?id=196968
+
+ Causing all WK2 Debug builds to exit early after Assertion
+ failures. (Requested by ShawnRoberts on #webkit).
+
+ Reverted changeset:
+
+ "URL set by document.open() is not communicated to the
+ UIProcess"
+ https://bugs.webkit.org/show_bug.cgi?id=196941
+ https://trac.webkit.org/changeset/244321
+
2019-04-15 Aakash Jain <[email protected]>
Regression (r244291): Broken API Test AutoLayoutRenderingProgressRelativeOrdering
Modified: trunk/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj (244330 => 244331)
--- trunk/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj 2019-04-16 15:58:59 UTC (rev 244330)
+++ trunk/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj 2019-04-16 16:28:28 UTC (rev 244331)
@@ -188,7 +188,6 @@
466C3843210637DE006A88DE /* notify-resourceLoadObserver.html in Copy Resources */ = {isa = PBXBuildFile; fileRef = 466C3842210637CE006A88DE /* notify-resourceLoadObserver.html */; };
467C565321B5ED130057516D /* GetSessionCookie.html in Copy Resources */ = {isa = PBXBuildFile; fileRef = 467C565121B5ECDF0057516D /* GetSessionCookie.html */; };
467C565421B5ED130057516D /* SetSessionCookie.html in Copy Resources */ = {isa = PBXBuildFile; fileRef = 467C565221B5ECDF0057516D /* SetSessionCookie.html */; };
- 468BC45522653A1000A36C96 /* open-window-then-write-to-it.html in Copy Resources */ = {isa = PBXBuildFile; fileRef = 468BC454226539C800A36C96 /* open-window-then-write-to-it.html */; };
46918EFC2237283C00468DFE /* DeviceOrientation.mm in Sources */ = {isa = PBXBuildFile; fileRef = 46918EFB2237283500468DFE /* DeviceOrientation.mm */; };
46A911592108E6780078D40D /* CustomUserAgent.mm in Sources */ = {isa = PBXBuildFile; fileRef = 46A911582108E66B0078D40D /* CustomUserAgent.mm */; };
46AE5A3720F9066D00E0873E /* SimpleServiceWorkerRegistrations-4.sqlite3 in Copy Resources */ = {isa = PBXBuildFile; fileRef = 4656A75720F9054F0002E21F /* SimpleServiceWorkerRegistrations-4.sqlite3 */; };
@@ -1244,7 +1243,6 @@
074994421EA5034B000DA44F /* ondevicechange.html in Copy Resources */,
CEA6CF2819CCF69D0064F5A7 /* open-and-close-window.html in Copy Resources */,
7CCB99231D3B4A46003922F6 /* open-multiple-external-url.html in Copy Resources */,
- 468BC45522653A1000A36C96 /* open-window-then-write-to-it.html in Copy Resources */,
290A9BB91735F63800D71BBC /* OpenNewWindow.html in Copy Resources */,
83148B09202AC78D00BADE99 /* override-builtins-test.html in Copy Resources */,
CEBCA1391E3A807A00C73293 /* page-with-csp-iframe.html in Copy Resources */,
@@ -1578,7 +1576,6 @@
466C3842210637CE006A88DE /* notify-resourceLoadObserver.html */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text.html; path = "notify-resourceLoadObserver.html"; sourceTree = "<group>"; };
467C565121B5ECDF0057516D /* GetSessionCookie.html */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text.html; path = GetSessionCookie.html; sourceTree = "<group>"; };
467C565221B5ECDF0057516D /* SetSessionCookie.html */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text.html; path = SetSessionCookie.html; sourceTree = "<group>"; };
- 468BC454226539C800A36C96 /* open-window-then-write-to-it.html */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text.html; path = "open-window-then-write-to-it.html"; sourceTree = "<group>"; };
46918EFB2237283500468DFE /* DeviceOrientation.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = DeviceOrientation.mm; sourceTree = "<group>"; };
46A911582108E66B0078D40D /* CustomUserAgent.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = CustomUserAgent.mm; sourceTree = "<group>"; };
46C519D81D355A7300DAA51A /* LocalStorageNullEntries.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = LocalStorageNullEntries.mm; sourceTree = "<group>"; };
@@ -3421,7 +3418,6 @@
C99B675E1E39735C00FC6C80 /* no-autoplay-with-controls.html */,
4A410F4D19AF7BEF002EBAB6 /* ondevicechange.html */,
CEA6CF2719CCF69D0064F5A7 /* open-and-close-window.html */,
- 468BC454226539C800A36C96 /* open-window-then-write-to-it.html */,
83148B08202AC76800BADE99 /* override-builtins-test.html */,
0EBBCC651FFF9DCE00FA42AB /* pop-up-check.html */,
F6FDDDD514241C48004F1729 /* push-state.html */,
Deleted: trunk/Tools/TestWebKitAPI/Tests/WebKit/open-window-then-write-to-it.html (244330 => 244331)
--- trunk/Tools/TestWebKitAPI/Tests/WebKit/open-window-then-write-to-it.html 2019-04-16 15:58:59 UTC (rev 244330)
+++ trunk/Tools/TestWebKitAPI/Tests/WebKit/open-window-then-write-to-it.html 2019-04-16 16:28:28 UTC (rev 244331)
@@ -1,13 +0,0 @@
-<!DOCTYPE html>
-<html>
-<body>
-<script>
-_onload_ = () => {
- w = window.open("#", "name", "width=400,height=400")
- w.document.open();
- w.document.write("TEST");
- w.document.close();
-};
-</script>
-</body>
-</html>
Modified: trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/OpenAndCloseWindow.mm (244330 => 244331)
--- trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/OpenAndCloseWindow.mm 2019-04-16 15:58:59 UTC (rev 244330)
+++ trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/OpenAndCloseWindow.mm 2019-04-16 16:28:28 UTC (rev 244331)
@@ -282,38 +282,3 @@
EXPECT_FALSE([openWindowFeatures _fullscreenDisplay].boolValue);
openWindowFeatures = nullptr;
}
-
-@interface OpenWindowThenDocumentOpenUIDelegate : NSObject <WKUIDelegate>
-@end
-
-@implementation OpenWindowThenDocumentOpenUIDelegate
-
-- (WKWebView *)webView:(WKWebView *)webView createWebViewWithConfiguration:(WKWebViewConfiguration *)configuration forNavigationAction:(WKNavigationAction *)navigationAction windowFeatures:(WKWindowFeatures *)windowFeatures
-{
- openedWebView = adoptNS([[WKWebView alloc] initWithFrame:NSMakeRect(0, 0, 800, 600) configuration:configuration]);
- [openedWebView setUIDelegate:sharedUIDelegate.get()];
- return openedWebView.get();
-}
-
-@end
-
-TEST(WebKit, OpenWindowThenDocumentOpen)
-{
- resetToConsistentState();
-
- auto webView = adoptNS([[WKWebView alloc] initWithFrame:NSMakeRect(0, 0, 800, 600)]);
-
- auto uiDelegate = adoptNS([[OpenWindowThenDocumentOpenUIDelegate alloc] init]);
- [webView setUIDelegate:uiDelegate.get()];
- [webView configuration].preferences._javascript_CanOpenWindowsAutomatically = YES;
-
- NSURLRequest *request = [NSURLRequest requestWithURL:[[NSBundle mainBundle] URLForResource:@"open-window-then-write-to-it" withExtension:@"html" subdirectory:@"TestWebKitAPI.resources"]];
- [webView loadRequest:request];
-
- while (!openedWebView)
- TestWebKitAPI::Util::sleep(0.1);
-
- // Both WebViews should have the same URL because of document.open().
- while (![[[openedWebView URL] absoluteString] isEqualToString:[[webView URL] absoluteString]])
- TestWebKitAPI::Util::sleep(0.1);
-}
