-
Notifications
You must be signed in to change notification settings - Fork 3k
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
WebKit export of https://bugs.webkit.org/show_bug.cgi?id=257397 #40325
WebKit export of https://bugs.webkit.org/show_bug.cgi?id=257397 #40325
Conversation
This patch has been exported from WebKit; it will be approved automatically once the downstream patch is r+. |
@@ -13,10 +14,10 @@ | |||
<script> | |||
test(function() { | |||
var client = new XMLHttpRequest(), | |||
urlstart = 'www1.'+location.host + location.pathname.replace(/\/[^\/]*$/, '/') | |||
urlstart = get_host_info().REMOTE_ORIGIN + location.pathname.replace(/\/[^\/]*$/, '/') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need for double space after equals sign, but also no need to address. This whole file looks kinda bad anyway :-)
@@ -17,7 +17,7 @@ | |||
{ | |||
var xhr = new XMLHttpRequest(); | |||
|
|||
xhr.open("POST", "http://nonexistent.{{host}}:{{ports[http][0]}}", false); | |||
xhr.open("POST", "http://{{host}}:3", false); // Bad port. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
3 is fine per https://fetch.spec.whatwg.org/#bad-port. Use 1?
91e3128
to
1bcb46a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The review process for this patch is being conducted in the WebKit project.
Use get-host-info.sub.js to make the tests work with the WebKit test infrastructure.