486826 - firefox isn't using its overridden netError.dtd

Status: VERIFIED FIXED

(Firefox Build System :: General, defect)

Description

[Magnus Melin [:mkmelin]]

Working on bug 483011 in surprisingly notice that firefox isn't actually using it's overridden netError.dtd, anymore! 

Looks like fallout from bug 411037, at least 
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/d049116865bd

Should probably be

% override chrome://global/locale/netError.dtd chrome://browser/locale/netError.dtd

% override chrome://global/locale/netErrorApp.dtd chrome://browser/locale/netError.dtd

At least making that corresponding change worked for my thunderbird patch.

---

STR: 
1) File | Work offline
2) load a page not in cache
3) See the text

Results: not the text from http://mxr.mozilla.org/comm-central/source/mozilla/browser/locales/en-US/chrome/overrides/netError.dtd#55 but from http://mxr.mozilla.org/comm-central/source/mozilla/dom/locales/en-US/chrome/netError.dtd#28

Comment 1

[Mike Beltzner [:beltzner, not reading bugmail]]

Is this Linux only?

Comment 2

[Mike Beltzner [:beltzner, not reading bugmail]]

http://www.grabup.com/uploads/55d49b6fc47fccac3cdefd32c2dc9ace.png is what I see when I do this in Firefox on OSX, fwiw.

Comment 3

[Reed Loden [:reed]]

(In reply to comment #1)
> Is this Linux only?

Not according to your screenshot, no.

Comment 4

[Johnathan Nightingale [:johnath]]

(In reply to comment #0)
> Should probably be
> 
> % override chrome://global/locale/netError.dtd
> chrome://browser/locale/netError.dtd
> 
> % override chrome://global/locale/netErrorApp.dtd
> chrome://browser/locale/netError.dtd
> 
> At least making that corresponding change worked for my thunderbird patch.

This is understandably confusing, but is fall out from Kairo's work in bug 403800.

The override here:

http://mxr.mozilla.org/mozilla-central/source/browser/locales/jar.mn#78

*is* correct, in that it says "replace the global netErrorApp.dtd, a file which contains supplemental strings likely to be replaced by implementors, with our own file - confusingly named netError.dtd instead of netErrorApp.dtd."

Given that firefox's version of netError.dtd contains many of the strings in the global netError.dtd, and given that the inclusion of "netErrorApp.dtd" happens at the bottom of netError.dtd ( http://mxr.mozilla.org/mozilla-central/source/dom/locales/en-US/chrome/netError.dtd#86 ) this has the effect of "redefining" many of the entities.

I've never checked up on what "proper DTD" behaviour is on duplicate entities, but it seems like we're seeing a problem here, and that might well be it.

CC'ng kairo, since this traces back to his patch.

Comment 5

[Mike Beltzner [:beltzner, not reading bugmail]]

I don't think this blocks, but we'd take a fix.

Comment 6

[Johnathan Nightingale [:johnath]]

Attachment: Fix the override

Magnus, re-reading my comment 4, I realize I was ambiguous.  When I said the override was "correct," I should have said it was "deliberate," in that it was intentional to separate out the "likely to be overridden" strings from the "likely to be just fine" ones.  However, as you've noted, Firefox provides substantially different text for most of the errors.

This one liner patch, as you suggest, replaces our partial override with a complete one.

I think this should land in 1.9.1 - these are the strings we intend to be showing. I'll copy l10n on this bug so that they know it's here, but no strings are changing here in the tree - we're just correctly using the ones in browser/ again.

Comment 7

[Johnathan Nightingale [:johnath]]

Landed on mozilla-central 

http://hg.mozilla.org/mozilla-central/rev/6f49c04c3a6c

Comment 8

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 371447 [details] [diff] [review]
Fix the override

a191=beltzner

Comment 9

[Johnathan Nightingale [:johnath]]

http://hg.mozilla.org/releases/mozilla-1.9.1/rev/a9b44083c2e3

Comment 10

[Aakash Desai [:aakashd]]

verified FIXED on builds:


Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2a1pre) Gecko/20090514 Minefield/3.6a1pre ID:20090514031229

and

Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1b5pre) Gecko/20090514 Shiretoko/3.5b5pre ID:20090514031203