Settings for crash reporting

[vesta0]

User Story
As a user I have the option to enable/disable crash reporting in settings.

Dependencies (Added by PM and Eng)

Next Steps

• Immediate task: Estimate/size user story with points (during backlog grooming)
• Engineers need to breakdown the user story, size all the individual tasks, and link the tasks to this user story (Done by Eng on Sprint Planning day)

Acceptance Criteria (Added by PM)

• I can enable/disable crash reporting in settings.
• When I have disabled crash reporting, no reports are sent to Sentry
• When I have disabled crash reporting, no reports are sent to Socorro
• When I have disabled crash reporting, Fenix will never ask me to send a crash report

┆Issue is synchronized with this Jira Task

[sblatz]

To confirm what this means:

Should this box just be unchecked if the user has switched their data preference for crash reporting off?

[sv-ohorvath]

There is also a checkbox on the app crashed dialog:

This should be removed/un-checked when the crash reporter is turned off.
*Note: there is no report sent to crash-stats.mozilla.com even if it's selected.

[sblatz]

@sv-ohorvath I will create a follow-up bug for this.

[sv-ohorvath]

Ok, so I can close this. Thanks!

[sblatz]

@brampitoyo I think we will need to add the checkbox back and just default it to the state the user has in the settings (or remove the setting altogether).

Unfortunately removing the checkbox is not a trivial change as I anticipated, and after spending over a day working on it I discovered an issue with the way crash reporting is handled by GV/A-C that makes this change incredibly difficult and time consuming.

I am also concerned that user would be sending crash reports without knowing it since it would be buried deep in the settings, rather than asking users on each prompt since the data could be identifiable.

Please let me know if you'd like to chat about this more!

[brampitoyo]

@sblatz wrote:

Unfortunately removing the checkbox is not a trivial change as I anticipated

Adding the checkbox back in is fine by me. I have updated the mockup to suit: https://mozilla.invisionapp.com/share/P7R8W1ZWHM6