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

Issue 674844 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

Regression : Picture preview is seen Missing on second time navigating to chrome://md-settings/changePicture page

Project Member Reported by mmanchala@chromium.org, Dec 16 2016

Issue description

Chrome Version: 57.0.2952.0/9090.0.0 Falco,Paine and Gnawty
OS: Chrome

What steps will reproduce the problem?
(1)Sign in to user -> Go to chrome://md-settings page
(2)Now click on profile icon under 'People' to navigate to chrome://md-settings/changePicture page -> Now Observe picture Preview is available
(3)Click on back arrow to navigate to chrome://md-settings page -> Again click on profile icon and now observe for Picture Preview (Please refer Video and screenshot)

Expected: Picture preview should be seen on Navigating to chrome://md-settings/changePicture page	
Actual: Instead Picture preview is seen Missing 

This is Regression Issue as same is working fine in 56.0.2924.27/9000.28.0 Beta channel Peppy.

Note:
Issue is not seen in Chrome://settings page
 
Actual_NoPreview.jpg
76.2 KB View Download
Actual_NoPreview.webm
905 KB View Download
Expected_Preview.webm
1.0 MB View Download
Labels: -Pri-1 Pri-2
Owner: tommycli@chromium.org
Status: Assigned (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Jan 10 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5846b9cf0505c478a930d79c3b6c96a264ee4e7d

commit 5846b9cf0505c478a930d79c3b6c96a264ee4e7d
Author: tommycli <tommycli@chromium.org>
Date: Tue Jan 10 23:35:55 2017

MD Settings People: Fix ChromeOS Change Picture selected image bug

Change Picture subpage can get into an odd state when the user navigates
away and then back again. This CL fixes that regression and adds a test.

BUG= 674844 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2626733002
Cr-Commit-Position: refs/heads/master@{#442727}

[modify] https://crrev.com/5846b9cf0505c478a930d79c3b6c96a264ee4e7d/chrome/browser/resources/settings/people_page/change_picture.js
[modify] https://crrev.com/5846b9cf0505c478a930d79c3b6c96a264ee4e7d/chrome/test/data/webui/settings/people_page_change_picture_test.js

Status: Fixed (was: Assigned)
Status: Verified (was: Fixed)
Verified on Chrome OS 9202.1.0, 57.0.2987.6

Sign in to add a comment