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

Issue 816861 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Delay is seen in appearing of Image icon at the bottom of Set wallpaper overlay

Project Member Reported by kebalaji@chromium.org, Feb 27 2018

Issue description

Chrome Version: 66.0.3355.0/10442.0.0 dev-channel Daisy,Reks and Kip
OS:Chrome OS

What steps will reproduce the problem?
(1)Sign-in to user>> Open set wallpaper overlay and observe

Actual: Delay is seen in appearing of Image icon at the bottom of Set wallpaper overlay
Expected: Image icon and the text should appear together

This is a Regression issue as same is working fine on M64 stable

NOTE: Issue is not seen in Windows and Linux
      Issue is seen on M65 dev also

@Wzang: Please confirm the issue
 
ActualWallpaperICon.webm
3.1 MB View Download
ExpectedWallpaperICon.webm
1.6 MB View Download

Comment 1 by wzang@chromium.org, Feb 27 2018

Status: Started (was: Assigned)
I will remove the delay, but please note that we have to see an empty frame for a moment before the image is available, like the very beginning of 'ExpectedWallpaperICon.webm' shows.
Project Member

Comment 2 by bugdroid1@chromium.org, Feb 28 2018

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

commit 091c7308920e79a399e4bee75da30c74defc5b9d
Author: Wenzhao Zang <wzang@chromium.org>
Date: Wed Feb 28 02:55:46 2018

cros: Do not hide the thumbnail in wallpaper attribution

Fixed a regression caused by
https://chromium-review.googlesource.com/c/chromium/src/+/887963

TBR=bshe@chromium.org

Bug:  816861 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ice884ea96f6dcb7ea306f7b513c0631adb089e1a
Reviewed-on: https://chromium-review.googlesource.com/939740
Commit-Queue: Wenzhao (Colin) Zang <wzang@chromium.org>
Reviewed-by: Wenzhao (Colin) Zang <wzang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539679}
[modify] https://crrev.com/091c7308920e79a399e4bee75da30c74defc5b9d/chrome/browser/resources/chromeos/wallpaper_manager/js/wallpaper_manager.js

Comment 3 by wzang@chromium.org, Feb 28 2018

Status: Fixed (was: Started)

Sign in to add a comment