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

Issue 691548 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Oct 22
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

WallpaperManagerBrowserTest.DisplayChange slightly flaky on ChromiumOS Ozone tests

Project Member Reported by msramek@chromium.org, Feb 13 2017

Issue description

WallpaperManagerBrowserTest.DisplayChange is flaky on ChromiumOS Ozone tests:

https://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Ozone%20Tests%20%281%29

Frequency: First spotted 6 builds ago, since then one more flake appeared.

==================================================================================================

WallpaperManagerBrowserTest.DisplayChange (run #1):
[ RUN      ] WallpaperManagerBrowserTest.DisplayChange
[28713:28713:0213/041358.258422:WARNING:audio_manager.cc(322)] Multiple instances of AudioManager detected
[28713:28713:0213/041358.258462:WARNING:audio_manager.cc(279)] Multiple instances of AudioManager detected
[28713:28738:0213/041358.276421:ERROR:service_manager.cc(209)] Connection InterfaceProviderSpec prevented service: content_browser from binding interface: prefs::mojom::PreferencesServiceFactory exposed by: preferences
[28713:28713:0213/041358.309151:WARNING:signin_screen_policy_provider.cc(61)] Denying load of Extension  : mhjfbmdgcfjbbpaeojofohoefgiehjai / Chromium PDF Viewer because of 'signin_screen' is only allowed for packaged apps, but this is a extension.
[28713:28734:0213/041358.312575:WARNING:freezer_cgroup_process_manager.cc(61)] Cgroup freezer does not exist or is not writable. Unable to freeze renderer processes.
[28713:28713:0213/041358.333452:WARNING:signin_screen_policy_provider.cc(61)] Denying load of Extension  : mhjfbmdgcfjbbpaeojofohoefgiehjai / Chromium PDF Viewer because of 'signin_screen' is only allowed for packaged apps, but this is a extension.
../../chrome/browser/chromeos/login/users/wallpaper/wallpaper_manager_browsertest.cc:668: Failure
Value of: WallpaperManager::Get()->GetAppropriateResolution()
  Actual: 1
Expected: WallpaperManager::WALLPAPER_RESOLUTION_LARGE
Which is: 0
[  FAILED  ] WallpaperManagerBrowserTest.DisplayChange, where TypeParam =  and GetParam() =  (304 ms)

==================================================================================================
 
Cc: bshe@chromium.org
Owner: achuith@chromium.org
Status: Assigned (was: Untriaged)
I currently don't see any recent CLs touching wallpaper, display size detection or such. Assigning to wallpaper owners to please analyze further.
Cc: achuith@chromium.org
Owner: x...@chromium.org
I'm not a good owner for this. Daisy?
Project Member

Comment 4 by bugdroid1@chromium.org, Feb 14 2017

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

commit 1e1ddf1e55bff7ead27411e3ac5dfab57b6966e4
Author: tyoshino <tyoshino@chromium.org>
Date: Tue Feb 14 09:34:38 2017

Disable WallpaperManagerBrowserTest.DisplayChange for Chrome OS with Ozone enabled

BUG= 691548 
TBR=bshe@chromium.org

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

[modify] https://crrev.com/1e1ddf1e55bff7ead27411e3ac5dfab57b6966e4/chrome/browser/chromeos/login/users/wallpaper/wallpaper_manager_browsertest.cc

Labels: -Hotlist-Sheriff-Chromium
Confirmed that it's skipped.

Removing Hotlist-Sheriff-Chromium label.
Labels: Sheriff-Chromium
Please use Sheriff-Chromium label instead of Hotlist-Sheriff-Chromium. Issues with Sheriff-Chromium label are shown in Sheriff-o-Matic and are tried by sheriffs directly.
Labels: -Sheriff-Chromium
Turns out search on Monorail is eventually consistent. Removing Sheriff-Chromium from this issue since Hotlist-Sheriff-Chromium was removed earlier.
Cc: x...@chromium.org
Owner: wzang@chromium.org
Pass to wzang@.
Status: WontFix (was: Assigned)
This test is moved to //ash.

Sign in to add a comment