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.

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

Microsoft's feedback on Feature Policy spec #62

Open
nolanlawson opened this issue Mar 28, 2017 · 3 comments
Open

Microsoft's feedback on Feature Policy spec #62

nolanlawson opened this issue Mar 28, 2017 · 3 comments

Comments

@nolanlawson
Copy link
Member

I spoke with @toddreifsteck and @travisleithead and we're +1 on the spec. It's a good direction for the web, and the explainer is extremely well-thought-out (covers all the odd edge cases with visual diagrams – love it! 😃).

My only other feedback is that while the framework itself is great, the lists of features could use some elaboration. For instance, I'd like to see more features related to performance optimizations, ala @tkadlec's and @yoavweiss's Content Performance Policy. Disallowing sync XHR is a good start, but how about more perf-boosting restrictions, like:

  • no document.write()
  • no touch/wheel events without passive=true
  • no sync storage
  • no animating top/left/etc., only transform/opacity
  • no excessive callbacks (e.g. a per-second budget on setTimeouts, rAFs, potentially others)
  • other ideas?

Arguably this could be kept to a separate repo, but it's something I'd like to start thinking about.

@zcorpan
Copy link
Member

zcorpan commented Mar 28, 2017

no animating top/left/etc., only transform/opacity

It's not clear that this makes sense long term. c.f. https://air.mozilla.org/bay-area-rust-meetup-february-2016/#@25m50s

@nolanlawson
Copy link
Member Author

Good point, maybe it should be defined more loosely then. Just an idea! :)

@raymeskhoury
Copy link

raymeskhoury commented Apr 4, 2017

@nolanlawson thanks so much for the feedback. I think we're all definitely on board with the idea of expanding the feature set. Specifically document.write is one that folks here have had in mind for a while :) There are some things to work out with respect to the failure scenarios, i.e. what happens if you try to use document.write if it's disabled. There are some security concerns around breaking applications in unexpected ways. The features currently in the spec already have well-defined failure modes. But it's not a major blocker, just something that has to be fleshed out a bit more.

@pabrai pabrai added the feedback label May 8, 2019
@pabrai pabrai added this to Feedback in FP Engagement May 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

5 participants