Changeset 276667 in webkit

Timestamp:

Apr 27, 2021, 2:07:27 PM (5 years ago)

Author:

Alan Coon

Message:

Cherry-pick r276352. rdar://problem/77211405

Use BlobURL::getOriginURL in more places
​https://bugs.webkit.org/show_bug.cgi?id=224857

Reviewed by Alex Christensen.

Covered by existing tests.

• loader/PolicyChecker.cpp: (WebCore::FrameLoader::PolicyChecker::extendBlobURLLifetimeIfNecessary const):

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

Location:

branches/safari-611-branch/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• loader/PolicyChecker.cpp (modified) (1 diff)

branches/safari-611-branch/Source/WebCore/ChangeLog

@@ +1 @@
+2021-04-27  Russell Epstein  <repstein@apple.com>
+
+        Cherry-pick r276352. rdar://problem/77211405
+
+    Use BlobURL::getOriginURL in more places
+    https://bugs.webkit.org/show_bug.cgi?id=224857
+    
+    Reviewed by Alex Christensen.
+    
+    Covered by existing tests.
+    
+    * loader/PolicyChecker.cpp:
+    (WebCore::FrameLoader::PolicyChecker::extendBlobURLLifetimeIfNecessary const):
+    
+    
+    git-svn-id: https://svn.webkit.org/repository/webkit/trunk@276352 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+    2021-04-21  Youenn Fablet  <youenn@apple.com>
+
+            Use BlobURL::getOriginURL in more places
+            https://bugs.webkit.org/show_bug.cgi?id=224857
+
+            Reviewed by Alex Christensen.
+
+            Covered by existing tests.
+
+            * loader/PolicyChecker.cpp:
+            (WebCore::FrameLoader::PolicyChecker::extendBlobURLLifetimeIfNecessary const):
+
 2021-04-27  Russell Epstein  <repstein@apple.com>
 

branches/safari-611-branch/Source/WebCore/loader/PolicyChecker.cpp

@@ -112 +112 @@
 
     // Create a new temporary blobURL in case this one gets revoked during the asynchronous navigation policy decision.
-    URL temporaryBlobURL = BlobURL::createPublicURL(&m_frame.document()->securityOrigin());
-    ThreadableBlobRegistry::registerBlobURL(&m_frame.document()->securityOrigin(), temporaryBlobURL, request.url());
+    auto origin = SecurityOrigin::create(BlobURL::getOriginURL(request.url()));
+    URL temporaryBlobURL = BlobURL::createPublicURL(origin.ptr());
+    ThreadableBlobRegistry::registerBlobURL(origin.ptr(), temporaryBlobURL, request.url());
     request.setURL(temporaryBlobURL);
     if (loader)