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

Issue 810224 link

Starred by 2 users
Status: Assigned
Owner:
█ reve...@chromium.org
Last visit > 30 days ago
Cc:
sdantul...@chromium.org
█ reve...@chromium.org
tbuck...@chromium.org
dhadd...@chromium.org
steve...@chromium.org
abod...@chromium.org
dpa...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug-Regression



Sign in to add a comment

User captured profile photo is not persisted in the gallery list after profile picture change.

Project Member Reported by mkarkada@chromium.org, Feb 8 2018 
CrOS version: M65- 10323.21.0, 65.0.3325.56 dev channel eve, coral Robo360, paine devices

What steps will reproduce the problem?
1. Goto Settings>People>Change picture page.
2. Click on 'Take photo' option and capture user photo.
3. User profile picture gets updated to the captured one.
4. Now again open Settings> Change picture and choose a picture from the gallery. 
5. User captured photo is still seen as part of the gallery.
6. Close settings and reopen settings> Change picture page and observe the gallery list.

What is the expected result?
User captured photo should have been persisted in the gallery list.

What happens instead?
User captured photo is vanished from the gallery list.

Comment 1 by reve...@chromium.org, Feb 8 2018

Cc: steve...@chromium.org
I think we need to update the code here:

https://cs.chromium.org/chromium/src/chrome/browser/ui/webui/settings/chromeos/change_picture_handler.cc?l=200

in order to not only send the external image to the settings UI when it's selected.

Comment 2 by steve...@chromium.org, May 16 2018

Cc: tbuck...@chromium.org
Labels: -Pri-1 Pri-2
Owner: reve...@chromium.org
Status: Assigned (was: Untriaged)
Did this get fixed? It looks like the code has changed since the link was pasted. I'm not clear what the fix here is.

I'm not sure that it's imported or even desired that we persist the captured photo if the user selects a different one. De-prioritizing, +tbuckley@
►

Sign in to add a comment