-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Split out P3A checkbox from first run dialog on macOS and Linux #14183
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Labels
features/P3A
OS/Desktop
OS/Linux
OS/macOS
priority/P3
The next thing for us to work on. It'll ride the trains.
QA Pass-Linux
QA Pass-macOS
QA/Yes
release-notes/exclude
Milestone
Comments
@simonhong Text update when you get a chance. We'll want to get this uplifted to into 1.21.x |
23 tasks
simonhong
added a commit
to brave/brave-core
that referenced
this issue
Feb 17, 2021
p3a prefs would not be affected by first run dialog. crash report checkbox is unchecked by default. fix brave/brave-browser#14183 fix brave/brave-browser#14160
Merged
7 tasks
Removing |
Verification PASSED on
|
Example |
Example |
Example |
---|---|---|
![]() |
![]() |
![]() |
Test Case #2
(enabled crash reporting)
- launched
1.21.68 Chromium: 89.0.4389.58
and ensured thatHelp improve Brave by automatically sending crash reports
was not being selected by default - selected
Help improve Brave by automatically sending crash reports
and clicked onStart Brave
- ensured that
Automatically send completely private product analytics to Brave
was enabled viabrave://settings/privacy
- ensured that
Help improve Brave's features and performance
was enabled viabrave://settings/privacy
- crashed Brave via
brave://crash
and ensured that the crash report appeared underbrave://crashes
and was automatically submitted - ensured that P3A is working/being updated via
brave://local-state
Example |
Example |
Example |
---|---|---|
![]() |
![]() |
![]() |
Test Case #3
(Onboarding)
- ensured that clicking on
completely private product analytics
opens the following: - ensured that
Settings
opensbrave://settings/privacy
Verification passed on
Brave | 1.21.70 Chromium: 89.0.4389.69 (Official Build) (64-bit) |
---|---|
Revision | 6df9174a477a0bea2118cb4669954d517be16104-refs/branch-heads/4389@{#1332} |
OS | Ubuntu 18.04 LTS |
Verified test plan from brave/brave-core#7978
Test Case #1
(leaving crash report unchecked)
- launched and ensured that
Help improve Brave by automatically sending crash reports
was not being selected by default - ensured that
Automatically send completely private product analytics to Brave
was enabled viabrave://settings/privacy
- ensured that
Help improve Brave's features and performance
was disabled - ensured that P3A is working/being updated via
brave://local-state
Example |
Example |
Example |
---|---|---|
![]() |
![]() |
![]() |
Test Case #2
(enabled crash reporting)
- launched and ensured that
Help improve Brave by automatically sending crash reports
was not being selected by default - selected
Help improve Brave by automatically sending crash reports
and clicked onStart Brave
- ensured that
Automatically send completely private product analytics to Brave
was enabled viabrave://settings/privacy
- ensured that
Help improve Brave's features and performance
was enabled viabrave://settings/privacy
- ensured that P3A is working/being updated via
brave://local-state
Example |
Example |
Example |
---|---|---|
![]() |
![]() |
![]() |
Test Case #3
(Onboarding)
- ensured that clicking on
completely private product analytics
opens the following: - ensured that
Settings
opensbrave://settings/privacy
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
features/P3A
OS/Desktop
OS/Linux
OS/macOS
priority/P3
The next thing for us to work on. It'll ride the trains.
QA Pass-Linux
QA Pass-macOS
QA/Yes
release-notes/exclude
Description
Currently, the first run dialog contains a checkbox that combines P3A and crash reporting consent:

After discussing with the security/privacy/data team about privacy concerns around crash reporting and the longterm vision of obtaining P3A consent, we have decided to separate it out from the crash reports checkbox.
Designs
Separate out P3A from the crash reports checkbox and make sure crash reports is unchecked. Uplift this change for the 1.21x/Mar 2nd release:

Assets
Figma: https://www.figma.com/file/MFHpcXJMg2RUQpUJMAZZ4R/Desktop-Onboarding?node-id=1340%3A999
The text was updated successfully, but these errors were encountered: