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

Start of port of WebKit CORS tests #116

Closed
wants to merge 1 commit into from

Conversation

odinho
Copy link
Contributor

@odinho odinho commented May 23, 2013

No description provided.

@sideshowbarker
Copy link
Contributor

@velmont another member of the one-year-plus clube

@sideshowbarker
Copy link
Contributor

I notice that there are merge conflicts for the PR. Given that the PR is more than a year old already anyway I wonder whether it should be closed and a new conflict-less PR should opened for it instead.

@odinho
Copy link
Contributor Author

odinho commented Jul 8, 2014

I should take a look through them and see if they make any sense. If there's cases tested here that currently is not in the CORS-suite we have, I could port that over.

//log(xhr.responseText);
};

test(accessControlBasicAllowAccessControlOriginHeader, "access-control-basic-allow-access-control-origin-header")
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Already have

@odinho
Copy link
Contributor Author

odinho commented Jul 8, 2014

Nope. These tests are extremely basic, and we already test all the cases in better detail.

Closed.

@odinho odinho closed this Jul 8, 2014
@tobie tobie added xhr fetch and removed xhr labels Feb 21, 2017
jugglinmike pushed a commit to bocoup/wpt that referenced this pull request Apr 11, 2017
…les-and-globals

Improve the handling of unforgeable interfaces and Global/PrimaryGlobal interfaces, for which operations don't live where they do for normal interfaces; r=Ms2ger
jgraham added a commit that referenced this pull request Apr 21, 2017
Small improvements for servodriver executor
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