Skip to content

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

Merged
jdm merged 1 commit intomasterfrom
annevk/cors-preflight-status
Mar 27, 2017
Merged

Fetch: add return statement to cors-preflight-status.any.js#5230
jdm merged 1 commit intomasterfrom
annevk/cors-preflight-status

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