New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 693442 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Regression: Unnecessary spinner is seen in Google profile photo icon of Choose a picture screen.

Project Member Reported by jbanavatu@chromium.org, Feb 17 2017

Issue description

Chrome Version: 58.0.3011.0/9287.0.0 dev channel Samus, Link, Butterfly
OS: Chrome

What steps will reproduce the problem?
(1)Recover build >> Give valid email and password in sign in screen and click on next button
(2)Now Observe for unnecessary spinner icon in Google profile photo icon of 'Choose a picture to display for you account after sign-in screen' screen

Expected: No such spinner icon should be seen in 'Choose a picture to display for you account after sign-in screen' screen.
Actual:Unnecessary spinner is seen in Google profile photo icon.

Note: 
1.This spinner icon is not seen in Change picture overlay in chrome://settings or chrome://md-settings
2. It is only seen before signing into account after giving valid sign in details.

This is regression issue as no spinner icon is seen in 57.0.2987.54/9202.28.0 beta channel quawks.

Attaching screen-cast for reference.

 
Expected.jpg
3.1 MB View Download
Actual.jpg
3.3 MB View Download
Labels: -Pri-1 Pri-2
Owner: tbuck...@chromium.org
Status: Assigned (was: Untriaged)
This looks intentional - spinner shows while waiting for the profile pic to download.  tbuckley@ can you confirm?

Regardless, not a P1
Cc: tbuck...@chromium.org
Owner: jbanavatu@chromium.org
Agreed that this seems intentional.

@jbanavatu did the profile picture eventually load?
Owner: tbuck...@chromium.org
Google profile photo icon doesn't load.Only spinner icon is seen.
Owner: zalcorn@chromium.org
While we were talking about using the UI from MD Settings, I don't think that's happened yet. @zalcorn, any thoughts what happened here? Part of MD OOBE work?
Cc: zalcorn@chromium.org
Labels: -Pri-2 Pri-1
Owner: alemate@chromium.org
I believe we use the UI from old settings.
@alemate, could MD settings have caused this?

Comment 6 by r...@chromium.org, Apr 7 2017

Owner: wzang@chromium.org

Comment 7 by wzang@chromium.org, Apr 7 2017

Zach, do we still need this for 58? If so I'll need to start ASAP.
Labels: -Pri-1 Pri-2
Colin - not strictly necessary for 58, but nice to have. Talk with alemate@ and see if MD settings could have caused this to break for us.
We need first to re-test this.
I reverted a CL for that screen that modified the code around profile image.
I this is probably fixed.

Comment 10 by wzang@chromium.org, Apr 11 2017

Status: WontFix (was: Assigned)
Cannot reproduce on a Link device, 58.0.3011.0/9287.0.0 dev channel. Fixed after reverting the CL.
IMG_20170411_161222.jpg
4.3 MB View Download

Comment 11 by wzang@chromium.org, Apr 12 2017

Status: Fixed (was: WontFix)
Status: Verified (was: Fixed)

Sign in to add a comment