Closed Bug 380421 Opened 14 years ago Closed 14 years ago

get crash reporter params from nsXULAppInfo

Categories

(Toolkit :: Crash Reporting, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: ted, Assigned: ted)

References

Details

Attachments

(1 file)

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
Flags: in-testsuite?
Resolution: --- → FIXED
Blocks: 380906
You need to log in before you can comment on or make changes to this bug.