Skip to content
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

[Desktop] google account info showing in profile manager #11444

Closed
LaurenWags opened this issue Aug 26, 2020 · 1 comment · Fixed by brave/brave-core#6516
Closed

[Desktop] google account info showing in profile manager #11444

LaurenWags opened this issue Aug 26, 2020 · 1 comment · Fixed by brave/brave-core#6516

Comments

@LaurenWags
Copy link
Member

LaurenWags commented Aug 26, 2020

Test Plan

Specified here: brave/brave-core#6516

Description

Broken out from #11425

In 1.14.x and 1.15.x (so far), there are multiple issues with the profile manager. Second one is:

  1. If you are signed into a google account, additional problems occur:
  • You get a "Manage your Brave sync chain" option that doesn't take you to sync settings (we probably want to remove this or fix where it leads to)
  • You get a "Sign out" option on profile menu (we probably want to remove this since signing into Google shouldn't affect the profile manager)
  • Your profile icon, previously a Brave image changes to your google account image (I had to restart to see this, but it did occur - we probably want to fix this as well)

Note - above was only seen when logged into one google account. Should also confirm logging into two google accounts (ex. work and personal) when retesting.

Steps to Reproduce

  1. Clean profile with 1.14.x or 1.15.x
  2. Create a second profile
  3. Click on one of the profile icons --> See large "Turn on sync" button ([Desktop] sync button in profile manager #11425)
  4. In one of the windows, sign into mail.google.com
  5. Click on one of the profile icons --> see "Manage your Brave sync chain" and "Sign out" options. If your profile icon hasn't changed, restart your browser and it will most likely update to your google account image.

Actual result:

Step 5:
Screen Shot 2020-08-25 at 3 20 14 PM

Expected result:

Logging into Google account shouldn't have any affect on profile manager:
Screen Shot 2020-08-25 at 3 39 19 PM

Reproduces how often:

easily

Brave version (brave://version info)

1.14.63 (Dev)
1.15.6 (Nightly)

Version/Channel Information:

  • Can you reproduce this issue with the current release? 1.12.x C84 no, 1.13.79 CR 85.0.4183.69 not atm, but it is currently behind the others in CR version
  • Can you reproduce this issue with the beta channel? yes, 1.14.63 CR 85.0.4183.78
  • Can you reproduce this issue with the nightly channel? yes 1.15.6 CR 85.0.4183.78

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields?
  • Does the issue resolve itself when disabling Brave Rewards?
  • Is the issue reproducible on the latest version of Chrome?

Miscellaneous Information:

cc @rebron @mkarolin @Brave-Matt @brave/legacy_qa

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Sep 9, 2020

Verification passed on


Brave | 1.14.79 Chromium: 85.0.4183.83 (Official Build) dev (64-bit)
-- | --
Revision | 94abc2237ae0c9a4cb5f035431c8adfb94324633-refs/branch-heads/4183@{#1658}
OS | Windows 10 OS Version 1903 (Build 18362.1016)

1. Verified the test plan from the PR https://github.com/brave/brave-core/pull/6516

Step 2: Confirmed no Accounts in Token Service or Refresh token events
image

Step 3: Confirmed Allow Google Login for extensions is off by default
image

Step 3: Confirmed toggling Allow Google Login for extensions prompts for a restart
image

Step 3: Confirmed Allow Google Login for extensions is on after the restart
image

Step 4: Able to use Google Keep extension
image

Step 5: Able to use Google Calendar extension
image

Step 6: Confirmed profile menu doesn't show any Google sign in data
image

Step 9: Confirmed after removing site data, disabling Allow Google Login for extensions and relaunching, when attempting to use the Google Keep extension the Sign in button does not work
image

Also verified the below scenarios:

Scenario 1 - install extension before signing into Google
  1. Installed build with fresh profile
  2. Navigated to brave://settings/extensions, enabled Allow Google Login for extensions and relaunched
  3. Confirmed setting retained after relaunch
  4. Navigate to https://chrome.google.com/webstore/detail/google-keep-chrome-extens/lpcaedmchfhocbbapmcbpinfpgnhiddi and install this extension
  5. Close Google login page that opens
  6. Open a tab and navigate to brave.com
  7. Open extension, see Sign in button as below
  8. Click on it and login on resulting Google page
  9. Confirmed able to login and use extension
Scenario 2 - toggle setting multiple times
  1. Installed build with fresh profile
  2. Navigated to brave://settings/extensions, enabled Allow Google Login for extensions and relaunched
  3. Confirmed setting is on
  4. Disabled Allow Google Login for extensions and relaunched
  5. Confirmed setting is off
  6. Enabled Allow Google Login for extensions and relaunched
  7. Confirmed setting is on
Scenario 3 - STR from /issues/11444
  1. Clean profile
  2. Create a second profile
  3. In one of the windows, sign into mail.google.com (this is Window A)
  4. Click on the profile icon of Window A
  5. Confirm you do not see "Manage your Brave sync chain" and "Sign out" options.
  6. Close and Relaunch.
  7. Click on profile icon of Window A
  8. Confirm profile icon has not changed to reflect the google account's image.
  9. Click on profile icon of Window B.
  10. Confirm you do not see "Manage your Brave sync chain" and "Sign out" options. Confirm profile icon has not changed.

image
image
image

Verification PASSED on macOS 10.15.6 x64 using the following build:

Brave | 1.14.81 Chromium: 85.0.4183.102 (Official Build) (64-bit)
-- | --
Revision | ffe848af6a5df4fa127e2929331116b7f9f1cb30-refs/branch-heads/4183@{#1770}
OS | macOS Version 10.15.6 (Build 19G73)

Ran through the STR/Cases outlined via brave/brave-core#6516 (comment) and ensured the following:

Case 1 - STR/Cases from brave/brave-core/pull/6516 (including extension checks)
  • ensured that both Accounts in Token Service & No token in Token Service were blank as per the following:
Screen Shot 2020-09-14 at 12 02 18 PM

Example of Allow Google login for extensions:

Screen Shot 2020-09-14 at 1 07 42 PM

Example of Google Calendar and Google Keep being installed:

Screen Shot 2020-09-14 at 1 12 40 PM
Google Login Enabled Google Login Enabled
Screen Shot 2020-09-14 at 1 13 55 PM Screen Shot 2020-09-14 at 1 24 24 PM
Google Login Disabled Google Login Disabled
Screen Shot 2020-09-14 at 1 17 03 PM Screen Shot 2020-09-14 at 1 17 09 PM
  • ensured that the profile menu didn't display any information regarding Google login as per the following:
Main Profile Second Profile (Created after enabling setting)
Screen Shot 2020-09-14 at 1 28 08 PM Screen Shot 2020-09-14 at 1 27 07 PM
Case 2 - Extensions (GCal & Keep

Ensured that enabling/disabling Allow Google login for extensions via brave://settings/extensions works as expected.

Google Login Enabled Google Login Enabled
Screen Shot 2020-09-14 at 1 13 55 PM Screen Shot 2020-09-14 at 1 24 24 PM
Google Login Disabled Google Login Disabled
Screen Shot 2020-09-14 at 1 17 03 PM Screen Shot 2020-09-14 at 1 17 09 PM
Case 3 - STR/Cases outlined via #11444
  • ensured that the profile menu didn't display any information regarding Google login as per the following:
Main Profile Second Profile (Created after enabling setting)
Screen Shot 2020-09-14 at 1 28 08 PM Screen Shot 2020-09-14 at 1 27 07 PM

Verification passed on

Brave 1.14.81 Chromium: 85.0.4183.102 (Official Build) (64-bit)
Revision ffe848af6a5df4fa127e2929331116b7f9f1cb30-refs/branch-heads/4183@{#1770}
OS Ubuntu 18.04 LTS
1. Verified the test plan from the PR https://github.com/brave/brave-core/pull/6516

Step 2: Confirmed no Accounts in Token Service or Refresh token events
image

Step 3: Confirmed Allow Google Login for extensions is off by default
image

Step 3: Confirmed toggling Allow Google Login for extensions prompts for a restart
image

Step 3: Confirmed Allow Google Login for extensions is on after the restart
image

Step 4: Able to use Google Keep extension
image

Step 5: Able to use Google Calendar extension
image

Step 6: Confirmed profile menu doesn't show any Google sign in data
image

Step 9: Confirmed after removing site data, disabling Allow Google Login for extensions and relaunching, when attempting to use the Google Keep extension the Sign in button does not work
image

Also verified the below scenarios:

Scenario 1 - install extension before signing into Google
  1. Installed build with fresh profile
  2. Navigated to brave://settings/extensions, enabled Allow Google Login for extensions and relaunched
  3. Confirmed setting retained after relaunch
  4. Navigate to https://chrome.google.com/webstore/detail/google-keep-chrome-extens/lpcaedmchfhocbbapmcbpinfpgnhiddi and install this extension
  5. Close Google login page that opens
  6. Open a tab and navigate to brave.com
  7. Open extension, see Sign in button as below
  8. Click on it and login on resulting Google page
  9. Confirmed able to login and use extension
Scenario 2 - toggle setting multiple times
  1. Installed build with fresh profile
  2. Navigated to brave://settings/extensions, enabled Allow Google Login for extensions and relaunched
  3. Confirmed setting is on
  4. Disabled Allow Google Login for extensions and relaunched
  5. Confirmed setting is off
  6. Enabled Allow Google Login for extensions and relaunched
  7. Confirmed setting is on
Scenario 3 - STR from /issues/11444
  1. Clean profile
  2. Create a second profile
  3. In one of the windows, sign into mail.google.com (this is Window A)
  4. Click on the profile icon of Window A
  5. Confirm you do not see "Manage your Brave sync chain" and "Sign out" options.
  6. Close and Relaunch.
  7. Click on profile icon of Window A
  8. Confirm profile icon has not changed to reflect the google account's image.
  9. Click on profile icon of Window B.
  10. Confirm you do not see "Manage your Brave sync chain" and "Sign out" options. Confirm profile icon has not changed.

image
image

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