commit 9903e1acb8d6215c058f36523d936fa6f5487796
author Daseul Lee <dslee@chromium.org> Mon Oct 03 23:38:31 2022
committer Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Mon Oct 03 23:45:13 2022
tree 5617d4d0b1d85349e7044122d64733833c090a50
parent 783b6d8975c43261f7fb9f4cd7819f0f69355f14

Remove queryPermission() usage from wpt/fs helper function.

Permission is not required in OPFS.
https://github.com/whatwg/fs/issues/42

Bug: 1362626
Change-Id: I2c75c0374c9f5a9c323a03950f7bd3bdd4a93750
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3923838
Commit-Queue: Daseul Lee <dslee@chromium.org>
Reviewed-by: Austin Sullivan <asully@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1054413}

fs/resources/messaging-serialize-helpers.js

diff --git a/fs/resources/messaging-serialize-helpers.js b/fs/resources/messaging-serialize-helpers.js
index 13c900c..c7dfc04 100644
--- a/fs/resources/messaging-serialize-helpers.js
+++ b/fs/resources/messaging-serialize-helpers.js
@@ -30,22 +30,9 @@
 // serialized properties shared by both FileSystemFileHandle and
 // FileSystemDirectoryHandle.
 async function serialize_file_system_handle(handle) {
-  let read_permission = "granted";
-  let write_permission = "granted";
-  // query-permission is part of the File System Acecss API
-  // https://wicg.github.io/file-system-access, which may not be supported
-  if ("queryPermission" in FileSystemHandle.prototype) {
-    read_permission =
-      await handle.queryPermission({ mode: 'read' });
-
-    write_permission =
-      await handle.queryPermission({ mode: 'readwrite' })
-  }
   return {
     kind: handle.kind,
-    name: handle.name,
-    read_permission,
-    write_permission
+    name: handle.name
   };
 }
 
@@ -54,9 +41,7 @@
 // dictionary.  Example output:
 // {
 //   kind: "file",
-//   name: "example-file-name"
-//   read_permission: "granted",
-//   write_permission: "granted",
+//   name: "example-file-name",
 //   contents: "example-file-contents"
 // }
 async function serialize_file_system_file_handle(file_handle) {
@@ -72,9 +57,7 @@
 // Example output:
 // {
 //   kind: "directory",
-//   name: "example-directory-name"
-//   read_permission: "granted",
-//   write_permission: "granted",
+//   name: "example-directory-name",
 //   files: [<first serialized file>, ...]
 //   directories: [<first serialized subdirectory>, ...]
 // }
@@ -161,14 +144,6 @@
   assert_equals(left.name, right.name,
     'Each FileSystemHandle instance must use the expected "name" ' +
     ' property.');
-
-  assert_equals(left.read_permission, right.read_permission,
-    'Each FileSystemHandle instance must have the expected read ' +
-    ' permission.');
-
-  assert_equals(left.write_permission, right.write_permission,
-    'Each FileSystemHandle instance must have the expected write ' +
-    ' permission.');
 }
 
 // Compares the output of serialize_file_system_file_handle()