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

Html coop parsing #12

Closed
wants to merge 3 commits into from
Closed

Html coop parsing #12

wants to merge 3 commits into from

Conversation

jugglinmike
Copy link
Member

No description provided.

@jugglinmike
Copy link
Member Author

Use same-origin instead of same-site since the latter should not be
supported.

(via web-platform-tests@bb1376b#commitcomment-36480961)

Definitely

As far as I can tell, it's possible to get duplicated headers with a
.headers file. I think that is nicer than .asis.

(via web-platform-tests@bb1376b#commitcomment-36481209)

And I think .asis is nicer :) We won't be able to move forward without more
substantive rationale, so I'll start.

The benefit of an .asis file is that all relevant information is expressed in
one place. Using .headers means that readers need to open two files to
understand the intent of a given file, and that assumes they even know to look.

@zcorpan
Copy link

zcorpan commented Dec 19, 2019

.headers is more used in general, so people need to be aware of them anyway. Using .asis also means people need to learn how that works, too.

However, I think either is OK, so we should just submit this as-is (ahem) 😊

@jugglinmike
Copy link
Member Author

Cool. Here it is: web-platform-tests#20873

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants