Closed Bug 1453238 Opened 7 years ago Closed 7 years ago

Fennec crash pings have the displayVersion field set incorrectly

Categories

(Firefox for Android Graveyard :: Metrics, enhancement)

enhancement
Not set
normal

Tracking

(firefox61 fixed)

RESOLVED FIXED
Firefox 61
Tracking Status
firefox61 --- fixed

People

(Reporter: gsvelto, Assigned: gsvelto)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1452745 +++ We recently added crash pings to Fennec in bug 1407557. The code that assembles the ping however is the same as what we had in the desktop crash reporter client and doesn't populate the displayVersion field correctly [1] so we should apply the a fix similar to bug 1452745. [1] https://searchfox.org/mozilla-central/rev/b55e1a1cbcaee34878e133fbac20c4c2af6e11b5/mobile/android/base/java/org/mozilla/gecko/telemetry/pingbuilders/TelemetryCrashPingBuilder.java#190
Summary: Fennec crash pings frequently have the displayVersion field set incorrectly → Fennec crash pings have the displayVersion field set incorrectly
This would be nice to get in before we branch for 61 beta, as it will help us to track crashes for fennec on that release (and later) via missioncontrol.
Taking this, it's a quick fix.
Assignee: nobody → gsvelto
Status: NEW → ASSIGNED
Comment on attachment 8967137 [details] Bug 1453238 - Populate the crash ping platformVersion and displayVersion fields correctly on Fennec; https://reviewboard.mozilla.org/r/235784/#review241592
Attachment #8967137 - Flags: review?(nchen) → review+
Pushed by gsvelto@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/48f24f61c8b6 Populate the crash ping platformVersion and displayVersion fields correctly on Fennec; r=jchen
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: