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

@marcoscaceres 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.

Fixed elsewhere (main) and rebased... now made it so once we merge this, it should link without needing the localBiblio. 

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

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