Skip to content

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

Closed
karenkliu opened this issue Feb 16, 2021 · 3 comments · Fixed by brave/brave-core#7978
Closed

Split out P3A checkbox from first run dialog on macOS and Linux #14183

karenkliu opened this issue Feb 16, 2021 · 3 comments · Fixed by brave/brave-core#7978

Comments

@karenkliu
Copy link

karenkliu commented Feb 16, 2021

Description

Currently, the first run dialog contains a checkbox that combines P3A and crash reporting consent:
image

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:
image

Assets

Figma: https://www.figma.com/file/MFHpcXJMg2RUQpUJMAZZ4R/Desktop-Onboarding?node-id=1340%3A999

@rebron rebron added the priority/P3 The next thing for us to work on. It'll ride the trains. label Feb 16, 2021
@rebron rebron changed the title Split out P3A checkbox from first run dialog on MacOS and Linux Split out P3A checkbox from first run dialog on macOS and Linux Feb 16, 2021
@rebron
Copy link
Collaborator

rebron commented Feb 16, 2021

@simonhong Text update when you get a chance. We'll want to get this uplifted to into 1.21.x

@LaurenWags
Copy link
Member

Removing QA/Blocked label, https://github.com/brave/brave-browser/releases/tag/v1.21.68 is available

@stephendonner
Copy link

stephendonner commented Feb 25, 2021

Verification PASSED on macOS 11.1 x64 using the following build:

Brave 1.21.68 Chromium: 89.0.4389.58 (Official Build) (x86_64)
Revision 1a139f28ecc27719439e37c6b1533cee999cb802-refs/branch-heads/4389@{#1134}
OS macOS Version 11.2.1 (Build 20D74)

Test Case #1 (leaving crash report unchecked)

  • launched 1.21.68 Chromium: 89.0.4389.58 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 via brave://settings/privacy
  • ensured that Help improve Brave's features and performance was disabled
  • crashed Brave via brave://crash and ensured that the crash report appeared under brave://crashes but wasn't submitted
  • ensured that restarting Brave didn't submit the crash report via brave://crashes
  • ensured that clicking on Send now under brave://crashes sends the crash report to BackTrace
  • ensured that P3A is working/being updated via brave://local-state
Example Example Example
Screen Shot 2021-02-25 at 9 09 54 AM Screen Shot 2021-02-25 at 9 12 24 AM Screen Shot 2021-02-25 at 9 13 10 AM

Test Case #2 (enabled crash reporting)

  • launched 1.21.68 Chromium: 89.0.4389.58 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 on Start Brave
  • ensured that Automatically send completely private product analytics to Brave was enabled via brave://settings/privacy
  • ensured that Help improve Brave's features and performance was enabled via brave://settings/privacy
  • crashed Brave via brave://crash and ensured that the crash report appeared under brave://crashes and was automatically submitted
  • ensured that P3A is working/being updated via brave://local-state
Example Example Example
Screen Shot 2021-02-25 at 9 34 01 AM Screen Shot 2021-02-25 at 9 34 41 AM Screen Shot 2021-02-25 at 9 35 06 AM

Test Case #3 (Onboarding)

Screen Shot 2021-02-25 at 9 37 40 AM


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 via brave://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
image image image

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 on Start Brave
  • ensured that Automatically send completely private product analytics to Brave was enabled via brave://settings/privacy
  • ensured that Help improve Brave's features and performance was enabled via brave://settings/privacy
  • ensured that P3A is working/being updated via brave://local-state
Example Example Example
image image image

Test Case #3 (Onboarding)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants