1185360 - PostMessageEvent should not have a different behavior if the main principal subsumes the destination one.

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Andrea Marchesini [:baku]]

Attachment: subsumes.patch

Comment 1

[Andrea Marchesini [:baku]]

Attachment: subsumes.patch

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8635807 [details] [diff] [review]
subsumes.patch

We need some tests here - and as far as I see, this doesn't work.
We end up using the same FileList object on both sides, but we should clone.
And that would just cause security exception to be thrown on the receiving side since it can't access the wrapper from the sending side if those are cross-origin.

Comment 3

[Andrea Marchesini [:baku]]

> We need some tests here - and as far as I see, this doesn't work.
> We end up using the same FileList object on both sides, but we should clone.
> And that would just cause security exception to be thrown on the receiving
> side since it can't access the wrapper from the sending side if those are
> cross-origin.

About FileList, all is fixed in bug 1185381. Here it's just about the subsumes thing.
Which kind of test do you want to see?

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

That we actually get a clone when passing FileList, and also the contained Files should be cloned.

Comment 5

[Andrea Marchesini [:baku]]

I added the test in bug 1185360. Can you take a look at this patch again when you have time? Thanks

Comment 6

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8635807 [details] [diff] [review]
subsumes.patch

Ok, so this must not land before the patch which makes FileList cloning work properly, but at the same time.

Comment 7

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/a9af8583cd34

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/a9af8583cd34