1514238 - [api] RuntimeTelemetry#getSnapshots should return a public type

Status: RESOLVED FIXED

(GeckoView :: General, enhancement, P1)

Description

[[ex-Mozilla] Agi Sferro | :agi]

RuntimeTelemetry#getSnapshots returns a GeckoBundle which is not something we want to expose in our API. We should figure out a way to return a public type.

Comment 1

[[ex-Mozilla] Agi Sferro | :agi]

Focus uses it here [0] and immediately transforms it to a JSON, maybe returning a JSON would be good enough? I'm not sure how a-c uses it (or wants to use it) yet.

[0]: https://github.com/mozilla-mobile/focus-android/blob/1c640a58ce8229159ddbea8b973e16b59299f4b3/app/src/main/java/org/mozilla/focus/web/GeckoWebViewProvider.kt#L671

Comment 2

[[ex-Mozilla] Agi Sferro | :agi]

Attachment: Bug 1514238 - Make RuntimeTelemetry.getSnapshot return a JSONObject.

Depends on D26026

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/91a004f3826f
Make RuntimeTelemetry.getSnapshot return a JSONObject. r=esawin,snorp

Comment 4

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/91a004f3826f