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

Issue 660835 link

Starred by 2 users

Issue metadata

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

Blocking:
issue 614589



Sign in to add a comment

Regression : 'Take photo' icon is seen missing on reloading chrome://md-settings/changePicture page.

Project Member Reported by jbanavatu@chromium.org, Oct 31 2016

Issue description

Version: 56.0.2905.0/8949.0.0 dev-channel nyan_blaze, samus, wolf
OS: ChromeOS

What steps will reproduce the problem?
(1)Sign in to chrome>> Navigate to chrome://md-settings/changePicture page>>Now click on reload icon and Observe 'Take photo' icon (Please refer video)

Expected: 'Take photo' icon should not vanish on reloading chrome://md-settings/changePicture page.
Actual: Instead 'Take photo'icon is seen missing.

This is Regression issue as same is working fine in chrome://settings page i.e. 'Take photo' icon is seen after reloading chrome://settings page.

Attaching screen-cast for reference.

 
Actual.webm
1.0 MB View Download
Owner: tbuck...@chromium.org
Status: Assigned (was: Untriaged)
Cc: dbeam@chromium.org
Labels: -M-56 Proj-MaterialDesign-WebUI Hotlist-MD-Settings-PeopleCrOS
Owner: mahmadi@chromium.org

Comment 3 by dbeam@chromium.org, Nov 2 2016

Blocking: 614589

Comment 4 by dbeam@chromium.org, Nov 2 2016

Cc: tommycli@chromium.org
Status: Started (was: Assigned)
Project Member

Comment 6 by bugdroid1@chromium.org, Nov 7 2016

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

commit 7fb78619cd5e3989365ac8763887f83ad4e1acd8
Author: mahmadi <mahmadi@chromium.org>
Date: Mon Nov 07 14:52:34 2016

[MD Settings][People] Fixes take picture button disappearing on page refresh

BUG= 660835 

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

[modify] https://crrev.com/7fb78619cd5e3989365ac8763887f83ad4e1acd8/chrome/browser/ui/webui/settings/chromeos/change_picture_handler.cc

Status: Fixed (was: Started)
Status: Verified (was: Fixed)
Verified on ChromeOS 8980.0.0, 56.0.2915.0

Sign in to add a comment