Diff
Modified: trunk/Source/WebCore/ChangeLog (239218 => 239219)
--- trunk/Source/WebCore/ChangeLog 2018-12-14 18:14:32 UTC (rev 239218)
+++ trunk/Source/WebCore/ChangeLog 2018-12-14 18:23:55 UTC (rev 239219)
@@ -1,3 +1,19 @@
+2018-12-14 Chris Dumez <cdu...@apple.com>
+
+ [PSON] Stop exposing PolicyAction::Suspend to WebCore
+ https://bugs.webkit.org/show_bug.cgi?id=192701
+
+ Reviewed by Brady Eidson.
+
+ Drop PolicyAction::Suspend enum value and stop dealing with it in WebCore.
+
+ * loader/DocumentLoader.cpp:
+ (WebCore::DocumentLoader::continueAfterContentPolicy):
+ * loader/FrameLoaderTypes.h:
+ * loader/PolicyChecker.cpp:
+ (WebCore::PolicyChecker::checkNavigationPolicy):
+ (WebCore::PolicyChecker::checkNewWindowPolicy):
+
2018-12-14 Youenn Fablet <you...@apple.com>
IDB should store RTCCertificate
Modified: trunk/Source/WebCore/loader/DocumentLoader.cpp (239218 => 239219)
--- trunk/Source/WebCore/loader/DocumentLoader.cpp 2018-12-14 18:14:32 UTC (rev 239218)
+++ trunk/Source/WebCore/loader/DocumentLoader.cpp 2018-12-14 18:23:55 UTC (rev 239219)
@@ -918,9 +918,6 @@
static_cast<ResourceLoader*>(mainResourceLoader())->didFail(interruptedForPolicyChangeError());
return;
}
- case PolicyAction::Suspend:
- // It is invalid to get a Suspend policy based on navigation response.
- RELEASE_ASSERT_NOT_REACHED();
case PolicyAction::Ignore:
if (ResourceLoader* mainResourceLoader = this->mainResourceLoader())
InspectorInstrumentation::continueWithPolicyIgnore(*m_frame, mainResourceLoader->identifier(), *this, m_response);
Modified: trunk/Source/WebCore/loader/FrameLoaderTypes.h (239218 => 239219)
--- trunk/Source/WebCore/loader/FrameLoaderTypes.h 2018-12-14 18:14:32 UTC (rev 239218)
+++ trunk/Source/WebCore/loader/FrameLoaderTypes.h 2018-12-14 18:23:55 UTC (rev 239219)
@@ -44,7 +44,6 @@
Use,
Download,
Ignore,
- Suspend, // FIXME: This is only used by WebKit2 so we shouldn't need this in the WebCore enum.
};
enum class ReloadOption : uint8_t {
@@ -153,8 +152,7 @@
WebCore::PolicyAction,
WebCore::PolicyAction::Use,
WebCore::PolicyAction::Download,
- WebCore::PolicyAction::Ignore,
- WebCore::PolicyAction::Suspend
+ WebCore::PolicyAction::Ignore
>;
};
Modified: trunk/Source/WebCore/loader/PolicyChecker.cpp (239218 => 239219)
--- trunk/Source/WebCore/loader/PolicyChecker.cpp 2018-12-14 18:14:32 UTC (rev 239218)
+++ trunk/Source/WebCore/loader/PolicyChecker.cpp 2018-12-14 18:23:55 UTC (rev 239219)
@@ -182,8 +182,6 @@
FALLTHROUGH;
case PolicyAction::Ignore:
return function({ }, nullptr, ShouldContinue::No);
- case PolicyAction::Suspend:
- RELEASE_ASSERT_NOT_REACHED();
case PolicyAction::Use:
if (!m_frame.loader().client().canHandleRequest(request)) {
handleUnimplementablePolicy(m_frame.loader().client().cannotShowURLError(request));
@@ -213,9 +211,6 @@
case PolicyAction::Ignore:
function({ }, nullptr, { }, { }, ShouldContinue::No);
return;
- case PolicyAction::Suspend:
- // It is invalid to get a "Suspend" policy for new windows, as the old document is not going away.
- RELEASE_ASSERT_NOT_REACHED();
case PolicyAction::Use:
function(request, makeWeakPtr(formState.get()), frameName, navigationAction, ShouldContinue::Yes);
return;
Modified: trunk/Source/WebKit/ChangeLog (239218 => 239219)
--- trunk/Source/WebKit/ChangeLog 2018-12-14 18:14:32 UTC (rev 239218)
+++ trunk/Source/WebKit/ChangeLog 2018-12-14 18:23:55 UTC (rev 239219)
@@ -1,3 +1,49 @@
+2018-12-14 Chris Dumez <cdu...@apple.com>
+
+ [PSON] Stop exposing PolicyAction::Suspend to WebCore
+ https://bugs.webkit.org/show_bug.cgi?id=192701
+
+ Reviewed by Brady Eidson.
+
+ Introduce a new WebPolicyAction enum that is used at WebKit2 layer and augments
+ WebCore::PolicyAction with a "Suspend" value.
+
+ * NetworkProcess/NetworkDataTaskBlob.cpp:
+ (WebKit::NetworkDataTaskBlob::dispatchDidReceiveResponse):
+ * NetworkProcess/cocoa/NetworkSessionCocoa.mm:
+ (toNSURLSessionResponseDisposition):
+ * Shared/WebPolicyAction.h: Added.
+ * UIProcess/WebFramePolicyListenerProxy.cpp:
+ (WebKit::WebFramePolicyListenerProxy::didReceiveSafeBrowsingResults):
+ (WebKit::WebFramePolicyListenerProxy::use):
+ (WebKit::WebFramePolicyListenerProxy::download):
+ (WebKit::WebFramePolicyListenerProxy::ignore):
+ * UIProcess/WebFramePolicyListenerProxy.h:
+ * UIProcess/WebFrameProxy.cpp:
+ (WebKit::WebFrameProxy::setUpPolicyListenerProxy):
+ * UIProcess/WebFrameProxy.h:
+ * UIProcess/WebPageProxy.cpp:
+ (WebKit::WebPageProxy::receivedNavigationPolicyDecision):
+ (WebKit::WebPageProxy::receivedPolicyDecision):
+ (WebKit::WebPageProxy::decidePolicyForNavigationAction):
+ (WebKit::WebPageProxy::decidePolicyForNavigationActionSync):
+ (WebKit::WebPageProxy::decidePolicyForNewWindowAction):
+ (WebKit::WebPageProxy::decidePolicyForResponse):
+ * UIProcess/WebPageProxy.h:
+ * UIProcess/WebPageProxy.messages.in:
+ * WebKit.xcodeproj/project.pbxproj:
+ * WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
+ (WebKit::WebFrameLoaderClient::dispatchDecidePolicyForResponse):
+ (WebKit::WebFrameLoaderClient::dispatchDecidePolicyForNavigationAction):
+ * WebProcess/WebPage/WebFrame.cpp:
+ (WebKit::toPolicyAction):
+ (WebKit::WebFrame::didReceivePolicyDecision):
+ * WebProcess/WebPage/WebFrame.h:
+ * WebProcess/WebPage/WebPage.cpp:
+ (WebKit::WebPage::didReceivePolicyDecision):
+ * WebProcess/WebPage/WebPage.h:
+ * WebProcess/WebPage/WebPage.messages.in:
+
2018-12-14 David Quesada <david_ques...@apple.com>
Remove a global 'using namespace WebKit' in WebViewImpl.mm
Modified: trunk/Source/WebKit/NetworkProcess/NetworkDataTaskBlob.cpp (239218 => 239219)
--- trunk/Source/WebKit/NetworkProcess/NetworkDataTaskBlob.cpp 2018-12-14 18:14:32 UTC (rev 239218)
+++ trunk/Source/WebKit/NetworkProcess/NetworkDataTaskBlob.cpp 2018-12-14 18:23:55 UTC (rev 239219)
@@ -311,9 +311,6 @@
m_buffer.resize(bufferSize);
read();
break;
- case PolicyAction::Suspend:
- LOG_ERROR("PolicyAction::Suspend encountered - Treating as PolicyAction::Ignore for now");
- FALLTHROUGH;
case PolicyAction::Ignore:
break;
case PolicyAction::Download:
Modified: trunk/Source/WebKit/NetworkProcess/cocoa/NetworkSessionCocoa.mm (239218 => 239219)
--- trunk/Source/WebKit/NetworkProcess/cocoa/NetworkSessionCocoa.mm 2018-12-14 18:14:32 UTC (rev 239218)
+++ trunk/Source/WebKit/NetworkProcess/cocoa/NetworkSessionCocoa.mm 2018-12-14 18:23:55 UTC (rev 239219)
@@ -58,9 +58,6 @@
static NSURLSessionResponseDisposition toNSURLSessionResponseDisposition(WebCore::PolicyAction disposition)
{
switch (disposition) {
- case WebCore::PolicyAction::Suspend:
- LOG_ERROR("PolicyAction::Suspend encountered - Treating as PolicyAction::Ignore for now");
- FALLTHROUGH;
case WebCore::PolicyAction::Ignore:
return NSURLSessionResponseCancel;
case WebCore::PolicyAction::Use:
Added: trunk/Source/WebKit/Shared/WebPolicyAction.h (0 => 239219)
--- trunk/Source/WebKit/Shared/WebPolicyAction.h (rev 0)
+++ trunk/Source/WebKit/Shared/WebPolicyAction.h 2018-12-14 18:23:55 UTC (rev 239219)
@@ -0,0 +1,54 @@
+/*
+ * Copyright (C) 2018 Apple Inc. All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
+ * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
+ * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
+ * THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#pragma once
+
+#include <wtf/Forward.h>
+
+namespace WebKit {
+
+enum class WebPolicyAction : uint8_t {
+ Use,
+ Download,
+ Ignore,
+ Suspend
+};
+
+}
+
+namespace WTF {
+
+template<> struct EnumTraits<WebKit::WebPolicyAction> {
+ using values = EnumValues<
+ WebKit::WebPolicyAction,
+ WebKit::WebPolicyAction::Use,
+ WebKit::WebPolicyAction::Download,
+ WebKit::WebPolicyAction::Ignore,
+ WebKit::WebPolicyAction::Suspend
+ >;
+};
+
+} // namespace WTF
+
Modified: trunk/Source/WebKit/UIProcess/WebFramePolicyListenerProxy.cpp (239218 => 239219)
--- trunk/Source/WebKit/UIProcess/WebFramePolicyListenerProxy.cpp 2018-12-14 18:14:32 UTC (rev 239218)
+++ trunk/Source/WebKit/UIProcess/WebFramePolicyListenerProxy.cpp 2018-12-14 18:23:55 UTC (rev 239219)
@@ -50,7 +50,7 @@
ASSERT(!m_safeBrowsingWarning);
if (m_policyResult) {
if (m_reply)
- m_reply(WebCore::PolicyAction::Use, m_policyResult->first.get(), m_policyResult->second, WTFMove(safeBrowsingWarning));
+ m_reply(WebPolicyAction::Use, m_policyResult->first.get(), m_policyResult->second, WTFMove(safeBrowsingWarning));
} else
m_safeBrowsingWarning = WTFMove(safeBrowsingWarning);
}
@@ -59,7 +59,7 @@
{
if (m_safeBrowsingWarning) {
if (m_reply)
- m_reply(WebCore::PolicyAction::Use, policies, processSwapRequestedByClient, WTFMove(*m_safeBrowsingWarning));
+ m_reply(WebPolicyAction::Use, policies, processSwapRequestedByClient, WTFMove(*m_safeBrowsingWarning));
} else if (!m_policyResult)
m_policyResult = {{ policies, processSwapRequestedByClient }};
}
@@ -67,13 +67,13 @@
void WebFramePolicyListenerProxy::download()
{
if (m_reply)
- m_reply(WebCore::PolicyAction::Download, nullptr, ProcessSwapRequestedByClient::No, { });
+ m_reply(WebPolicyAction::Download, nullptr, ProcessSwapRequestedByClient::No, { });
}
void WebFramePolicyListenerProxy::ignore()
{
if (m_reply)
- m_reply(WebCore::PolicyAction::Ignore, nullptr, ProcessSwapRequestedByClient::No, { });
+ m_reply(WebPolicyAction::Ignore, nullptr, ProcessSwapRequestedByClient::No, { });
}
} // namespace WebKit
Modified: trunk/Source/WebKit/UIProcess/WebFramePolicyListenerProxy.h (239218 => 239219)
--- trunk/Source/WebKit/UIProcess/WebFramePolicyListenerProxy.h 2018-12-14 18:14:32 UTC (rev 239218)
+++ trunk/Source/WebKit/UIProcess/WebFramePolicyListenerProxy.h 2018-12-14 18:23:55 UTC (rev 239219)
@@ -26,6 +26,7 @@
#pragma once
#include "APIObject.h"
+#include "WebPolicyAction.h"
#include <wtf/CompletionHandler.h>
#include <wtf/Vector.h>
@@ -33,10 +34,6 @@
class WebsitePolicies;
}
-namespace WebCore {
-enum class PolicyAction : uint8_t;
-}
-
namespace WebKit {
class SafeBrowsingWarning;
@@ -47,7 +44,7 @@
class WebFramePolicyListenerProxy : public API::ObjectImpl<API::Object::Type::FramePolicyListener> {
public:
- using Reply = CompletionHandler<void(WebCore::PolicyAction, API::WebsitePolicies*, ProcessSwapRequestedByClient, RefPtr<SafeBrowsingWarning>&&)>;
+ using Reply = CompletionHandler<void(WebPolicyAction, API::WebsitePolicies*, ProcessSwapRequestedByClient, RefPtr<SafeBrowsingWarning>&&)>;
static Ref<WebFramePolicyListenerProxy> create(Reply&& reply, ShouldExpectSafeBrowsingResult expect)
{
return adoptRef(*new WebFramePolicyListenerProxy(WTFMove(reply), expect));
Modified: trunk/Source/WebKit/UIProcess/WebFrameProxy.cpp (239218 => 239219)
--- trunk/Source/WebKit/UIProcess/WebFrameProxy.cpp 2018-12-14 18:14:32 UTC (rev 239218)
+++ trunk/Source/WebKit/UIProcess/WebFrameProxy.cpp 2018-12-14 18:23:55 UTC (rev 239219)
@@ -177,11 +177,11 @@
m_title = title;
}
-WebFramePolicyListenerProxy& WebFrameProxy::setUpPolicyListenerProxy(CompletionHandler<void(WebCore::PolicyAction, API::WebsitePolicies*, ProcessSwapRequestedByClient, RefPtr<SafeBrowsingWarning>&&)>&& completionHandler, ShouldExpectSafeBrowsingResult expect)
+WebFramePolicyListenerProxy& WebFrameProxy::setUpPolicyListenerProxy(CompletionHandler<void(WebPolicyAction, API::WebsitePolicies*, ProcessSwapRequestedByClient, RefPtr<SafeBrowsingWarning>&&)>&& completionHandler, ShouldExpectSafeBrowsingResult expect)
{
if (m_activeListener)
m_activeListener->ignore();
- m_activeListener = WebFramePolicyListenerProxy::create([this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)] (WebCore::PolicyAction action, API::WebsitePolicies* policies, ProcessSwapRequestedByClient processSwapRequestedByClient, RefPtr<SafeBrowsingWarning>&& safeBrowsingWarning) mutable {
+ m_activeListener = WebFramePolicyListenerProxy::create([this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)] (WebPolicyAction action, API::WebsitePolicies* policies, ProcessSwapRequestedByClient processSwapRequestedByClient, RefPtr<SafeBrowsingWarning>&& safeBrowsingWarning) mutable {
completionHandler(action, policies, processSwapRequestedByClient, WTFMove(safeBrowsingWarning));
m_activeListener = nullptr;
}, expect);
Modified: trunk/Source/WebKit/UIProcess/WebFrameProxy.h (239218 => 239219)
--- trunk/Source/WebKit/UIProcess/WebFrameProxy.h 2018-12-14 18:14:32 UTC (rev 239218)
+++ trunk/Source/WebKit/UIProcess/WebFrameProxy.h 2018-12-14 18:23:55 UTC (rev 239219)
@@ -117,7 +117,7 @@
void didSameDocumentNavigation(const URL&); // eg. anchor navigation, session state change.
void didChangeTitle(const String&);
- WebFramePolicyListenerProxy& setUpPolicyListenerProxy(CompletionHandler<void(WebCore::PolicyAction, API::WebsitePolicies*, ProcessSwapRequestedByClient, RefPtr<SafeBrowsingWarning>&&)>&&, ShouldExpectSafeBrowsingResult);
+ WebFramePolicyListenerProxy& setUpPolicyListenerProxy(CompletionHandler<void(WebPolicyAction, API::WebsitePolicies*, ProcessSwapRequestedByClient, RefPtr<SafeBrowsingWarning>&&)>&&, ShouldExpectSafeBrowsingResult);
#if ENABLE(CONTENT_FILTERING)
void contentFilterDidBlockLoad(WebCore::ContentFilterUnblockHandler contentFilterUnblockHandler) { m_contentFilterUnblockHandler = WTFMove(contentFilterUnblockHandler); }
Modified: trunk/Source/WebKit/UIProcess/WebPageProxy.cpp (239218 => 239219)
--- trunk/Source/WebKit/UIProcess/WebPageProxy.cpp 2018-12-14 18:14:32 UTC (rev 239218)
+++ trunk/Source/WebKit/UIProcess/WebPageProxy.cpp 2018-12-14 18:23:55 UTC (rev 239219)
@@ -2594,7 +2594,7 @@
class WebPageProxy::PolicyDecisionSender : public RefCounted<PolicyDecisionSender> {
public:
- using SendFunction = CompletionHandler<void(WebCore::PolicyAction, uint64_t newNavigationID, DownloadID, std::optional<WebsitePoliciesData>)>;
+ using SendFunction = CompletionHandler<void(WebPolicyAction, uint64_t newNavigationID, DownloadID, std::optional<WebsitePoliciesData>)>;
static Ref<PolicyDecisionSender> create(SendFunction&& sendFunction)
{
return adoptRef(*new PolicyDecisionSender(WTFMove(sendFunction)));
@@ -2612,7 +2612,7 @@
SendFunction m_sendFunction;
};
-void WebPageProxy::receivedNavigationPolicyDecision(PolicyAction policyAction, API::Navigation* navigation, ProcessSwapRequestedByClient processSwapRequestedByClient, WebFrameProxy& frame, API::WebsitePolicies* policies, Ref<PolicyDecisionSender>&& sender)
+void WebPageProxy::receivedNavigationPolicyDecision(WebPolicyAction policyAction, API::Navigation* navigation, ProcessSwapRequestedByClient processSwapRequestedByClient, WebFrameProxy& frame, API::WebsitePolicies* policies, Ref<PolicyDecisionSender>&& sender)
{
std::optional<WebsitePoliciesData> data;
if (policies) {
@@ -2621,7 +2621,7 @@
changeWebsiteDataStore(policies->websiteDataStore()->websiteDataStore());
}
- if (policyAction != PolicyAction::Use || !frame.isMainFrame() || !navigation) {
+ if (policyAction != WebPolicyAction::Use || !frame.isMainFrame() || !navigation) {
receivedPolicyDecision(policyAction, navigation, WTFMove(data), WTFMove(sender));
return;
}
@@ -2634,7 +2634,7 @@
}
if (processForNavigation.ptr() != &process()) {
- policyAction = isPageOpenedByDOMShowingInitialEmptyDocument() ? PolicyAction::Ignore : PolicyAction::Suspend;
+ policyAction = isPageOpenedByDOMShowingInitialEmptyDocument() ? WebPolicyAction::Ignore : WebPolicyAction::Suspend;
RELEASE_LOG_IF_ALLOWED(ProcessSwapping, "%p - WebPageProxy::decidePolicyForNavigationAction, swapping process %i with process %i for navigation, reason: %{public}s", this, processIdentifier(), processForNavigation->processIdentifier(), reason.utf8().data());
LOG(ProcessSwapping, "(ProcessSwapping) Switching from process %i to new process (%i) for navigation %" PRIu64 " '%s'", processIdentifier(), processForNavigation->processIdentifier(), navigation->navigationID(), navigation->loggingString());
} else
@@ -2648,23 +2648,23 @@
});
}
-void WebPageProxy::receivedPolicyDecision(PolicyAction action, API::Navigation* navigation, std::optional<WebsitePoliciesData>&& websitePolicies, Ref<PolicyDecisionSender>&& sender)
+void WebPageProxy::receivedPolicyDecision(WebPolicyAction action, API::Navigation* navigation, std::optional<WebsitePoliciesData>&& websitePolicies, Ref<PolicyDecisionSender>&& sender)
{
if (!isValid()) {
- sender->send(PolicyAction::Ignore, 0, DownloadID(), std::nullopt);
+ sender->send(WebPolicyAction::Ignore, 0, DownloadID(), std::nullopt);
return;
}
auto transaction = m_pageLoadState.transaction();
- if (action == PolicyAction::Ignore)
+ if (action == WebPolicyAction::Ignore)
m_pageLoadState.clearPendingAPIRequestURL(transaction);
- if (navigation && navigation->shouldForceDownload() && action == PolicyAction::Use)
- action = ""
+ if (navigation && navigation->shouldForceDownload() && action == WebPolicyAction::Use)
+ action = ""
DownloadID downloadID = { };
- if (action == PolicyAction::Download) {
+ if (action == WebPolicyAction::Download) {
// Create a download proxy.
auto* download = m_process->processPool().createDownloadProxy(m_decidePolicyForResponseRequest, this);
if (navigation) {
@@ -4314,7 +4314,7 @@
#if ENABLE(CONTENT_FILTERING)
if (frame.didHandleContentFilterUnblockNavigation(request))
- return receivedPolicyDecision(PolicyAction::Ignore, m_navigationState->navigation(newNavigationID), std::nullopt, WTFMove(sender));
+ return receivedPolicyDecision(WebPolicyAction::Ignore, m_navigationState->navigation(newNavigationID), std::nullopt, WTFMove(sender));
#else
UNUSED_PARAM(newNavigationID);
#endif
@@ -4323,9 +4323,9 @@
if (!m_preferences->safeBrowsingEnabled())
shouldExpectSafeBrowsingResult = ShouldExpectSafeBrowsingResult::No;
- auto listener = makeRef(frame.setUpPolicyListenerProxy([this, protectedThis = makeRef(*this), frame = makeRef(frame), sender = WTFMove(sender), navigation] (WebCore::PolicyAction policyAction, API::WebsitePolicies* policies, ProcessSwapRequestedByClient processSwapRequestedByClient, RefPtr<SafeBrowsingWarning>&& safeBrowsingWarning) mutable {
+ auto listener = makeRef(frame.setUpPolicyListenerProxy([this, protectedThis = makeRef(*this), frame = makeRef(frame), sender = WTFMove(sender), navigation] (WebPolicyAction policyAction, API::WebsitePolicies* policies, ProcessSwapRequestedByClient processSwapRequestedByClient, RefPtr<SafeBrowsingWarning>&& safeBrowsingWarning) mutable {
- auto completionHandler = [this, protectedThis = protectedThis.copyRef(), frame = frame.copyRef(), sender = WTFMove(sender), navigation = WTFMove(navigation), processSwapRequestedByClient, policies = makeRefPtr(policies)] (PolicyAction policyAction) mutable {
+ auto completionHandler = [this, protectedThis = protectedThis.copyRef(), frame = frame.copyRef(), sender = WTFMove(sender), navigation = WTFMove(navigation), processSwapRequestedByClient, policies = makeRefPtr(policies)] (WebPolicyAction policyAction) mutable {
receivedNavigationPolicyDecision(policyAction, navigation.get(), processSwapRequestedByClient, frame, policies.get(), WTFMove(sender));
};
@@ -4337,12 +4337,12 @@
if (safeBrowsingWarning) {
m_pageClient->showSafeBrowsingWarning(*safeBrowsingWarning, [protectedThis = WTFMove(protectedThis), completionHandler = WTFMove(completionHandler), policyAction] (auto&& result) mutable {
switchOn(result, [&] (const URL& url) {
- completionHandler(PolicyAction::Ignore);
+ completionHandler(WebPolicyAction::Ignore);
protectedThis->loadRequest({ url });
}, [&] (ContinueUnsafeLoad continueUnsafeLoad) {
switch (continueUnsafeLoad) {
case ContinueUnsafeLoad::No:
- completionHandler(PolicyAction::Ignore);
+ completionHandler(WebPolicyAction::Ignore);
break;
case ContinueUnsafeLoad::Yes:
completionHandler(policyAction);
@@ -4405,7 +4405,7 @@
decidePolicyForNavigationAction(*frame, WTFMove(frameSecurityOrigin), navigationID, WTFMove(navigationActionData), WTFMove(frameInfoData), originatingPageID, originalRequest, WTFMove(request), WTFMove(requestBody), WTFMove(redirectResponse), userData, sender.copyRef());
// If the client did not respond synchronously, proceed with the load.
- sender->send(PolicyAction::Use, navigationID, DownloadID(), std::nullopt);
+ sender->send(WebPolicyAction::Use, navigationID, DownloadID(), std::nullopt);
}
void WebPageProxy::decidePolicyForNewWindowAction(uint64_t frameID, const SecurityOriginData& frameSecurityOrigin, NavigationActionData&& navigationActionData, ResourceRequest&& request, const String& frameName, uint64_t listenerID, const UserData& userData)
@@ -4416,7 +4416,7 @@
MESSAGE_CHECK(frame);
MESSAGE_CHECK_URL(request.url());
- auto listener = makeRef(frame->setUpPolicyListenerProxy([this, protectedThis = makeRef(*this), listenerID, frameID] (WebCore::PolicyAction policyAction, API::WebsitePolicies*, ProcessSwapRequestedByClient processSwapRequestedByClient, RefPtr<SafeBrowsingWarning>&& safeBrowsingWarning) mutable {
+ auto listener = makeRef(frame->setUpPolicyListenerProxy([this, protectedThis = makeRef(*this), listenerID, frameID] (WebPolicyAction policyAction, API::WebsitePolicies*, ProcessSwapRequestedByClient processSwapRequestedByClient, RefPtr<SafeBrowsingWarning>&& safeBrowsingWarning) mutable {
// FIXME: Assert the API::WebsitePolicies* is nullptr here once clients of WKFramePolicyListenerUseWithPolicies go away.
RELEASE_ASSERT(processSwapRequestedByClient == ProcessSwapRequestedByClient::No);
ASSERT_UNUSED(safeBrowsingWarning, !safeBrowsingWarning);
@@ -4454,7 +4454,7 @@
MESSAGE_CHECK_URL(response.url());
RefPtr<API::Navigation> navigation = navigationID ? m_navigationState->navigation(navigationID) : nullptr;
- auto listener = makeRef(frame->setUpPolicyListenerProxy([this, protectedThis = makeRef(*this), frameID, listenerID, navigation = WTFMove(navigation)] (WebCore::PolicyAction policyAction, API::WebsitePolicies*, ProcessSwapRequestedByClient processSwapRequestedByClient, RefPtr<SafeBrowsingWarning>&& safeBrowsingWarning) mutable {
+ auto listener = makeRef(frame->setUpPolicyListenerProxy([this, protectedThis = makeRef(*this), frameID, listenerID, navigation = WTFMove(navigation)] (WebPolicyAction policyAction, API::WebsitePolicies*, ProcessSwapRequestedByClient processSwapRequestedByClient, RefPtr<SafeBrowsingWarning>&& safeBrowsingWarning) mutable {
// FIXME: Assert the API::WebsitePolicies* is nullptr here once clients of WKFramePolicyListenerUseWithPolicies go away.
RELEASE_ASSERT(processSwapRequestedByClient == ProcessSwapRequestedByClient::No);
ASSERT_UNUSED(safeBrowsingWarning, !safeBrowsingWarning);
Modified: trunk/Source/WebKit/UIProcess/WebPageProxy.h (239218 => 239219)
--- trunk/Source/WebKit/UIProcess/WebPageProxy.h 2018-12-14 18:14:32 UTC (rev 239218)
+++ trunk/Source/WebKit/UIProcess/WebPageProxy.h 2018-12-14 18:23:55 UTC (rev 239219)
@@ -962,8 +962,8 @@
#endif
class PolicyDecisionSender;
- void receivedPolicyDecision(WebCore::PolicyAction, API::Navigation*, std::optional<WebsitePoliciesData>&&, Ref<PolicyDecisionSender>&&);
- void receivedNavigationPolicyDecision(WebCore::PolicyAction, API::Navigation*, ProcessSwapRequestedByClient, WebFrameProxy&, API::WebsitePolicies*, Ref<PolicyDecisionSender>&&);
+ void receivedPolicyDecision(WebPolicyAction, API::Navigation*, std::optional<WebsitePoliciesData>&&, Ref<PolicyDecisionSender>&&);
+ void receivedNavigationPolicyDecision(WebPolicyAction, API::Navigation*, ProcessSwapRequestedByClient, WebFrameProxy&, API::WebsitePolicies*, Ref<PolicyDecisionSender>&&);
void backForwardRemovedItem(const WebCore::BackForwardItemIdentifier&);
Modified: trunk/Source/WebKit/UIProcess/WebPageProxy.messages.in (239218 => 239219)
--- trunk/Source/WebKit/UIProcess/WebPageProxy.messages.in 2018-12-14 18:14:32 UTC (rev 239218)
+++ trunk/Source/WebKit/UIProcess/WebPageProxy.messages.in 2018-12-14 18:23:55 UTC (rev 239219)
@@ -107,7 +107,7 @@
# Policy messages
DecidePolicyForResponse(uint64_t frameID, struct WebCore::SecurityOriginData frameSecurityOrigin, uint64_t navigationID, WebCore::ResourceResponse response, WebCore::ResourceRequest request, bool canShowMIMEType, uint64_t listenerID, WebKit::UserData userData)
DecidePolicyForNavigationActionAsync(uint64_t frameID, struct WebCore::SecurityOriginData frameSecurityOrigin, uint64_t navigationID, struct WebKit::NavigationActionData navigationActionData, struct WebKit::FrameInfoData originatingFrameInfoData, uint64_t originatingPageID, WebCore::ResourceRequest originalRequest, WebCore::ResourceRequest request, IPC::FormDataReference requestBody, WebCore::ResourceResponse redirectResponse, WebKit::UserData userData, uint64_t listenerID)
- DecidePolicyForNavigationActionSync(uint64_t frameID, bool isMainFrame, struct WebCore::SecurityOriginData frameSecurityOrigin, uint64_t navigationID, struct WebKit::NavigationActionData navigationActionData, struct WebKit::FrameInfoData originatingFrameInfoData, uint64_t originatingPageID, WebCore::ResourceRequest originalRequest, WebCore::ResourceRequest request, IPC::FormDataReference requestBody, WebCore::ResourceResponse redirectResponse, WebKit::UserData userData) -> (enum:uint8_t WebCore::PolicyAction policyAction, uint64_t newNavigationID, WebKit::DownloadID downloadID, std::optional<WebKit::WebsitePoliciesData> websitePolicies) Delayed
+ DecidePolicyForNavigationActionSync(uint64_t frameID, bool isMainFrame, struct WebCore::SecurityOriginData frameSecurityOrigin, uint64_t navigationID, struct WebKit::NavigationActionData navigationActionData, struct WebKit::FrameInfoData originatingFrameInfoData, uint64_t originatingPageID, WebCore::ResourceRequest originalRequest, WebCore::ResourceRequest request, IPC::FormDataReference requestBody, WebCore::ResourceResponse redirectResponse, WebKit::UserData userData) -> (enum:uint8_t WebKit::WebPolicyAction policyAction, uint64_t newNavigationID, WebKit::DownloadID downloadID, std::optional<WebKit::WebsitePoliciesData> websitePolicies) Delayed
DecidePolicyForNewWindowAction(uint64_t frameID, struct WebCore::SecurityOriginData frameSecurityOrigin, struct WebKit::NavigationActionData navigationActionData, WebCore::ResourceRequest request, String frameName, uint64_t listenerID, WebKit::UserData userData)
UnableToImplementPolicy(uint64_t frameID, WebCore::ResourceError error, WebKit::UserData userData)
Modified: trunk/Source/WebKit/WebKit.xcodeproj/project.pbxproj (239218 => 239219)
--- trunk/Source/WebKit/WebKit.xcodeproj/project.pbxproj 2018-12-14 18:14:32 UTC (rev 239218)
+++ trunk/Source/WebKit/WebKit.xcodeproj/project.pbxproj 2018-12-14 18:23:55 UTC (rev 239219)
@@ -893,6 +893,7 @@
466BC03C1FA266DA002FA9C1 /* WebSWContextManagerConnection.h in Headers */ = {isa = PBXBuildFile; fileRef = 466BC0391FA266C9002FA9C1 /* WebSWContextManagerConnection.h */; };
46A2B6091E5676A600C3DEDA /* BackgroundProcessResponsivenessTimer.h in Headers */ = {isa = PBXBuildFile; fileRef = 46A2B6071E5675A200C3DEDA /* BackgroundProcessResponsivenessTimer.h */; };
46DF063C1F3905F8001980BB /* NetworkCORSPreflightChecker.h in Headers */ = {isa = PBXBuildFile; fileRef = 46DF063A1F3905E5001980BB /* NetworkCORSPreflightChecker.h */; };
+ 46F636D021C41A1D00413010 /* WebPolicyAction.h in Headers */ = {isa = PBXBuildFile; fileRef = 46F636CF21C41A1D00413010 /* WebPolicyAction.h */; };
4A3CC18B19B0640F00D14AEF /* UserMediaPermissionRequestManagerProxy.h in Headers */ = {isa = PBXBuildFile; fileRef = 4A410F3A19AF7B04002EBAB5 /* UserMediaPermissionRequestManagerProxy.h */; };
4A3CC18D19B0641900D14AEF /* UserMediaPermissionRequestProxy.h in Headers */ = {isa = PBXBuildFile; fileRef = 4A410F3C19AF7B04002EBAB5 /* UserMediaPermissionRequestProxy.h */; };
4A3CC18F19B07B8A00D14AEF /* WKUserMediaPermissionRequest.h in Headers */ = {isa = PBXBuildFile; fileRef = 4A410F3619AF7AC3002EBAB5 /* WKUserMediaPermissionRequest.h */; settings = {ATTRIBUTES = (Private, ); }; };
@@ -3095,6 +3096,7 @@
46A2B6071E5675A200C3DEDA /* BackgroundProcessResponsivenessTimer.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = BackgroundProcessResponsivenessTimer.h; sourceTree = "<group>"; };
46DF06391F3905E5001980BB /* NetworkCORSPreflightChecker.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = NetworkCORSPreflightChecker.cpp; sourceTree = "<group>"; };
46DF063A1F3905E5001980BB /* NetworkCORSPreflightChecker.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = NetworkCORSPreflightChecker.h; sourceTree = "<group>"; };
+ 46F636CF21C41A1D00413010 /* WebPolicyAction.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; path = WebPolicyAction.h; sourceTree = "<group>"; };
4A410F3519AF7AC3002EBAB5 /* WKUserMediaPermissionRequest.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = WKUserMediaPermissionRequest.cpp; sourceTree = "<group>"; };
4A410F3619AF7AC3002EBAB5 /* WKUserMediaPermissionRequest.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = WKUserMediaPermissionRequest.h; sourceTree = "<group>"; };
4A410F3919AF7B04002EBAB5 /* UserMediaPermissionRequestManagerProxy.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = UserMediaPermissionRequestManagerProxy.cpp; sourceTree = "<group>"; };
@@ -5126,6 +5128,7 @@
BC7B625112A43C9600D174A4 /* WebPageGroupData.cpp */,
BC7B625012A43C9600D174A4 /* WebPageGroupData.h */,
C0337DDC127A521C008FF4F4 /* WebPlatformTouchPoint.cpp */,
+ 46F636CF21C41A1D00413010 /* WebPolicyAction.h */,
BC5744ED12638FB3006F0F12 /* WebPopupItem.cpp */,
BC5744EE12638FB3006F0F12 /* WebPopupItem.h */,
7CDE739F1F9DA37A00390312 /* WebPreferences.yaml */,
@@ -9454,6 +9457,7 @@
0F850FE71ED7C39F00FB77A7 /* WebPerformanceLoggingClient.h in Headers */,
1A3E736111CC2659007BD539 /* WebPlatformStrategies.h in Headers */,
31D5929F166E060000E6BF02 /* WebPlugInClient.h in Headers */,
+ 46F636D021C41A1D00413010 /* WebPolicyAction.h in Headers */,
BC5744F012638FB3006F0F12 /* WebPopupItem.h in Headers */,
D3B9484711FF4B6500032B39 /* WebPopupMenu.h in Headers */,
BC574E631267D080006F0F12 /* WebPopupMenuProxy.h in Headers */,
Modified: trunk/Source/WebKit/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp (239218 => 239219)
--- trunk/Source/WebKit/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp 2018-12-14 18:14:32 UTC (rev 239218)
+++ trunk/Source/WebKit/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp 2018-12-14 18:23:55 UTC (rev 239219)
@@ -753,7 +753,7 @@
Ref<WebFrame> protector(*m_frame);
uint64_t listenerID = m_frame->setUpPolicyListener(WTFMove(function), WebFrame::ForNavigationAction::No);
if (!webPage->send(Messages::WebPageProxy::DecidePolicyForResponse(m_frame->frameID(), SecurityOriginData::fromFrame(coreFrame), navigationID, response, request, canShowResponse, listenerID, UserData(WebProcess::singleton().transformObjectsToHandles(userData.get()).get()))))
- m_frame->didReceivePolicyDecision(listenerID, PolicyAction::Ignore, 0, { }, { });
+ m_frame->didReceivePolicyDecision(listenerID, WebPolicyAction::Ignore, 0, { }, { });
}
void WebFrameLoaderClient::dispatchDecidePolicyForNewWindowAction(const NavigationAction& navigationAction, const ResourceRequest& request, FormState* formState, const String& frameName, FramePolicyFunction&& function)
@@ -890,12 +890,12 @@
if (policyDecisionMode == PolicyDecisionMode::Synchronous) {
uint64_t newNavigationID;
- PolicyAction policyAction;
+ WebPolicyAction policyAction;
DownloadID downloadID;
std::optional<WebsitePoliciesData> websitePolicies;
if (!webPage->sendSync(Messages::WebPageProxy::DecidePolicyForNavigationActionSync(m_frame->frameID(), m_frame->isMainFrame(), SecurityOriginData::fromFrame(coreFrame), documentLoader->navigationID(), navigationActionData, originatingFrameInfoData, originatingPageID, navigationAction.resourceRequest(), request, IPC::FormDataReference { request.httpBody() }, redirectResponse, UserData(WebProcess::singleton().transformObjectsToHandles(userData.get()).get())), Messages::WebPageProxy::DecidePolicyForNavigationActionSync::Reply(policyAction, newNavigationID, downloadID, websitePolicies))) {
- m_frame->didReceivePolicyDecision(listenerID, PolicyAction::Ignore, 0, { }, { });
+ m_frame->didReceivePolicyDecision(listenerID, WebPolicyAction::Ignore, 0, { }, { });
return;
}
@@ -905,7 +905,7 @@
ASSERT(policyDecisionMode == PolicyDecisionMode::Asynchronous);
if (!webPage->send(Messages::WebPageProxy::DecidePolicyForNavigationActionAsync(m_frame->frameID(), SecurityOriginData::fromFrame(coreFrame), documentLoader->navigationID(), navigationActionData, originatingFrameInfoData, originatingPageID, navigationAction.resourceRequest(), request, IPC::FormDataReference { request.httpBody() }, redirectResponse, UserData(WebProcess::singleton().transformObjectsToHandles(userData.get()).get()), listenerID)))
- m_frame->didReceivePolicyDecision(listenerID, PolicyAction::Ignore, 0, { }, { });
+ m_frame->didReceivePolicyDecision(listenerID, WebPolicyAction::Ignore, 0, { }, { });
}
void WebFrameLoaderClient::cancelPolicyCheck()
Modified: trunk/Source/WebKit/WebProcess/WebPage/WebFrame.cpp (239218 => 239219)
--- trunk/Source/WebKit/WebProcess/WebPage/WebFrame.cpp 2018-12-14 18:14:32 UTC (rev 239218)
+++ trunk/Source/WebKit/WebProcess/WebPage/WebFrame.cpp 2018-12-14 18:23:55 UTC (rev 239219)
@@ -43,6 +43,7 @@
#include "WebDocumentLoader.h"
#include "WebPage.h"
#include "WebPageProxyMessages.h"
+#include "WebPolicyAction.h"
#include "WebProcess.h"
#include "WebsitePoliciesData.h"
#include <_javascript_Core/APICast.h>
@@ -253,10 +254,26 @@
completionHandler();
}
-void WebFrame::didReceivePolicyDecision(uint64_t listenerID, PolicyAction action, uint64_t navigationID, DownloadID downloadID, std::optional<WebsitePoliciesData>&& websitePolicies)
+static WebCore::PolicyAction toPolicyAction(WebPolicyAction policyAction)
{
+ switch (policyAction) {
+ case WebPolicyAction::Use:
+ return WebCore::PolicyAction::Use;
+ case WebPolicyAction::Ignore:
+ return WebCore::PolicyAction::Ignore;
+ case WebPolicyAction::Download:
+ return WebCore::PolicyAction::Download;
+ case WebPolicyAction::Suspend:
+ break;
+ }
+ ASSERT_NOT_REACHED();
+ return WebCore::PolicyAction::Ignore;
+}
+
+void WebFrame::didReceivePolicyDecision(uint64_t listenerID, WebPolicyAction action, uint64_t navigationID, DownloadID downloadID, std::optional<WebsitePoliciesData>&& websitePolicies)
+{
if (!m_coreFrame || !m_policyListenerID || listenerID != m_policyListenerID || !m_policyFunction) {
- if (action == PolicyAction::Suspend)
+ if (action == WebPolicyAction::Suspend)
page()->send(Messages::WebPageProxy::DidFailToSuspendAfterProcessSwap());
return;
}
@@ -276,12 +293,12 @@
}
bool shouldSuspend = false;
- if (action == PolicyAction::Suspend) {
+ if (action == WebPolicyAction::Suspend) {
shouldSuspend = true;
- action = ""
+ action = ""
}
- function(action);
+ function(toPolicyAction(action));
if (shouldSuspend)
page()->suspendForProcessSwap();
Modified: trunk/Source/WebKit/WebProcess/WebPage/WebFrame.h (239218 => 239219)
--- trunk/Source/WebKit/WebProcess/WebPage/WebFrame.h 2018-12-14 18:14:32 UTC (rev 239218)
+++ trunk/Source/WebKit/WebProcess/WebPage/WebFrame.h 2018-12-14 18:23:55 UTC (rev 239219)
@@ -65,6 +65,8 @@
struct FrameInfoData;
struct WebsitePoliciesData;
+enum class WebPolicyAction : uint8_t;
+
class WebFrame : public API::ObjectImpl<API::Object::Type::BundleFrame> {
public:
static Ref<WebFrame> createWithCoreMainFrame(WebPage*, WebCore::Frame*);
@@ -85,7 +87,7 @@
enum class ForNavigationAction { No, Yes };
uint64_t setUpPolicyListener(WebCore::FramePolicyFunction&&, ForNavigationAction);
void invalidatePolicyListener();
- void didReceivePolicyDecision(uint64_t listenerID, WebCore::PolicyAction, uint64_t navigationID, DownloadID, std::optional<WebsitePoliciesData>&&);
+ void didReceivePolicyDecision(uint64_t listenerID, WebPolicyAction, uint64_t navigationID, DownloadID, std::optional<WebsitePoliciesData>&&);
uint64_t setUpWillSubmitFormListener(CompletionHandler<void()>&&);
void continueWillSubmitForm(uint64_t);
Modified: trunk/Source/WebKit/WebProcess/WebPage/WebPage.cpp (239218 => 239219)
--- trunk/Source/WebKit/WebProcess/WebPage/WebPage.cpp 2018-12-14 18:14:32 UTC (rev 239218)
+++ trunk/Source/WebKit/WebProcess/WebPage/WebPage.cpp 2018-12-14 18:23:55 UTC (rev 239219)
@@ -112,6 +112,7 @@
#include "WebPerformanceLoggingClient.h"
#include "WebPlugInClient.h"
#include "WebPluginInfoProvider.h"
+#include "WebPolicyAction.h"
#include "WebPopupMenu.h"
#include "WebPreferencesDefinitions.h"
#include "WebPreferencesKeys.h"
@@ -3026,12 +3027,12 @@
m_page->setSessionID(sessionID);
}
-void WebPage::didReceivePolicyDecision(uint64_t frameID, uint64_t listenerID, PolicyAction policyAction, uint64_t navigationID, const DownloadID& downloadID, std::optional<WebsitePoliciesData>&& websitePolicies)
+void WebPage::didReceivePolicyDecision(uint64_t frameID, uint64_t listenerID, WebPolicyAction policyAction, uint64_t navigationID, const DownloadID& downloadID, std::optional<WebsitePoliciesData>&& websitePolicies)
{
WebFrame* frame = WebProcess::singleton().webFrame(frameID);
if (!frame)
return;
- if (policyAction == PolicyAction::Suspend) {
+ if (policyAction == WebPolicyAction::Suspend) {
ASSERT(frame == m_mainFrame);
setIsSuspended(true);
Modified: trunk/Source/WebKit/WebProcess/WebPage/WebPage.h (239218 => 239219)
--- trunk/Source/WebKit/WebProcess/WebPage/WebPage.h 2018-12-14 18:14:32 UTC (rev 239218)
+++ trunk/Source/WebKit/WebProcess/WebPage/WebPage.h 2018-12-14 18:23:55 UTC (rev 239219)
@@ -238,6 +238,7 @@
enum FindOptions : uint16_t;
enum class DragControllerAction : uint8_t;
+enum class WebPolicyAction : uint8_t;
struct AssistedNodeInformation;
struct AttributedString;
@@ -1293,7 +1294,7 @@
bool parentProcessHasServiceWorkerEntitlement() const { return true; }
#endif
- void didReceivePolicyDecision(uint64_t frameID, uint64_t listenerID, WebCore::PolicyAction, uint64_t navigationID, const DownloadID&, std::optional<WebsitePoliciesData>&&);
+ void didReceivePolicyDecision(uint64_t frameID, uint64_t listenerID, WebPolicyAction, uint64_t navigationID, const DownloadID&, std::optional<WebsitePoliciesData>&&);
void continueWillSubmitForm(uint64_t frameID, uint64_t listenerID);
void setUserAgent(const String&);
void setCustomTextEncodingName(const String&);
Modified: trunk/Source/WebKit/WebProcess/WebPage/WebPage.messages.in (239218 => 239219)
--- trunk/Source/WebKit/WebProcess/WebPage/WebPage.messages.in 2018-12-14 18:14:32 UTC (rev 239218)
+++ trunk/Source/WebKit/WebProcess/WebPage/WebPage.messages.in 2018-12-14 18:23:55 UTC (rev 239219)
@@ -167,7 +167,7 @@
DidRemoveBackForwardItem(struct WebCore::BackForwardItemIdentifier backForwardItemID)
UpdateWebsitePolicies(struct WebKit::WebsitePoliciesData websitePolicies)
- DidReceivePolicyDecision(uint64_t frameID, uint64_t listenerID, enum:uint8_t WebCore::PolicyAction policyAction, uint64_t navigationID, WebKit::DownloadID downloadID, std::optional<WebKit::WebsitePoliciesData> websitePolicies)
+ DidReceivePolicyDecision(uint64_t frameID, uint64_t listenerID, enum:uint8_t WebKit::WebPolicyAction policyAction, uint64_t navigationID, WebKit::DownloadID downloadID, std::optional<WebKit::WebsitePoliciesData> websitePolicies)
ContinueWillSubmitForm(uint64_t frameID, uint64_t listenerID)
ClearSelection()