690639 - "ASSERTION: Content should not be calling this" (nsIndexedToHTML::DoOnStartRequest calling nsChromeRegistryContent::IsLocaleRTL)

Status: NEW

(Core :: Networking: File, defect, P5)

Description

[Jesse Ruderman]

Attachment: testcase (must be local)

Loading a directory listing in Fennec (on Linux) hits

###!!! ASSERTION: Content should not be calling this: 'Not Reached', file /home/jruderman/mozilla-central/chrome/src/nsChromeRegistryContent.cpp, line 245

because nsIndexedToHTML::DoOnStartRequest calls nsChromeRegistryContent::IsLocaleRTL.

Comment 1

[Jesse Ruderman]

Attachment: stack trace

Comment 2

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Mmm this looks like a bad one.

Maybe we should use the same hack as in bug 576507.  Or if the fennec frontend deals with this another way, maybe we should move that into gecko here.

nsIndexedToHTML::DoOnStartRequest looks scary and I don't know what it does (is it trying to build a directory listing?), so moving to XPCOM.  May be better elsewhere.

Comment 3

[Boris Zbarsky [:bzbarsky]]

nsIndexedToHTML is something that takes a directory listing in a non-HTML format (I believe a format that both the FTP channel and file channel produce for directory listings) and produces some HTML the browser can render.

Comment 4

[Mark Finkle (:mfinkle) (use needinfo?)]

(In reply to Chris Jones [:cjones] [:warhammer] from comment #2)
> Mmm this looks like a bad one.
> 
> Maybe we should use the same hack as in bug 576507.  Or if the fennec
> frontend deals with this another way, maybe we should move that into gecko
> here.

The front-end doesn't handle things like this. It's a slipperly slope, but extending the hack in bug 576507 would work OK.

Comment 5

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Does fennec not support RTL locales then?

Comment 6

[(no longer active)]

(In reply to Chris Jones [:cjones] [:warhammer] from comment #5)
> Does fennec not support RTL locales then?

I think it does.

Comment 7

[Firefox Bug Husbandry Bot]

Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258