- From: Mike Taylor <notifications@github.com>
- Date: Fri, 12 Jan 2024 14:27:25 -0800
- To: w3c/permissions <permissions@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Friday, 12 January 2024 22:27:30 UTC
@miketaylr requested changes on this pull request. Looks good - just some minor nits. > </li> <li>Let |targets| be a <a>list</a> containing all [=environment settings objects=] whose - [=environment settings object/origin=] is [=same origin=] as the [=environment settings - object/origin=] of |settings|. + [=environment settings object/origin=] is [=same origin=] as |target origin|. ```suggestion [=environment settings object/origin=] is [=same origin=] with |target origin|. ``` > @@ -1232,14 +1232,15 @@ <h2 id="automation"> <div class="algorithm"> <p> To <dfn data-for="WebDriver">set a permission</dfn> given a {{PermissionDescriptor}} - |descriptor:PermissionDescriptor|, and a {{PermissionState}} |state:PermissionState|: + |descriptor:PermissionDescriptor|, a {{PermissionState}} |state:PermissionState|, and an + |origin|: Let's change this to "and an optional origin", then we don't have to pass in null down on line 1328. -- Reply to this email directly or view it on GitHub: https://github.com/w3c/permissions/pull/436#pullrequestreview-1819187415 You are receiving this because you are subscribed to this thread. Message ID: <w3c/permissions/pull/436/review/1819187415@github.com>
Received on Friday, 12 January 2024 22:27:30 UTC