Fix StorageAccess to work with RemoteFrames.
When a page with StorageAccess is loaded into a cross-process frame,
the `storage_access_api_status` value is missing. This CL plumbs the
value through the RenderFrameProxyHost, and updates tests to exercise
both the same-process and cross-process states.
Bug: 40259221
Change-Id: I63173be82f5b8fd956f2456efb2c4a09c67cc2f8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5669691
Reviewed-by: Brendon Tiszka <[email protected]>
Reviewed-by: Charlie Reis <[email protected]>
Reviewed-by: Chris Fredrickson <[email protected]>
Commit-Queue: James Maclean <[email protected]>
Reviewed-by: Daniel Cheng <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1328121}
diff --git a/content/browser/renderer_host/navigator.h b/content/browser/renderer_host/navigator.h
index 84324a8..22051fa 100644
--- a/content/browser/renderer_host/navigator.h
+++ b/content/browser/renderer_host/navigator.h
@@ -17,6 +17,7 @@
#include "content/public/browser/navigation_discard_reason.h"
#include "mojo/public/cpp/bindings/pending_associated_remote.h"
#include "mojo/public/cpp/bindings/pending_remote.h"
+#include "net/storage_access_api/status.h"
#include "third_party/blink/public/common/navigation/impression.h"
#include "third_party/blink/public/mojom/frame/triggering_event_info.mojom-shared.h"
#include "third_party/blink/public/mojom/navigation/navigation_initiator_activation_and_ad_status.mojom.h"
@@ -170,6 +171,8 @@
bool force_new_browsing_instance = false,
bool is_container_initiated = false,
bool has_rel_opener = false,
+ net::StorageAccessApiStatus storage_access_api_status =
+ net::StorageAccessApiStatus::kNone,
std::optional<std::u16string> embedder_shared_storage_context =
std::nullopt);
