904131 - Set breakpad.reportURL pref

Status: RESOLVED FIXED

(Firefox Graveyard :: Webapp Runtime, defect)

Description

[Marco Castelluccio [:marco]]

Attachment: fix_crash_reports

Without this pref, crashes aren't shown in about:crashes.

Comment 1

[Myk Melez [:myk] [@mykmelez]]

Does this pref enable crashes in the runtime to show up in about:crashes in Firefox or in the runtime?  And if the latter, then how do you load about:crashes in the runtime, given bug 753801?  The patch itself looks reasonable, but it's unclear how exactly it makes things better.  Some additional details would be helpful here. :-)

Comment 2

[Marco Castelluccio [:marco]]

(In reply to Myk Melez [:myk] [@mykmelez] from comment #1)
> Does this pref enable crashes in the runtime to show up in about:crashes in
> Firefox or in the runtime?  And if the latter, then how do you load
> about:crashes in the runtime, given bug 753801?  The patch itself looks
> reasonable, but it's unclear how exactly it makes things better.  Some
> additional details would be helpful here. :-)

The patch allows the about:crashes page to actually work in the webapp runtime. Without the patch, the page doesn't show anything (actually, it shows this text: "This application has not been configured to display crash reports. The preference <code>breakpad.reportURL</code> must be set.")

We'll still need to allow developers/users to access that page somehow, this patch just allows the page to show something.

Comment 3

[Myk Melez [:myk] [@mykmelez]]

And how do you manage to open that page in the runtime?

Comment 4

[Myk Melez [:myk] [@mykmelez]]

Comment on attachment 789038 [details] [diff] [review]
fix_crash_reports

Marco and I chatted offline, and opening the page in the runtime will still require additional work (currently, he has to modify the runtime to display it), but this is a necessary prerequisite and a useful intermediate step.

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/fx-team/rev/41d9ac6247ce

Comment 6

[Tim Taubert [:ttaubert] (inactive)]

https://hg.mozilla.org/mozilla-central/rev/41d9ac6247ce