Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix link to Permissions Policy’s "Parse policy directive" algorithm #107

Closed
wants to merge 2 commits into from
+4 −4
Closed

Fix link to Permissions Policy’s "Parse policy directive" algorithm #107

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
c7f75c4
Fix link to Permissions Policy’s "Parse policy directive" algorithm
eeeps Apr 13, 2022
585b337
Link Permissions Policy algorithms as algorithms
eeeps Apr 13, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions index.bs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,7 @@ Note: This metadata *appends* [=client hints token=]s to the [=environment setti
<li>If |settingsObject| is a [=non-secure context=], abort these steps.
<li>Let |browsingContext| be |settingsObject|'s [=environment settings object/global object=]'s [=Window/browsing context=].
<li>If the [=top-level browsing context=] does not equal |browsingContext|, abort these steps.
<li>Let |policyDirective| be the result of running [$ parse-policy-directive $] on |acceptCHValue| and the [=url/origin=] of the |metaElement|'s node document.
<li>Let |policyDirective| be the result of running [$PERMISSIONS-POLICY/parse-policy-directive$] on |acceptCHValue| and the [=url/origin=] of the |metaElement|'s node document.
<ol>
<li>For each |feature|->|allowList| of |policyDirective|:
<li>If |feature| is not a [=client hints token=], then continue.
Expand Down Expand Up @@ -219,7 +219,7 @@ following steps:
|request|'s [=environment settings object/client hints set=], [=set/append=] |requestHint| to
|hintSet|.
<li>For each |possibleHint| in |hintSet|, if |request| is a [=subresource request=] and the result of
running [[permissions-policy#algo-should-request-be-allowed-to-use-feature]] given |request| and
running [$should-request-be-allowed-to-use-feature$] given |request| and
|possibleHint|'s associated feature in [[#policy-controlled-features]] returns `false`,
[=list/remove=] |possibleHint| from |hintSet|.
<li>For each |hintName| in |hintSet|, if the |request|'s [=request/header list=] [=header list/does not
Expand All @@ -239,11 +239,11 @@ When asked to <dfn abstract-op>update client hints from redirect if needed</dfn>
<ol>
<li><p>Set <var>hintName</var> to "Sec-" concatenated with <var>hintName</var>.
<li><p>If <var>request</var>'s <a for=request>header list</a> <a for="header list">contains</a>
<var>hintName</var> and if the result of running [=Should request be allowed to use feature?=],
<var>hintName</var> and if the result of running [$PERMISSIONS-POLICY/should-request-be-allowed-to-use-feature$],
given <var>request</var> and <var>hintName</var>’s <a href="#policy-controlled-features">associated
policy-controlled feature</a>, returns <code>false</code>, then remove <var>hintName</var> from |request|'s [=request/header list=].
<li><p>Else if <var>request</var>'s <a for=request>header list</a> doesn't <a for="header list">contain</a>
<var>hintName</var> and if the result of running [=Should request be allowed to use feature?=],
<var>hintName</var> and if the result of running [$PERMISSIONS-POLICY/should-request-be-allowed-to-use-feature$],
given <var>request</var> and <var>hintName</var>’s <a href="#policy-controlled-features">associated
policy-controlled feature</a>, returns <code>true</code>, then add <var>hintName</var> to |request|'s [=request/header list=].
</ol>
Expand Down