[Mac] Show custom avatar instead of GAIA picture when user signs in |
|||||
Issue descriptionChrome 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.
,
Sep 17
,
Sep 18
,
Sep 18
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.
,
Sep 21
> 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.
,
Sep 21
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)?
,
Sep 21
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?
,
Sep 23
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.
,
Sep 24
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?
,
Sep 29
Yeah, that workaround in c#9 SGTM for now, Thomas. Thanks! |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by gov...@chromium.org
, Sep 17Labels: Needs-Triage-M69 OS-Mac