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

Fetch: add return statement to cors-preflight-status.any.js #5230

Merged
merged 1 commit into from
Mar 27, 2017

Conversation

annevk
Copy link
Member

@annevk annevk commented Mar 27, 2017

Supersedes #5228.

@wpt-pr-bot
Copy link
Collaborator

Notifying @jdm, @mnot, and @youennf. (Learn how reviewing works.)

@annevk
Copy link
Member Author

annevk commented Mar 27, 2017

This fixes #5138.

@jugglinmike
Copy link
Contributor

Here's hoping we can avoid this kind of mistake in the future :) w3c/testharness.js#252

@ghost
Copy link

ghost commented Mar 27, 2017

View the complete job log.

Firefox (nightly channel)

Testing web-platform-tests at revision 0b59e712dcacd856a621d93cc6edff7ba383b19e
Using browser at version BuildID 20170326110231; SourceStamp f5e214144799889e2408c4841351f4053f00544e
Starting 10 test iterations
No tests run.

@ghost
Copy link

ghost commented Mar 27, 2017

View the complete job log.

Chrome (unstable channel)

Testing web-platform-tests at revision 0b59e712dcacd856a621d93cc6edff7ba383b19e
Using browser at version 59.0.3047.0 dev
Starting 10 test iterations
No tests run.

@jdm jdm merged commit 2c544f0 into master Mar 27, 2017
@annevk annevk deleted the annevk/cors-preflight-status branch March 27, 2017 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants