Boris Zbarsky [:bzbarsky] Assignee
	Description • 8 years ago

Because we don't seem to have one right now... Note, though, that this test assumes Promise.prototype.then acts like most Web IDL APIs that take callbacks in terms of its incumbent global handling.

	Bobby Holley (:bholley)
	Comment 2 • 8 years ago

Comment on attachment 8732880 [details] [diff] [review]
Add a test for incumbent global handling when invoking promise callbacks

Review of attachment 8732880 [details] [diff] [review]:
-----------------------------------------------------------------

It seems like this should either be a WPT, or be simplified to use a SpecialPowers API to retrieve the incumbent more simply. r=me though.

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 3 • 8 years ago

Good point; I just wasn't sure how to do multiple hostnames with wpt and didn't find it in the docs offhand.  James, is there something that can make that work easily?

	James Graham [:jgraham]
	Comment 4 • 8 years ago

wpt can do multiple subdomains, but not multiple top level host names at present (there is a feature request for the latter, but it's a little bit of work). I think you need the latter, but if you do in fact only need the former, there are various precanned ones and you can do something like {{domains[www]}} in a .sub.html file to get the full www.web-platform.test (or www.w3c-test.org, or whatever) name.

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 5 • 8 years ago

> I think you need the latter

No, I just need two hostnames that can set document.domain to the same value.  A domain and its subdomain, or two subdomains of the same domain, would both be just fine.

I'll give that a shot.

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 6 • 8 years ago

James, I don't understand what mach web-platform-tests-create did to the manifest here... Can you double-check it?

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 7 • 8 years ago

Comment on attachment 8733434 [details] [diff] [review]
Now as a wpt

James said on IRC this change was just removing leftover stuff that should have been gone anyway..

	Pulsebot
	Comment 8 • 8 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/0dc03a76f51b

	Carsten Book [:Tomcat]
	Comment 9 • 8 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/0dc03a76f51b