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

[WIP] Cross origin wrappers #16501

Closed
wants to merge 13 commits into from
Prev

enumerate should work. but is a dirty hack

  • Loading branch information
avadacatavra committed Jun 15, 2017
commit ff8d2cdbbfc7a9dc7f38b7dd47cb350fde39388f
@@ -286,21 +286,12 @@
*/
//FIXME fails
addTest(function() {
console.log("absurd");
// try {
for (var prop in C) {
// //console.log("this is nonsense");
// console.log(prop);
assert_unreached("Shouldn't have been able to enumerate " + prop + " on cross-origin Window");
}
// } catch (err) {
// console.log(err);
// }
// works
// for (var prop in C.location) {
// assert_unreached("Shouldn't have been able to enumerate " + prop + " on cross-origin Location");
// }
console.log("finish")
for (var prop in C.location) {
assert_unreached("Shouldn't have been able to enumerate " + prop + " on cross-origin Location");
}
}, "[[Enumerate]] should return an empty iterator");


ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.