1855694 - Only send the remote type prefix

Status: RESOLVED FIXED

(Firefox for Android :: Experimentation and Telemetry, task)

Description

[William Durand [:willdurand]]

To make sure we're sending the same remote type values as the (legacy?) crash annotations, we should also trim the values in the ChildCrashHandler like it's done here: https://searchfox.org/mozilla-central/rev/6b91b922725838e2732aeb478b13e5b33e33ce1b/dom/ipc/ContentParent.cpp#726-735

Comment 1

[William Durand [:willdurand]]

Attachment: Bug 1855694 - Only send the remote type prefix. r?gsvelto!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/51fa8a3c99f0
Only send the remote type prefix. r=gsvelto,geckoview-reviewers,amejiamarmol

Comment 3

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/51fa8a3c99f0

Comment 4

[William Durand [:willdurand]]

Comment on attachment 9355566 [details]
Bug 1855694 - Only send the remote type prefix. r?gsvelto!

Beta/Release Uplift Approval Request

• User impact if declined: None. This is a fix to a Glean probe.
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: Yes
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): This is low risk because the patch has automated tests and it only affects crash pings. The change is to trim a value to only report remote type prefixes instead of full values.
• String changes made/needed:
• Is Android affected?: Yes

Comment 5

[Dianna Smith [:diannaS]]

Comment on attachment 9355566 [details]
Bug 1855694 - Only send the remote type prefix. r?gsvelto!

Approved for 119.0b5

Comment 6

[Pulsebot]

https://hg.mozilla.org/releases/mozilla-beta/rev/e5bffcfa00f5