From 73906df41f9ec737a5575b7cd301c6d2cacc11a8 Mon Sep 17 00:00:00 2001
From: Brad Triebwasser <btriebw@chromium.org>
Date: Wed, 8 Feb 2023 18:18:29 -0800
Subject: [PATCH] Rename "window-placement" permission to "window-management"
 in web platform tests.

Rename "window-placement" permission to
"window-management" in web platform tests.

Bug: 1328581
Change-Id: I983a8e4e20393b973d69be86ed11eb048df07016
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4000319
Reviewed-by: Mike Wasserman <msw@chromium.org>
Commit-Queue: Brad Triebwasser <btriebw@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1103075}
---
 .../element-request-fullscreen-options.tentative.html  |  2 +-
 .../getScreenDetails.tentative.https.window.js         | 10 +++++-----
 screen-details/permission.https.window.js              |  8 ++++----
 window-placement/resources/helpers.js                  |  2 +-
 4 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/fullscreen/api/element-request-fullscreen-options.tentative.html b/fullscreen/api/element-request-fullscreen-options.tentative.html
index b2eb60088a3bee..f3b99e84e69681 100644
--- a/fullscreen/api/element-request-fullscreen-options.tentative.html
+++ b/fullscreen/api/element-request-fullscreen-options.tentative.html
@@ -8,7 +8,7 @@
 <script>
     promise_test(async (t) => {
         await test_driver.set_permission(
-            { name: "window-placement" },
+            { name: "window-management" },
             "granted"
         );
 
diff --git a/screen-details/getScreenDetails.tentative.https.window.js b/screen-details/getScreenDetails.tentative.https.window.js
index 914c72e5b8726b..06f0344f3ad989 100644
--- a/screen-details/getScreenDetails.tentative.https.window.js
+++ b/screen-details/getScreenDetails.tentative.https.window.js
@@ -8,7 +8,7 @@ promise_test(async t => {
 }, 'getScreenDetails() is present');
 
 promise_test(async t => {
-  await test_driver.set_permission({name: 'window-placement'}, 'granted');
+  await test_driver.set_permission({name: 'window-management'}, 'granted');
   const screenDetails = await self.getScreenDetails();
   assert_true(ScreenDetails.prototype.isPrototypeOf(screenDetails));
   assert_true(ScreenDetailed.prototype.isPrototypeOf(screenDetails.currentScreen));
@@ -19,14 +19,14 @@ promise_test(async t => {
 }, 'getScreenDetails() returns the expected interface with permission granted');
 
 promise_test(async t => {
-  await test_driver.set_permission({name: 'window-placement'}, 'granted');
+  await test_driver.set_permission({name: 'window-management'}, 'granted');
   assert_greater_than((await self.getScreenDetails()).screens.length, 0);
-  await test_driver.set_permission({name: 'window-placement'}, 'denied');
+  await test_driver.set_permission({name: 'window-management'}, 'denied');
   await promise_rejects_dom(t, 'NotAllowedError', self.getScreenDetails());
 }, 'getScreenDetails() rejects the promise with permission denied');
 
 promise_test(async t => {
-  await test_driver.set_permission({name: 'window-placement'}, 'granted');
+  await test_driver.set_permission({name: 'window-management'}, 'granted');
   let iframe = document.body.appendChild(document.createElement('iframe'));
   assert_greater_than((await iframe.contentWindow.getScreenDetails()).screens.length, 0);
 
@@ -50,7 +50,7 @@ promise_test(async t => {
 }, 'getScreenDetails() resolves for attached iframe; rejects for detached iframe');
 
 promise_test(async t => {
-  await test_driver.set_permission({name: 'window-placement'}, 'granted');
+  await test_driver.set_permission({name: 'window-management'}, 'granted');
   let iframe = document.body.appendChild(document.createElement('iframe'));
   const screenDetails = await iframe.contentWindow.getScreenDetails();
   assert_greater_than(screenDetails.screens.length, 0);
diff --git a/screen-details/permission.https.window.js b/screen-details/permission.https.window.js
index 32596fac9ea3bd..f477bd4230bbfa 100644
--- a/screen-details/permission.https.window.js
+++ b/screen-details/permission.https.window.js
@@ -4,17 +4,17 @@
 "use strict";
 
 promise_test(async t => {
-  await test_driver.set_permission({ name: "window-placement" }, "denied");
+  await test_driver.set_permission({ name: "window-management" }, "denied");
 
-  const status = await navigator.permissions.query({ name:"window-placement" });
+  const status = await navigator.permissions.query({ name:"window-management" });
   assert_class_string(status, "PermissionStatus");
   assert_equals(status.state, "denied");
 }, "Deny window management permission should work.");
 
 promise_test(async t => {
-  await test_driver.set_permission({ name: "window-placement" }, "granted");
+  await test_driver.set_permission({ name: "window-management" }, "granted");
 
-  const status = await navigator.permissions.query({ name: "window-placement" });
+  const status = await navigator.permissions.query({ name: "window-management" });
   assert_class_string(status, "PermissionStatus");
   assert_equals(status.state, "granted");
 }, "Grant window management permission should work.");
diff --git a/window-placement/resources/helpers.js b/window-placement/resources/helpers.js
index f6acb54c267d2b..92442f913324fd 100644
--- a/window-placement/resources/helpers.js
+++ b/window-placement/resources/helpers.js
@@ -28,7 +28,7 @@ async function setUpWindowManagement(setUpTest, setUpButton) {
     log(`WARNING: Run via 'wpt serve'; file URLs lack permission support`);
 
   try {  // Support manual testing where test_driver is not running.
-    await test_driver.set_permission({ name: 'window-placement' }, 'granted');
+    await test_driver.set_permission({ name: 'window-management' }, 'granted');
   } catch {
   }
   const setUpWatcher = new EventWatcher(setUpTest, setUpButton, ['click']);