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

Ensure templated tests do set a Content-Type: text/html HTTP header #12450

Merged
merged 1 commit into from Aug 13, 2018

Conversation

youennf
Copy link
Contributor

@youennf youennf commented Aug 13, 2018

No description provided.

Copy link
Member

@gsnedders gsnedders left a comment

Choose a reason for hiding this comment

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

This LGTM, though I'm sad about the lack of tests. If it's blocking you I guess we shouldn't block everything on someone figuring out how to test this… 🙁

@gsnedders gsnedders merged commit 36c74ce into web-platform-tests:master Aug 13, 2018
hubot pushed a commit to WebKit/WebKit-http that referenced this pull request Aug 13, 2018
…ader

https://bugs.webkit.org/show_bug.cgi?id=188523

Reviewed by Alexey Proskuryakov.

Importing change made in web-platform-tests/wpt#12450
This helps running WPT tests in more platforms.

* web-platform-tests/tools/serve/serve.py:
(HtmlWrapperHandler):


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@234827 268f45cc-cd09-0410-ab3c-d52691b4dbfc
ryanhaddad pushed a commit to WebKit/WebKit that referenced this pull request Dec 22, 2020
…ader

https://bugs.webkit.org/show_bug.cgi?id=188523

Reviewed by Alexey Proskuryakov.

Importing change made in web-platform-tests/wpt#12450
This helps running WPT tests in more platforms.

* web-platform-tests/tools/serve/serve.py:
(HtmlWrapperHandler):


Canonical link: https://commits.webkit.org/203625@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@234827 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants