set wallpaper then the PNG file will display incorrect (overlap with last image ) |
|||||||||||||
Issue descriptionMoved from https://code.google.com/p/chrome-os-partner/issues/detail?id=50472. See there for more detailed history. Short version: 1: Set one wallpaper. 2: Then set another. 3: Flip back and forth. Sometimes, depending on the wallpaper you'll get pieces of both wallpapers displaying at the same time.
,
May 6 2016
xdai, any progress on this one?
,
May 6 2016
Not yet. I was distracted by other issues.
,
May 18 2016
Move back to 'Assigned' since I won't be able to work on it for the next several months.
,
May 18 2016
xiaoyinh@ can you dig into this since xdai@ will be occupied with other issues for a while?
,
May 19 2016
I'm able to reproduce this issue. One observation: Once I change the image to jpeg format, this issue goes away.
,
May 20 2016
In code review right now: https://codereview.chromium.org/1996793002/
,
May 20 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5fb57605dc341abde4f2eace5ec578520b74b224 commit 5fb57605dc341abde4f2eace5ec578520b74b224 Author: xiaoyinh <xiaoyinh@chromium.org> Date: Fri May 20 21:45:42 2016 This change is to fill the canvas with black background every time before painting a new wallpaper, so that the new wallpaper with partial transparency won't be painted upon a previous wallpaper. BUG= 606434 Review-Url: https://codereview.chromium.org/1996793002 Cr-Commit-Position: refs/heads/master@{#395175} [modify] https://crrev.com/5fb57605dc341abde4f2eace5ec578520b74b224/ash/desktop_background/desktop_background_view.cc
,
May 21 2016
,
May 23 2016
Issue 613944 has been merged into this issue.
,
May 24 2016
FYI: Fix landed in Chromium 53.0.2744.0.
,
Jun 3 2016
Verified on 53.0.2754.0/8406.0.0
,
Jun 6 2016
Can we merge this to M52?
,
Jun 6 2016
,
Jun 6 2016
Your change meets the bar and is auto-approved for M52 (branch: 2743)
,
Jun 6 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/9b6f08c731efaf219e4db595a2a6a972521d1701 commit 9b6f08c731efaf219e4db595a2a6a972521d1701 Author: xdai <xdai@chromium.org> Date: Mon Jun 06 19:01:14 2016 [Merge to M52]Fill the canvas with black background before painting wallpaper. Merge this CL to M52 on behalf of xiaoyinh@. This change is to fill the canvas with black background every time before painting a new wallpaper, so that the new wallpaper with partial transparency won't be painted upon a previous wallpaper. BUG= 606434 TBR=achuith@chromium.org, xiaoyinh@chromium.org Review-Url: https://codereview.chromium.org/1996793002 Cr-Commit-Position: refs/heads/master@{#395175} (cherry picked from commit 5fb57605dc341abde4f2eace5ec578520b74b224) Review URL: https://codereview.chromium.org/2039173003 . Cr-Commit-Position: refs/branch-heads/2743@{#244} Cr-Branched-From: 2b3ae3b8090361f8af5a611712fc1a5ab2de53cb-refs/heads/master@{#394939} [modify] https://crrev.com/9b6f08c731efaf219e4db595a2a6a972521d1701/ash/desktop_background/desktop_background_view.cc
,
Jun 6 2016
,
Jun 15 2016
Verified on TOT (8456.0.0, 53.0.2764.0) and M52 (8350.31.0, 52.0.2743.41) |
|||||||||||||
►
Sign in to add a comment |
|||||||||||||
Comment 1 by x...@chromium.org
, Apr 28 2016