1453238 - Fennec crash pings have the displayVersion field set incorrectly

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Metrics, enhancement)

Description

[Gabriele Svelto [:gsvelto]]

+++ 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

Comment 1

[William Lachance (:wlach)]

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.

Comment 2

[Gabriele Svelto [:gsvelto]]

Taking this, it's a quick fix.

Comment 3

[Gabriele Svelto [:gsvelto]]

Attachment: Bug 1453238 - Populate the crash ping platformVersion and displayVersion fields correctly on Fennec;

Review commit: https://reviewboard.mozilla.org/r/235784/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/235784/

Comment 4

[(inactive) Jim Chen [:jchen] [:darchons]]

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

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/48f24f61c8b6
Populate the crash ping platformVersion and displayVersion fields correctly on Fennec; r=jchen

Comment 6

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/48f24f61c8b6