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

NEW
Unassigned

Status

()

Core
Networking: File
P5
normal
6 years ago
2 months ago

People

(Reporter: Jesse Ruderman, Unassigned)

Tracking

(Blocks: 1 bug, {assertion, testcase})

Trunk
x86_64
Linux
assertion, testcase
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [necko-would-take])

Attachments

(2 attachments)

(Reporter)

Description

6 years ago
Created attachment 563635 [details]
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.
(Reporter)

Comment 1

6 years ago
Created attachment 563636 [details]
stack trace
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.
Component: IPC → XPCOM
QA Contact: ipc → xpcom
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.
(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.

Updated

6 years ago
Component: XPCOM → Networking: File
QA Contact: xpcom → networking.file
Does fennec not support RTL locales then?

Comment 6

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

I think it does.
Whiteboard: [necko-would-take]
Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258
Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.