Device policy wallpaper is lost when rotating the device |
||||||||
Issue descriptionChrome Version: 64.0.3258.0 OS: Chrome What steps will reproduce the problem? Set a device wallpaper, and rotate the device at login screen. What is the expected result? The device policy wallpaper should not change. What happens instead? The device policy wallpaper is changed into the default wallpaper. See video: (ignore the delay when rotating the device, that should be caused by 779311) https://drive.google.com/a/google.com/file/d/1jEVSEOqHViT0Yooqn8RVEwPSP7DednCs/view?usp=sharing
,
Nov 8 2017
,
Nov 8 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/37ebcfb0c42669055273c31dad7920ed11a258e1 commit 37ebcfb0c42669055273c31dad7920ed11a258e1 Author: Wenzhao Zang <wzang@chromium.org> Date: Wed Nov 08 20:09:32 2017 Prevent device wallpaper loss when rotating the device This CL tried to fix the issue without changing anything else to avoid regression. We could consider merging this to M63. UpdateWallpaper() function will be refactored at a later time. Bug: 782008 Change-Id: I8f60c8e3dc833233c9b725c036475089f2810511 Reviewed-on: https://chromium-review.googlesource.com/756180 Commit-Queue: Wenzhao (Colin) Zang <wzang@chromium.org> Reviewed-by: Xiaoqian Dai <xdai@chromium.org> Cr-Commit-Position: refs/heads/master@{#514920} [modify] https://crrev.com/37ebcfb0c42669055273c31dad7920ed11a258e1/chrome/browser/chromeos/login/users/wallpaper/wallpaper_manager.cc
,
Nov 8 2017
,
Nov 8 2017
,
Nov 8 2017
Waiting for verification on tot before considering merge to M63
,
Nov 9 2017
This bug requires manual review: M63 has already been promoted to the beta branch, so this requires manual review Please contact the milestone owner if you have questions. Owners: cmasso@(Android), cmasso@(iOS), gkihumba@(ChromeOS), govind@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Nov 10 2017
Verified to work on ToT and latest Canary.
,
Nov 13 2017
,
Nov 14 2017
,
Nov 20 2017
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Nov 20 2017
|
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by dtapu...@chromium.org
, Nov 8 2017