Re: [mediacapture-main] Iframe sandboxing options for gUM

I believe fullscreen is made that way because it preexists sandbox for
 some impl (which seems to be Gecko), and later other impls adopted 
that. IIRC, Gecko had a plan about converting `allowfullscreen` 
attribute to a sandbox flag, but never made it. (I failed to find 
that, though)

Given this, I suppose `allowfullscreen` is just an exception which we 
do not want to have any follower. New things should all go to sandbox 
rather than adding another `allowfullscreen`-like attribute.

Some reasons why I think we should prefer the sandbox way:

1. Using attribute makes logic unnecessarily more complicated. e.g. 
how should the new attribute interact with the sandbox mechanism? The 
logic was broken for fullscreen in the spec, and now we are going to 
completely decouple them.
2. Adding as a sandbox token allows external specs to make use of it. 
e.g. CSP spec reuses tokens of `sandbox` attribute to control the 
ability of even top level document. 

-- 
GitHub Notification of comment by upsuper
Please view or discuss this issue at 
https://github.com/w3c/mediacapture-main/issues/268#issuecomment-230465078
 using your GitHub account

Received on Tuesday, 5 July 2016 12:32:30 UTC