1792916 - Crash reporter: Allow including page address

Status: NEW

(Firefox for Android :: Crash Reporting, task, P2)

Description

[Christian Sadilek [:csadilek]]

From github: https://github.com/mozilla-mobile/android-components/issues/3260.

Fennec allows users to attach the page address to crash reports.

That has been a helpful feature for debugging Gecko(View) crashes:
https://bugzilla.mozilla.org/show_bug.cgi?id=1555108

┆Issue is synchronized with this Jira Task

Change performed by the Move to Bugzilla add-on.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:cpeterson, could you have a look please?

For more information, please visit auto_nag documentation.

Comment 2

[Chris Peterson [:cpeterson]]

gsvelto says our Android code just needs to add the appropriate crash annotations (URL, Comments) when sending the crash report:

https://searchfox.org/mozilla-central/rev/416f31a51174620f04fc994d248b664b54517699/mobile/android/geckoview/src/main/java/org/mozilla/geckoview/CrashReporter.java#137

The existing code is deliberately stripping out the URL key at the moment because we don't have UI to ask for user permission to share URLs:

https://searchfox.org/mozilla-central/rev/416f31a51174620f04fc994d248b664b54517699/mobile/android/geckoview/src/main/java/org/mozilla/geckoview/CrashReporter.java#45
https://searchfox.org/mozilla-central/rev/416f31a51174620f04fc994d248b664b54517699/mobile/android/geckoview/src/main/java/org/mozilla/geckoview/CrashReporter.java#53-54

We will need to add update the crash reporter UI to ask for user opt-in and maybe show the URL or (since URLs can be long and ugly) just the domain name.