Skip to content

Commit 7bd5202

Browse files
[Do not submit yet] Include unpartitioned cookie availability check in hasStorageAccess()
This change is based on spec PR privacycg/storage-access#174. Bug: 1433013 Change-Id: I6c29b2a2afddb288d40d946040dc73fbe76b6fcb
1 parent 4bdf59d commit 7bd5202

File tree

4 files changed

+9
-0
lines changed

4 files changed

+9
-0
lines changed

storage-access-api/hasStorageAccess.sub.https.window.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
// META: script=helpers.js
2+
// META: script=/resources/testdriver.js
3+
// META: script=/resources/testdriver-vendor.js
24
'use strict';
35

46
const {testPrefix, topLevelDocument} = processQueryParams();
@@ -9,6 +11,7 @@ promise_test(async () => {
911
}, "[" + testPrefix + "] document.hasStorageAccess() should exist on the document interface");
1012

1113
promise_test(async () => {
14+
await MaybeSetStorageAccess("*", "*", "blocked");
1215
const hasAccess = await document.hasStorageAccess();
1316
if (topLevelDocument || testPrefix.includes('same-origin')) {
1417
assert_true(hasAccess, "Access should be granted in top-level frame or iframe that is in first-party context by default.");

storage-access-api/resources/hasStorageAccess-ABA-iframe.https.html

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,8 @@
22
<meta charset=utf-8>
33

44
<script src="/resources/testharness.js"></script>
5+
<script src="/resources/testdriver.js"></script>
6+
<script src="/resources/testdriver-vendor.js"></script>
57
<script src="/storage-access-api/helpers.js"></script>
68
<body>
79
<script src="/storage-access-api/resources/hasStorageAccess-ABA-iframe.sub.https.window.js"></script>

storage-access-api/resources/hasStorageAccess-ABA-iframe.sub.https.window.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
// META: script=../helpers.js
2+
// META: script=/resources/testdriver.js
3+
// META: script=/resources/testdriver-vendor.js
24
'use strict';
35

46
// This expects to be run in an iframe that is cross-site to the top-level frame.

storage-access-api/resources/hasStorageAccess-iframe.https.html

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,8 @@
22
<meta charset=utf-8>
33

44
<script src="/resources/testharness.js"></script>
5+
<script src="/resources/testdriver.js"></script>
6+
<script src="/resources/testdriver-vendor.js"></script>
57
<!-- no testharnessreport.js -->
68
<script src="../helpers.js"></script>
79
<div id=log></div>

0 commit comments

Comments
 (0)