1541152 - Disable GV's about:crashes

Status: NEW

(GeckoView :: General, defect, P3)

Description

[Chris Peterson [:cpeterson]]

This issue was originally reported in the Fenix issue tracker:
https://github.com/mozilla-mobile/fenix/issues/1221

Since GV delegates crash reporting to the app, GV's about:crashes doesn't show any crash reports. To avoid user confusion, GV should disable about:crashes. If the app wants to support about:crashes, it can write its own about:crashes handler and show the Socorro and Sentry crash reports it has caught and submitted.

Comment 1

[Chris Peterson [:cpeterson]]

Should we also disable about:telemetry? Fenix issue https://github.com/mozilla-mobile/fenix/issues/1347 says the about:telemetry page crashes GV.

Comment 2

[Chris Peterson [:cpeterson]]

(In reply to Chris Peterson [:cpeterson] from comment #1)

Should we also disable about:telemetry? Fenix issue https://github.com/mozilla-mobile/fenix/issues/1347 says the about:telemetry page crashes GV.

Correction: we don't need to disable about:telemetry because it loads for me. However, GV's about:telemetry will only know about Gecko's telemetry probes. It won't know about any custom Fenix probes. The app side knows about all the Gecko telemetry, so perhaps the app should be responsible for about:telemetry so it can show both app and Gecko telemetry. (Just like the proposal for the app to be responsible for about:crashes so it can show both app and Gecko crash reports.)

Also, Fenix hung (once) when I tried to view the "Raw JSON" link (bug 1542410).

Comment 3

[Chris Peterson [:cpeterson]]

A-C bug to implement (something like) about:crashes:
https://github.com/mozilla-mobile/android-components/issues/1705