From e7837bfae6f5cffb344d725a86a605109404e1b5 Mon Sep 17 00:00:00 2001
From: Marcos Caceres <marcos@marcosc.com>
Date: Mon, 19 Jun 2023 11:22:03 +1000
Subject: [PATCH] Remove allowpaymentrequest permission policy tests

---
 feature-policy/resources/featurepolicy.js            |  4 ++--
 ...-policy-attribute-redirect-on-load.https.sub.html | 11 -----------
 ...payment-default-permissions-policy.https.sub.html | 12 ------------
 ...ent-disabled-by-permissions-policy.https.sub.html | 12 ------------
 permissions-policy/resources/permissions-policy.js   |  4 ++--
 5 files changed, 4 insertions(+), 39 deletions(-)

diff --git a/feature-policy/resources/featurepolicy.js b/feature-policy/resources/featurepolicy.js
index 864c434c6633ca..e9b8804f964117 100644
--- a/feature-policy/resources/featurepolicy.js
+++ b/feature-policy/resources/featurepolicy.js
@@ -24,8 +24,8 @@ function assert_feature_policy_supported() {
 //      feature (https://wicg.github.io/feature-policy/#features).
 //      See examples at:
 //      https://github.com/WICG/feature-policy/blob/master/features.md
-//    allow_attribute: Optional argument, only used for testing fullscreen:
-//      "allowfullscreen"
+//    allow_attribute: Optional argument, only used for testing fullscreen
+//      by passing "allowfullscreen".
 function test_feature_availability(
     feature_description, test, src, expect_feature_available, feature_name,
     allow_attribute) {
diff --git a/permissions-policy/payment-allowed-by-permissions-policy-attribute-redirect-on-load.https.sub.html b/permissions-policy/payment-allowed-by-permissions-policy-attribute-redirect-on-load.https.sub.html
index e43f4c80ced1f5..b95deed7cef5ae 100644
--- a/permissions-policy/payment-allowed-by-permissions-policy-attribute-redirect-on-load.https.sub.html
+++ b/permissions-policy/payment-allowed-by-permissions-policy-attribute-redirect-on-load.https.sub.html
@@ -24,16 +24,5 @@
         expect_feature_unavailable_default, 'payment');
   }, header + ' disallows cross-origin navigation in an iframe.');
 
-  async_test(t => {
-    test_feature_availability(
-        'PaymentRequest()', t, same_origin_src,
-        expect_feature_available_default, 'payment', 'allowpaymentrequest');
-  }, header + ' allowpaymentrequest=true allows same-origin navigation in an iframe.');
-
-  async_test(t => {
-    test_feature_availability(
-        'PaymentRequest()', t, cross_origin_src,
-        expect_feature_unavailable_default, 'payment', 'allowpaymentrequest');
-  }, header + ' allowpaymentrequest=true disallows cross-origin navigation in an iframe.');
   </script>
 </body>
diff --git a/permissions-policy/payment-default-permissions-policy.https.sub.html b/permissions-policy/payment-default-permissions-policy.https.sub.html
index 71ab15ae89c276..da5fe80f66f0c2 100644
--- a/permissions-policy/payment-default-permissions-policy.https.sub.html
+++ b/permissions-policy/payment-default-permissions-policy.https.sub.html
@@ -31,17 +31,5 @@
     test_feature_availability('PaymentRequest()', t, cross_origin_src,
         expect_feature_unavailable_default);
   }, header + ' disallows cross-origin iframes.');
-
-  async_test(t => {
-    test_feature_availability(
-        'PaymentRequest()', t, same_origin_src,
-        expect_feature_available_default, undefined, 'allowpaymentrequest');
-  }, header + ' allowpaymentrequest=true allows same-origin iframes.');
-
-  async_test(t => {
-    test_feature_availability(
-        'PaymentRequest()', t, cross_origin_src,
-        expect_feature_available_default, undefined, 'allowpaymentrequest');
-  }, header + ' allowpaymentrequest=true allows cross-origin iframes.');
   </script>
 </body>
diff --git a/permissions-policy/payment-disabled-by-permissions-policy.https.sub.html b/permissions-policy/payment-disabled-by-permissions-policy.https.sub.html
index 90f3a0ee031c78..cc358a1c0f6623 100644
--- a/permissions-policy/payment-disabled-by-permissions-policy.https.sub.html
+++ b/permissions-policy/payment-disabled-by-permissions-policy.https.sub.html
@@ -29,17 +29,5 @@
     test_feature_availability('PaymentRequest()', t, cross_origin_src,
         expect_feature_unavailable_default,);
   }, header + ' disallows cross-origin iframes.');
-
-  async_test(t => {
-    test_feature_availability(
-        'PaymentRequest()', t, same_origin_src,
-        expect_feature_unavailable_default, undefined, 'allowpaymentrequest');
-  }, header + ' allowpaymentrequest=true disallows same-origin iframes.');
-
-  async_test(t => {
-    test_feature_availability(
-        'PaymentRequest()', t, cross_origin_src,
-        expect_feature_unavailable_default, undefined, 'allowpaymentrequest');
-  }, header + ' allowpaymentrequest=true disallows cross-origin iframes.');
   </script>
 </body>
diff --git a/permissions-policy/resources/permissions-policy.js b/permissions-policy/resources/permissions-policy.js
index 62f8dcdf9155ae..32fb4cfd4a9535 100644
--- a/permissions-policy/resources/permissions-policy.js
+++ b/permissions-policy/resources/permissions-policy.js
@@ -24,8 +24,8 @@ function assert_permissions_policy_supported() {
 //      feature (https://w3c.github.io/webappsec-permissions-policy/#features).
 //      See examples at:
 //      https://github.com/w3c/webappsec-permissions-policy/blob/main/features.md
-//    allow_attribute: Optional argument, only used for testing fullscreen or
-//      payment: either "allowfullscreen" or "allowpaymentrequest" is passed.
+//    allow_attribute: Optional argument, only used for testing fullscreen
+//      by passing "allowfullscreen".
 //    is_promise_test: Optional argument, true if this call should return a
 //    promise. Used by test_feature_availability_with_post_message_result()
 function test_feature_availability(