WebFrameLoaderClient::dispatchDecidePolicyForNavigationAction() may not be able to
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 31 Jul 2017 21:26:26 +0000 (21:26 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 31 Jul 2017 21:26:26 +0000 (21:26 +0000)
determine page that initiated navigation
https://bugs.webkit.org/show_bug.cgi?id=174531

Remove a comment from WebFrameLoaderClient::dispatchDecidePolicyForNavigationAction()
about determining the originating page independently from the originating frame. There
does not seem to be much value to fixing this. We can always revisit this issue if we
discover a strong motivating example. See <https://bugs.webkit.org/show_bug.cgi?id=174531#c1>
for more details.

* WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
(WebKit::WebFrameLoaderClient::dispatchDecidePolicyForNavigationAction):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@220074 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit/ChangeLog
Source/WebKit/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp

index a5c00f2..904ebeb 100644 (file)
@@ -1,3 +1,18 @@
+2017-07-31  Daniel Bates  <dabates@apple.com>
+
+        WebFrameLoaderClient::dispatchDecidePolicyForNavigationAction() may not be able to
+        determine page that initiated navigation
+        https://bugs.webkit.org/show_bug.cgi?id=174531
+
+        Remove a comment from WebFrameLoaderClient::dispatchDecidePolicyForNavigationAction()
+        about determining the originating page independently from the originating frame. There
+        does not seem to be much value to fixing this. We can always revisit this issue if we
+        discover a strong motivating example. See <https://bugs.webkit.org/show_bug.cgi?id=174531#c1>
+        for more details.
+
+        * WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
+        (WebKit::WebFrameLoaderClient::dispatchDecidePolicyForNavigationAction):
+
 2017-07-31  Sam Weinig  <sam@webkit.org>
 
         Remove unnecessary exceptions from storage code
index 9f5f984..8eb97a5 100644 (file)
@@ -822,9 +822,6 @@ void WebFrameLoaderClient::dispatchDecidePolicyForNavigationAction(const Navigat
     // Notify the UIProcess.
     Ref<WebFrame> protect(*m_frame);
     WebsitePolicies websitePolicies;
-    // FIXME: Determine the originating page independently from the originating frame as it may exist even if
-    // the originating frame does not exist. This can happen if the originating frame was removed from the page.
-    // See <https://bugs.webkit.org/show_bug.cgi?id=174531>.
     if (!webPage->sendSync(Messages::WebPageProxy::DecidePolicyForNavigationAction(m_frame->frameID(), SecurityOriginData::fromFrame(coreFrame), documentLoader->navigationID(), navigationActionData, originatingFrameInfoData, originatingFrame && originatingFrame->page() ? originatingFrame->page()->pageID() : 0, navigationAction.resourceRequest(), request, listenerID, UserData(WebProcess::singleton().transformObjectsToHandles(userData.get()).get())), Messages::WebPageProxy::DecidePolicyForNavigationAction::Reply(newNavigationID, policyAction, downloadID, websitePolicies))) {
         m_frame->didReceivePolicyDecision(listenerID, PolicyIgnore, 0, { });
         return;