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

Jump to bottom

Update the spec: include iframe header and relaxed filtering #147

Merged
merged 2 commits into from
Mar 30, 2023
Merged

Update the spec: include iframe header and relaxed filtering #147

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
e5e1d06
Update the spec to include new behaviors
xyaoinum Mar 29, 2023
44ffb81
incorporate review feedback
xyaoinum Mar 30, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Prev Previous commit
incorporate review feedback
  • Loading branch information
@xyaoinum
xyaoinum committed Mar 30, 2023
commit 44ffb81a0cc0779ec7312dddb96ff3eb26e63c3a
9 changes: 3 additions & 6 deletions spec.bs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -417,13 +417,10 @@ spec: html; urlPrefix: https://www.rfc-editor.org/rfc/
<span class=XXX>TODO: make the modification directly to the fetch spec.</span>

<h3 id="browsing-topics-content-attribute-for-iframe-element-header">browsingtopics content attribute for HTMLIframeElement</h3>
The <a href="https://html.spec.whatwg.org/multipage/iframe-embed-object.html#the-iframe-element">iframe</a> element contains a <dfn for="content attribute">browsingtopics</dfn> <a spec=html>content attribute</a>. The IDL attribute `browsingTopics` <a spec=html>reflects</a> the [=content attribute/browsingtopics=] <a spec=html>content
attribute</a>.
The <a href="https://html.spec.whatwg.org/multipage/iframe-embed-object.html#the-iframe-element">iframe</a> element contains a <dfn element-attr for="iframe">browsingtopics</dfn> <a spec=html>content attribute</a>. The IDL attribute <dfn attribute for="HTMLIFrameElement">browsingTopics</dfn> <a spec=html>reflects</a> the <{iframe/browsingtopics}> <a spec=html>content attribute</a>.

<pre class=idl>
interface HTMLIFrameElement : HTMLElement {
// existing attributes ...

partial interface HTMLIFrameElement {
[CEReactions] attribute boolean browsingTopics;
};
</pre>
Expand Down Expand Up @@ -453,7 +450,7 @@ spec: html; urlPrefix: https://www.rfc-editor.org/rfc/
<h3 id="modification-to-create-navigation-params-by-fetching-steps-header">Modification to "create navigation params by fetching" steps</h3>
The following step will be added to the <a href="https://html.spec.whatwg.org/multipage/browsing-the-web.html#create-navigation-params-by-fetching">create navigation params by fetching steps</a>, after step "Let |request| be a new [=Request/request=], with ...":

1. If <var ignore=''>navigable</var>'s [=container=] is an <a href="https://html.spec.whatwg.org/multipage/iframe-embed-object.html#the-iframe-element">iframe</a> element, and if it has <a spec=html>content attribute</a> [=content attribute/browsingtopics=], then set |request|'s [=request/send browsing topics header boolean=] to true.
1. If <var ignore=''>navigable</var>'s [=container=] is an <{iframe}> element, and if it has a <{iframe/browsingtopics}> <a spec=html>content attribute</a>, then set |request|'s [=request/send browsing topics header boolean=] to true.

<span class=XXX>TODO: make the modification directly to the html spec.</span>

Expand Down