Re: [w3c/permissions] Switch PermissionName to DOMString (PR #353)

@miketaylr commented on this pull request.



> @@ -726,6 +721,19 @@ <h4 id="query-method">
             <var>permissionDesc</var>:
           </p>
           <ol class="algorithm">
+            <li>If |permissionDesc|'s {{PermissionDescriptor/name}} member is not supported, return
+            [=a promise rejected with=] a {{TypeError}}.
+              <aside class="note" title="Why is this not an enum?">
+                <p>
+                  This is deliberately design to work the same a WebIDL [=enumeration=] (`enum`)
+                  and implementers are encouraged to use their own custom `enum` here. The reason
+                  this is not an enum in the specification vary greatly in the powerful features
+                  they support. Using a {{DOMString}} to identify a powerful feature gives
+                  implementers the freedom to pick and choose which of the power features from the
+                  [[[powerful-feature-registry]]] they wish to support.

This links out to https://w3c.github.io/poweful-features-registry/ (note the typo in powerful) - presumably we fix that elsewhere.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/w3c/permissions/pull/353#pullrequestreview-878067397
You are receiving this because you are subscribed to this thread.

Message ID: <w3c/permissions/pull/353/review/878067397@github.com>