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

Issue 666671 link

Starred by 2 users
Status: Verified
Owner:
tommycli@chromium.org
Closed: Dec 2016
Cc:
ajha@chromium.org
█ kavvaru@chromium.org
█ brajkumar@chromium.org
█ durga.behera@chromium.org
dpa...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Regression : 'Photo from internal camera' Tool tip is seen missing for captured picture in chrome://md-settings/changePicture page

Project Member Reported by mmanchala@chromium.org, Nov 18 2016 
Version: 56.0.2923.0/8999.0.0 dev-channel Blaze,Falco and Gnawty
OS: Chrome

What steps will reproduce the problem?
(1)Sign in to user -> Go to chrome://md-settings/changePicture page 
(2)Take a photo -> Now hover the mouse pointer at captured picture and observe for Tool tip (Please refer screenshot)

Expected: 'Photo from internal camera' Tool tip should be seen for captured picture on hovering the mouse pointer
Actual: Instead Tool tip is seen missing 

This is Regression Issue as same is working fine in chrome://settings page i.e. Tool tip is seen.
Actual.jpg
130 KB View Download
Expected_Tooltip.jpg
138 KB View Download

Comment 1 Deleted

Comment 2 by steve...@chromium.org, Nov 22 2016

Labels: -M-56 M-57
Owner: tommycli@chromium.org
Status: Assigned (was: Aaa)
Minor bug, we can fix this for 57.
Project Member

Comment 3 by bugdroid1@chromium.org, Nov 30 2016 

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

commit eab2da39cd9f081c229a29bdaa16fc95b967a10b
Author: tommycli <tommycli@chromium.org>
Date: Wed Nov 30 18:00:07 2016

MD Settings: Add tooltip for Old Image in Change Picture.

The tooltip is now "Existing photo from camera or file", as the code
does not track where it came from.

In the old Options version, it said "Photo from internal camera" in the
first preview icon immediately after taking it, but not on subsequent
openings of the dialog.

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

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

[modify] https://crrev.com/eab2da39cd9f081c229a29bdaa16fc95b967a10b/chrome/app/settings_strings.grdp
[modify] https://crrev.com/eab2da39cd9f081c229a29bdaa16fc95b967a10b/chrome/browser/resources/settings/people_page/change_picture.html
[modify] https://crrev.com/eab2da39cd9f081c229a29bdaa16fc95b967a10b/chrome/browser/ui/webui/settings/md_settings_localized_strings_provider.cc

Comment 4 by tommycli@chromium.org, Dec 1 2016

Status: Fixed (was: Assigned)

Comment 5 by sdantul...@chromium.org, Dec 13 2016

Status: Verified (was: Fixed)
Verified on ChromeOS 9077.0.0, 57.0.2948.0
►

Sign in to add a comment