New issue
Advanced search Search tips

Issue 815310 link

Starred by 1 user

Issue metadata

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

Blocking:
issue 810169



Sign in to add a comment

Surprise me wallpapers set from the new picker have low resolution

Project Member Reported by wzang@chromium.org, Feb 23 2018

Issue description

This is because the new Backdrop wallpapers use display size as suffix, but the app background page doesn't have access to it.
 
Project Member

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

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

commit d9f77afefccb6fc2d4705647839a4056a55df4ad
Author: Wenzhao Zang <wzang@chromium.org>
Date: Tue Feb 27 03:08:08 2018

cros: Add high resolution suffix for surprise me wallpapers

Bug:  815310 
Test: Turn on "Enable new wallpaper picker" in chrome://flags
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Id1907d83d5961c57413d4a5e2050e00680889f5b
Reviewed-on: https://chromium-review.googlesource.com/935754
Commit-Queue: Wenzhao (Colin) Zang <wzang@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539386}
[modify] https://crrev.com/d9f77afefccb6fc2d4705647839a4056a55df4ad/chrome/browser/chromeos/extensions/wallpaper_private_api.cc
[modify] https://crrev.com/d9f77afefccb6fc2d4705647839a4056a55df4ad/chrome/browser/resources/chromeos/wallpaper_manager/js/constants.js
[modify] https://crrev.com/d9f77afefccb6fc2d4705647839a4056a55df4ad/chrome/browser/resources/chromeos/wallpaper_manager/js/event_page.js
[modify] https://crrev.com/d9f77afefccb6fc2d4705647839a4056a55df4ad/chrome/browser/resources/chromeos/wallpaper_manager/js/util.js
[modify] https://crrev.com/d9f77afefccb6fc2d4705647839a4056a55df4ad/chrome/browser/resources/chromeos/wallpaper_manager/js/wallpaper_manager.js
[modify] https://crrev.com/d9f77afefccb6fc2d4705647839a4056a55df4ad/chrome/test/data/chromeos/wallpaper_manager/unit_tests/api_mock.js
[modify] https://crrev.com/d9f77afefccb6fc2d4705647839a4056a55df4ad/chrome/test/data/chromeos/wallpaper_manager/unit_tests/event_page_unittest.js

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

Status: Fixed (was: Assigned)

Sign in to add a comment