1399170 - Write test to make sure that data: URIs work together correctly with mozbrowser

Status: RESOLVED FIXED

(Core :: DOM: Security, enhancement, P2)

Comment 1

[Christoph Kerschbaumer [:ckerschb]]

Attachment: bug_1399170_mozbrowser_test.patch

Hey Smaug, since you are already familiar with my changes, I guess it makes sense to wait till you are back from PTO.
Whenver you get a chance, could you please review this patch? Thanks.

Comment 2

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

Comment on attachment 8908134 [details] [diff] [review]
bug_1399170_mozbrowser_test.patch

This is fine, but I'd like to see also a test where first non-data url is loaded to mozbrowser and then one tries to load data url

Comment 3

[Christoph Kerschbaumer [:ckerschb]]

Attachment: bug_1399170_mozbrowser_test.patch

(In reply to Olli Pettay [:smaug] from comment #2)
> This is fine, but I'd like to see also a test where first non-data url is
> loaded to mozbrowser and then one tries to load data url

I added a second test - I suppose this is what you meant, right?

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9072e9c0d0aa
Test mozbrowser can not load data URI in iframe. r=smaug

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/9072e9c0d0aa