Title: [280848] branches/safari-611.3.10.0-branch/Source/WebKit
Revision
280848
Author
repst...@apple.com
Date
2021-08-10 10:08:15 -0700 (Tue, 10 Aug 2021)

Log Message

Cherry-pick r279254. rdar://problem/80635931

    Crash in WebFrameLoaderClient::dispatchDidStartProvisionalLoad after provisionalLoader is gone.
    https://bugs.webkit.org/show_bug.cgi?id=226979

    Patch by Venky Dass <yaranamavenkataram...@apple.com> on 2021-06-24
    Reviewed by Ryosuke Niwa.

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

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

Modified Paths

Diff

Modified: branches/safari-611.3.10.0-branch/Source/WebKit/ChangeLog (280847 => 280848)


--- branches/safari-611.3.10.0-branch/Source/WebKit/ChangeLog	2021-08-10 17:08:11 UTC (rev 280847)
+++ branches/safari-611.3.10.0-branch/Source/WebKit/ChangeLog	2021-08-10 17:08:15 UTC (rev 280848)
@@ -1,3 +1,28 @@
+2021-08-10  Russell Epstein  <repst...@apple.com>
+
+        Cherry-pick r279254. rdar://problem/80635931
+
+    Crash in WebFrameLoaderClient::dispatchDidStartProvisionalLoad after provisionalLoader is gone.
+    https://bugs.webkit.org/show_bug.cgi?id=226979
+    
+    Patch by Venky Dass <yaranamavenkataram...@apple.com> on 2021-06-24
+    Reviewed by Ryosuke Niwa.
+    
+    * WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
+    (WebKit::WebFrameLoaderClient::dispatchDidStartProvisionalLoad):
+    
+    git-svn-id: https://svn.webkit.org/repository/webkit/trunk@279254 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+    2021-06-24  Venky Dass  <yaranamavenkataram...@apple.com>
+
+            Crash in WebFrameLoaderClient::dispatchDidStartProvisionalLoad after provisionalLoader is gone.
+            https://bugs.webkit.org/show_bug.cgi?id=226979
+
+            Reviewed by Ryosuke Niwa.
+
+            * WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
+            (WebKit::WebFrameLoaderClient::dispatchDidStartProvisionalLoad):
+
 2021-06-17  Alan Coon  <alanc...@apple.com>
 
         Cherry-pick r278870. rdar://problem/79473993

Modified: branches/safari-611.3.10.0-branch/Source/WebKit/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp (280847 => 280848)


--- branches/safari-611.3.10.0-branch/Source/WebKit/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp	2021-08-10 17:08:11 UTC (rev 280847)
+++ branches/safari-611.3.10.0-branch/Source/WebKit/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp	2021-08-10 17:08:15 UTC (rev 280848)
@@ -518,17 +518,20 @@
     webPage->findController().hideFindUI();
     webPage->sandboxExtensionTracker().didStartProvisionalLoad(m_frame.ptr());
 
-    WebDocumentLoader& provisionalLoader = static_cast<WebDocumentLoader&>(*m_frame->coreFrame()->loader().provisionalDocumentLoader());
-    auto& url = ""
+
     RefPtr<API::Object> userData;
 
     // Notify the bundle client.
     webPage->injectedBundleLoaderClient().didStartProvisionalLoadForFrame(*webPage, m_frame, userData);
+    RefPtr provisionalLoader = static_cast<WebDocumentLoader*>(m_frame->coreFrame()->loader().provisionalDocumentLoader());
 
-    auto& unreachableURL = provisionalLoader.unreachableURL();
-
+    if (!provisionalLoader)
+        return;
+    
+    auto& url = ""
+    auto& unreachableURL = provisionalLoader->unreachableURL();
     // Notify the UIProcess.
-    webPage->send(Messages::WebPageProxy::DidStartProvisionalLoadForFrame(m_frame->frameID(), m_frame->info(), provisionalLoader.request(), provisionalLoader.navigationID(), url, unreachableURL, UserData(WebProcess::singleton().transformObjectsToHandles(userData.get()).get())));
+    webPage->send(Messages::WebPageProxy::DidStartProvisionalLoadForFrame(m_frame->frameID(), m_frame->info(), provisionalLoader->request(), provisionalLoader->navigationID(), url, unreachableURL, UserData(WebProcess::singleton().transformObjectsToHandles(userData.get()).get())));
 }
 
 static constexpr unsigned maxTitleLength = 1000; // Closest power of 10 above the W3C recommendation for Title length.
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to