Comment 209 for bug 25830

Revision history for this message
In , Sylvain Pasche (sylvain-pasche) wrote :

Created an attachment (id=252900)
backend patch, v4 updated

This is an update of the latest patch against trunk. It does not address Darin's comments yet. As one could expect, it did not apply cleanly, so I had to manually merge some stuff.

I had not much success with this new patch applied: If I select for instance text/html for the mime to redispatch, I'm getting assertion:

###!!! ASSERTION: nsExpatDriver didn't get an nsIExpatSink: 'Error', file /[...]/mozilla/parser/htmlparser/src/nsExpatDriver.cpp, line 1180

Then the originating window is staying in loading state (the spinner is running), and if I click on the window, I can see the assertions:

WARNING: NS_ENSURE_TRUE(aPresContext->Document()->GetWindow()) failed: file /[...]/mozilla/content/events/src/nsIMEStateManager.cpp, line 174

When stepping inside the nsDocumentOpenInfo::ReDispatch function, I can see that the
m_targetStreamListener->OnDataAvailable() call is failing and is in the callstack when the nsIExpatSink assertion is thrown.

If you have any hints about this, they are welcome.