Closed Bug 380421 Opened 14 years ago Closed 14 years ago
get crash reporter params from ns
We should be parsing parameters from application.ini instead of crashreporter.ini. We need to get them out of there to allow l10n on that file.
This patch adds a crashReporterURL to nsXULAppInfo, and also adds a flag, NS_XRE_ENABLE_CRASH_REPORTER. They are settable in application.ini like so: [Crash Reporter] Enabled=1 ServerURL=http://example.com/submit The ServerURL gets passed to the crash reporter client via the extra data API. This patch tweaks Firefox's appData settings to have the right server URL, but not to enable crash reporting by default. I kept the MOZ_AIRBAG=1 env var hack for now so we can continue to test Firefox. The patch also cleans up a few warnings from the crash reporter client code.
Attachment #264994 - Flags: review?(benjamin)
Comment on attachment 264994 [details] [diff] [review] get enabled and server url from nsXULAppInfo Please add an envvar check to override the submission URL, and make equivalent nsXULAppInfo changes for Thunderbird.
Attachment #264994 - Flags: review?(benjamin) → review+
Checked in. Need to figure out some tests for this.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.