622370 - "ASSERTION: Must have a principal!"

Status: NEW

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

Description

[Jesse Ruderman]

Attachment: stack trace

###!!! ASSERTION: Must have a principal!: 'mOwner', file netwerk/protocol/wyciwyg/nsWyciwygChannel.cpp, line 279

Any idea what's going on here?

Comment 1

[Boris Zbarsky [:bzbarsky]]

Not given just that stack.  Do we have anything resembling a testcase, even a non-minimal one?

Comment 2

[Jesse Ruderman]

Not yet :(

Thanks, bz.

Comment 3

[Jesse Ruderman]

With a bunch of help from bz, I figured out enough about this bug to make a testcase!

Comment 4

[Jesse Ruderman]

Attachment: patch for debugging

Comment 5

[Jesse Ruderman]

Attachment: testcase (place in ~ and enable popups)

Comment 6

[Jesse Ruderman]

Attachment: stack traces

"ASSERTION: Chrome trying to load wyciwyg URI" fires before "ASSERTION: Must have a principal!".

Comment 7

[Jesse Ruderman]

The problem seems to be useDefaultIcon assuming that if a document is an ImageDocument, then its URL is sane.

http://hg.mozilla.org/mozilla-central/annotate/969691cfe40e/browser/base/content/tabbrowser.xml#l677

Comment 8

[Boris Zbarsky [:bzbarsky]]

Hmm.  Perhaps we should make document.write fail on ImageDocuments.

Comment 9

[Jesse Ruderman]

I guess that would fix bug 622778, too.

Comment 10

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

(In reply to comment #8)
> Hmm.  Perhaps we should make document.write fail on ImageDocuments.
Yes.

Are there other features we should disable on ImageDocuments?

Comment 11

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1472046

Move all DOM bugs that haven’t been updated in more than 3 years and has no one currently assigned to P5.

If you have questions, please contact :mdaly.