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

Issue 676782 link

Starred by 3 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 : 'Google Profile photo(loading)' Tool tip is seen instead of 'Google Profile photo' in chrome://md-settings/changePicture page

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

Issue description

Chrome Version: 57.0.2959.0/9112.0.0 Wolf,Daisy and Candy
OS: Chrome

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

Expected:'Google Profile photo' Tool tip should be seen
Actual: Instead 'Google Profile photo(loading)' Tool tip is seen

This is Regression Issue as same is working fine in chrome://settings page i.e. 'Google Profile photo' Tool tip is seen.
 
Actual_Tooltip1.jpg
121 KB View Download
Actual_Tooltip2.jpg
124 KB View Download
Expected_Tooltip.jpg
127 KB View Download
Labels: -Pri-1 Pri-2
Owner: tommycli@chromium.org
Status: Assigned (was: Untriaged)
It looks like we are setting 'title' in the html with (loading), but 'alt' in the JS in receiveProfileImage_ without (loading). I'm not sure what the difference between 'title' and 'alt' is, but this seems like a quick fix.

Project Member

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

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

commit 506bf2924d4de457f71448594c43c79315b86783
Author: tommycli <tommycli@chromium.org>
Date: Wed Jan 11 01:22:31 2017

MD Settings People: Fix ChromeOS Change Picture Profile Icon tooltip

This fixes a regression where the tooltip to the profile picture is
never updated.

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

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

[modify] https://crrev.com/506bf2924d4de457f71448594c43c79315b86783/chrome/browser/resources/settings/people_page/change_picture.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