[cr134] BlobURLStoreImpl passing is_top_level_navigation
This argument is being propagated in several places, and it has affect an override, however the argument is only passed along. Chromium change: https://chromium.googlesource.com/chromium/src/+/4fe86b8660f18e2cd5dea36b3dabbe587c8ce72c commit 4fe86b8660f18e2cd5dea36b3dabbe587c8ce72c Author: Janice Liu <janiceliu@chromium.org> Date: Fri Jan 31 12:24:53 2025 -0800 [Blob URL] Partition Blob URL Subframe Navigations `noopener` is enforced for Blob URL navigations, but navigations that aren't top-level should be partitioned to align with Firefox and Safari. Bug: 381211737
This commit is contained in:
@@ -31,14 +31,15 @@ void BlobURLStoreImpl::ResolveAsURLLoaderFactory(
|
||||
void BlobURLStoreImpl::ResolveForNavigation(
|
||||
const GURL& url,
|
||||
mojo::PendingReceiver<blink::mojom::BlobURLToken> token,
|
||||
bool is_top_level_navigation,
|
||||
ResolveForNavigationCallback callback) {
|
||||
if (!IsBlobResolvable(url)) {
|
||||
std::move(callback).Run(std::nullopt);
|
||||
return;
|
||||
}
|
||||
|
||||
BlobURLStoreImpl_ChromiumImpl::ResolveForNavigation(url, std::move(token),
|
||||
std::move(callback));
|
||||
BlobURLStoreImpl_ChromiumImpl::ResolveForNavigation(
|
||||
url, std::move(token), is_top_level_navigation, std::move(callback));
|
||||
}
|
||||
|
||||
bool BlobURLStoreImpl::IsBlobResolvable(const GURL& url) const {
|
||||
|
||||
@@ -38,6 +38,7 @@ class COMPONENT_EXPORT(STORAGE_BROWSER) BlobURLStoreImpl
|
||||
void ResolveForNavigation(
|
||||
const GURL& url,
|
||||
mojo::PendingReceiver<blink::mojom::BlobURLToken> token,
|
||||
bool is_top_level_navigation,
|
||||
ResolveForNavigationCallback callback) override;
|
||||
|
||||
private:
|
||||
|
||||
Reference in New Issue
Block a user