Changeset 276352 in webkit

Timestamp:

Apr 21, 2021, 1:44:42 AM (5 years ago)

Author:

youenn@apple.com

Message:

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):

Location:

trunk/Source/WebCore

Files:

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

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+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-21  Tim Horton  <timothy_horton@apple.com>
 

trunk/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)