New issue
Advanced search Search tips

Issue 884850 link

Starred by 2 users

Issue metadata

Status: Started
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 3
Type: Bug



Sign in to add a comment

[Mac] Show custom avatar instead of GAIA picture when user signs in

Project Member Reported by bdeboursetty@google.com, Sep 17

Issue description

Chrome Version       : 69.0.3497.100 (Official build) on Mac OS X

1) Be signed into Google with two distinct profiles "A" and "B" and two distinct accounts "1" and "2", respectively.
2) Quit out of Chrome
3) Open Chrome
[Profile "A" shows up with account 1 profile photo in the toolbar]
4) Click account 1 profile photo in the toolbar
[A menu appear, inviting me to sync account 1 and offering the option to switch to other profiles. Profile "B" is in this list.]
5) Click on profile B in the list.
[A window for profile B appears]

Observed:
- After (4), in the list of other profiles, profile B shows up under some local name with local profile avatar
- After (5), profile B shows up with account 1 avatar and no visible local profile name
https://screenshot.googleplex.com/AtouaeWAaQc
This makes me unsure that I've switched to the correct profile.

Expected:
- Profile B is represented in a consistent way after (4) and after (5) so that I know that I've switched to the intended profile.

 
Cc: pbomm...@chromium.org
Labels: Needs-Triage-M69 OS-Mac
Components: UI>Browser>Profiles
Cc: droger@chromium.org tangltom@chromium.org markchang@chromium.org ew...@chromium.org sabineb@chromium.org msarda@chromium.org
Status: Available (was: Unconfirmed)
Just to clarify, in your second observation, you mean that "profile B shows up with account *2*," not "account 1," correct?

This bug can basically be boiled down to: we should show the custom Chrome avatar in the toolbar when the user has explicitly set one, instead of the user's GAIA photo.
> Just to clarify, in your second observation, you mean that "profile B shows up with account *2*," not "account 1," correct?

Yes, correct, sorry about that mistake

> This bug can basically be boiled down to: we should show the custom Chrome avatar in the toolbar when the user has explicitly set one, instead of the user's GAIA photo.

This would be consistent, yes. The other way would be to show the user's Gaia photo (account 2) in the profile switcher for profile A but for me the custom Chrome avatar is more meaningful in both places.
This should actually already be the case (to show the custom avatar if there is one explicitly set).

Could you check if this problem still exists in Beta (i.e. M70)?
Owner: tangltom@chromium.org
Status: Assigned (was: Available)
Summary: [Mac] Show custom avatar instead of GAIA picture when user signs in (was: Inconsistent representation of a profile between (a) its own toolbar and (b) other profiles' profile-switcher-menu)
Repro steps:
(1) Create a new profile. In the "profile creation" flow, select a non-default avatar
(2) Navigate to google.com and sign into a Google Account
(3) Notice that the avatar in the toolbar is changed to your Google Account picture (this is the bug).
(4) Navigate to chrome://settings/manageProfile. Click on the custom avatar from that settings page that you've already selected.
(5) Notice that the toolbar picture is changed back to the custom avatar.

Thomas, what's strange is that it seems like the GAIA photo overrides the custom avatar just when the user signs in. If I go back to settings and click the custom avatar again, then it fixes itself.

@bde: could you try navigating to chrome://settings/manageProfile and clicking on the custom avatar you want to be displayed in the toolbar again? Does that fix the issue for you?
Labels: OS-Linux OS-Windows
Status: Started (was: Assigned)
I looked a bit into the code and apparently
 - the preference that says whether a user has the default avatar is only changed when they go to chrome://settings/manageProfile
 - the preference is set to true when a new profile is created no matter what

This is obviously a bug. I'll try to fix it on Monday.
So, it seems that this change is not as trivial as I thought. The "Profile Creator"-view is a web view which consists of multiple stacked layers calling into the C++, interacting with profile configuration code.

Basically, there's a pref saying that a user is using the default avatar icon or not. This pref is initialized with true when a profile is created, independent of whether the user chose an icon during creation. If the user doesn't select an icon, the first one (i.e. the white avatar) is selected.

This makes me suggest an alternative fix:
We could add the assumption that users having the first avatar icon (i.e. the white avatar) haven't set a custom icon.
So all users with this icon are treated as having never set an icon.
This would make the change a lot easier.

What do you think Eli?
Yeah, that workaround in c#9 SGTM for now, Thomas. Thanks!

Sign in to add a comment