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

Snapshot the allowpaymentrequest attribute #2195

Merged
merged 4 commits into from
Jan 14, 2017
Merged

Snapshot the allowpaymentrequest attribute #2195

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
b2e82f1
Snapshot the allowpaymentrequest attribute
zcorpan Dec 16, 2016
edcbb1b
Actually set the flag on the document when it is created
zcorpan Dec 23, 2016
fbfc90a
Move allowusermedia to the snapshot model
zcorpan Jan 12, 2017
c602fe7
simplify warning now allowusermedia is snapshotted
annevk Jan 13, 2017
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
55 changes: 50 additions & 5 deletions source
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28615,14 +28615,55 @@ interface <dfn>HTMLIFrameElement</dfn> : <span>HTMLElement</span> {
<li><p>If <var>document</var>'s <span data-x="concept-document-bc">browsing context</span> is a
<span>top-level browsing context</span>, then return true.</p></li>

<li><p>If <var>document</var>'s <span data-x="concept-document-bc">browsing context</span> has a
<span>browsing context container</span> that is an <code>iframe</code> element with an
<var>allowattribute</var> attribute specified, and whose <span>node document</span> is
<span>allowed to use</span> the feature indicated by <var>allowattribute</var>, then return
true.</p></li>
<li>
<p>If <var>allowattribute</var> is <code
data-x="attr-iframe-allowfullscreen">allowfullscreen</code>, then follow these substeps:</p>

<ol>
<li><p>If <var>document</var>'s <span data-x="concept-document-bc">browsing context</span> has
a <span>browsing context container</span> that is an <code>iframe</code> element with an
<var>allowattribute</var> attribute specified, and whose <span>node document</span> is
<span>allowed to use</span> the feature indicated by <var>allowattribute</var>, then return
true.</p></li>
</ol>

<p class="big-issue">This step is legacy behavior for the <code
data-x="attr-iframe-allowfullscreen">allowfullscreen</code> attribute that should not be used
for new features. If possible this will be removed, see <a
href="https://github.com/whatwg/html/issues/2143#issuecomment-265514585">issue 2143</a>.</p>
</li>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should remove allowusermedia from this message. I'll add a commit.


<li>
<p>Otherwise, follow these substeps:</p>

<ol>
<li><p>If <var>document</var> has the <var><var>allowattribute</var> flag</var> set, and
<var>document</var>'s <span data-x="concept-document-bc">browsing context</span> has a
<span>browsing context container</span> that is an <code>iframe</code> element whose
<span>node document</span> is <span>allowed to use</span> the feature indicated by
<var>allowattribute</var>, then return true.</p></li>
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The nested var here is maybe a bit unclear? Any ideas? Adding "«" and "»" before/after with CSS helps I think...

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wasn't it <var>allow<var>attribute</var> flag</var> elsewhere? That was reasonbly clear I thought.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess I was misremembering. Some CSS solution is probably good; not sure I'd use those quotes though since we use them to denote Infra lists.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

whatwg/resources.whatwg.org#38

</ol>
</li>

<li><p>Return false.</p></li>
</ol>

<p>To <dfn>set the allow* flags</dfn> for a <code>Document</code> <var>document</var> means to
run these steps:</p>

<ol>
<li><p>If <var>document</var>'s <span data-x="concept-document-bc">browsing context</span> has a
<span>browsing context container</span> that is an <code>iframe</code> element, let
<var>iframe</var> be that element. Otherwise, abort these steps.</p></li>

<li><p>If <var>iframe</var> has an <code
data-x="attr-iframe-allowpaymentrequest">allowpaymentrequest</code> attribute specified, then
set the <var>allowpaymentrequest flag</var> on <var>document</var>.</p></li>

<li><p>If <var>iframe</var> has an <code
data-x="attr-iframe-allowusermedia">allowusermedia</code> attribute specified, then set the
<var>allowusermedia flag</var> on <var>document</var>.</p></li>
</ol>
</div>

<hr> <!-- DIM ATTRIBUTES -->
Expand Down Expand Up @@ -76907,6 +76948,8 @@ dictionary <dfn>DragEventInit</dfn> : <span>MouseEventInit</span> {

<li><p><span>Implement the sandboxing</span> for <var>document</var>.</p></li>

<li><p><span>Set the allow* flags</span> for <var>document</var>.</p></li>

<li><p>Set <var>settingsObject</var>'s <span
data-x="concept-environment-execution-ready-flag">execution ready flag</span>.</p></li>

Expand Down Expand Up @@ -82163,6 +82206,8 @@ State: &lt;OUTPUT NAME=I>1&lt;/OUTPUT> &lt;INPUT VALUE="Increment" TYPE=BUTTON O

<li><p><span>Implement the sandboxing</span> for the <code>Document</code>.</p></li>

<li><p><span>Set the allow* flags</span> for the <code>Document</code>.</p></li>

<li><p>Set <var>settingsObject</var>'s <span
data-x="concept-environment-execution-ready-flag">execution ready flag</span>.</p></li>

Expand Down