- From: Brian Burg <notifications@github.com>
- Date: Wed, 04 Oct 2017 16:11:51 +0000 (UTC)
- To: w3c/permissions <permissions@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
- Message-ID: <w3c/permissions/pull/151/review/67120915@github.com>
burg commented on this pull request. > + <li>If |permissionDesc| is not a JSON object, return <a>error</a> with + <a>error code</a> <a>invalid argument</a>.</li> + <li>Let |rootDesc| be the object |permissionDesc| refers to, <a>converted + to an IDL value</a> of type {{PermissionDescriptor}}. If this throws an + exception, return <a>error</a> with <a>error code</a> <a>invalid + argument</a>.</li> + <li>Let |typedDescriptor| be the object |permissionDesc| refers to, + <a>converted to an IDL value</a> of + <code>|rootDesc|.{{PermissionDescriptor/name}}</code>'s <a>permission + descriptor type</a>. If this throws an exception, return <a>error</a> + with <a>error code</a> <a>invalid argument</a>.</li> + <li>Let |settings| be the <a>current settings object</a>.</li> + <li>If |settings| is a <a>non-secure context</a> and + <code>|rootDesc|.{{PermissionDescriptor/name}}</code> isn't <a>allowed in + non-secure contexts</a>, return <a>error</a> with <a>error code</a> + <a>invalid argument</a>.</li> >From a WebDriver spec point of view, it is confusing that "is a secure context" is something to decide about a settings object rather than a browsing context. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/w3c/permissions/pull/151#discussion_r142718185
Received on Wednesday, 4 October 2017 16:12:15 UTC