Crash reporter is only partially localized

VERIFIED FIXED in Firefox 35



Crash Reporting
4 years ago
3 years ago


(Reporter: flod, Assigned: rstrong)



Mac OS X
Dependency tree / graph

Firefox Tracking Flags

(firefox35 verified)



(3 attachments, 2 obsolete attachments)



4 years ago
Created attachment 8502336 [details]
Screenshot of crash reporter (Italian build)

Firefox crashed this morning, and Crash Reporter it's a mess of mostly untranslated strings (2 strings are translated).

Last time Firefox crashed (a month ago), I didn't notice anything strange in the window.

Current build: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:35.0) Gecko/20100101 Firefox/35.0

Comment 1

4 years ago
Can you check if /Applications/ is localized for you?

I confess, I didn't pay attention to the UI in my last few crashes.

Comment 2

4 years ago
(related to bug 1047584? I see at least a couple of Crash Reporter bugs depending from that)

/Applications/ is fully localized.
If someone could point me in the general direction of where the localization happens, I'd be happy to take a look.
(In reply to Francesco Lodolo [:flod] from comment #2)
> (related to bug 1047584? I see at least a couple of Crash Reporter bugs
> depending from that)
> /Applications/
> Contents/Resources/crashreporter.ini is fully localized.

I'm assuming (hoping) this was meant to be:

(Note the "MacOS" instead of "Resources" in the path)
Flags: needinfo?(francesco.lodolo)

Comment 5

4 years ago
There's a in both MacOS and Resources.

The one in Resources has a localized crashreporter.ini

The one in MacOS has another crashreporter.ini which is completely in English
Flags: needinfo?(francesco.lodolo)
Okay, I think we've figured out what's going on. $(FINAL_TARGET) is currently pointing to Contents/Resources[1] but should point to Contents/MacOS. This makes it appear as if there was an additional under However, through inspection of the package it can be seen that there's actually only a localized crashreporter.ini under There are no binaries or other files in this .app bundle, hence it's not a valid bundle.

Created attachment 8502898 [details] [diff] [review]
patch rev1
Assignee: nobody → robert.strong.bugs
Created attachment 8502922 [details]
screenshot with patch (Italian build)
Attachment #8502898 - Attachment is obsolete: true
I'm going to verify the path does the right thing when crash reporter is disabled before requesting review.
Blocks: 1047584
No longer blocks: 928397
Created attachment 8502927 [details] [diff] [review]
patch - fix indentation
Attachment #8502925 - Attachment is obsolete: true
Attachment #8502927 - Flags: review?(ted)
I verified that everything works with packaging and l10n repacks with crash reporter enabled and disabled.

Comment 13

4 years ago
(In reply to Robert Strong [:rstrong] (use needinfo to contact me) from comment #8)
> Created attachment 8502922 [details]
> screenshot with patch (Italian build)

For those curious, the truncated checkbox is bug 528800.
Attachment #8502927 - Flags: review?(ted) → review+
Pushed to mozilla-inbound
Target Milestone: --- → mozilla35
Last Resolved: 4 years ago
Resolution: --- → FIXED
Landed on aurora in the Mac V2 signing combined patch in bug 1047584
status-firefox35: --- → fixed
Keywords: verifyme
Tested on a few l10n builds that the crashreporter is localized (zh-TW, ko, pt-BR, de, es-ES, fr, ja, pl, ru) using latest Aurora/Developer Edition on Mac OS X 10.9.5.
status-firefox35: fixed → verified


3 years ago
Depends on: 1115892
You need to log in before you can comment on or make changes to this bug.