New issue
Advanced search Search tips

Issue 907665 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Closed: Nov 22
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----



Sign in to add a comment

PreviewsLitePageServerTimeoutBrowserTest.LitePagePreviewsTimeout failing on linux-chromeos-rel

Project Member Reported by sheriff-...@appspot.gserviceaccount.com, Nov 21

Issue description

Filed by sheriff-o-matic@appspot.gserviceaccount.com on behalf of gayane@chromium.org

PreviewsLitePageServerTimeoutBrowserTest.LitePagePreviewsTimeout failing on linux-chromeos-rel

Builders failed on: 
- linux-chromeos-rel: 
  https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/linux-chromeos-rel


Error log from https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8929212149298839648/+/steps/viz_browser_tests/0/logs/PreviewsLitePageServerTimeoutBrowserTest.LitePagePreviewsTimeout/0
 RUN      ] PreviewsLitePageServerTimeoutBrowserTest.LitePagePreviewsTimeout
[1350:1350:1121/135731.637788:WARNING:chrome_browser_main_chromeos.cc(544)] Running as stub user with profile dir: test-user
[1350:1350:1121/135731.782733:WARNING:easy_unlock_service_regular.cc(522)] EasyUnlockServiceRegular::IsInLegacyHostMode: DeviceSyncClient not ready. Returning false.
[1350:1350:1121/135731.816565:INFO:remote_commands_service.cc(38)] Fetching remote commands.
[1350:1350:1121/135731.816651:WARNING:remote_commands_service.cc(40)] Client is not registered.
[1350:1350:1121/135731.816676:INFO:remote_commands_invalidator.cc(32)] Initialize RemoteCommandsInvalidator.
[1350:1350:1121/135731.816689:INFO:remote_commands_invalidator.cc(57)] Starting RemoteCommandsInvalidator.
[1350:1350:1121/135731.816700:INFO:remote_commands_invalidator.cc(123)] RemoteCommandsInvalidator ReloadPolicyData.
[1350:1350:1121/135731.816712:INFO:remote_commands_invalidator.cc(167)] Unregister RemoteCommandsInvalidator.
[1350:1350:1121/135731.830647:WARNING:wallpaper_controller_client.cc(358)] Cannot get wallpaper files id in RemovePolicyWallpaper. This should never happen under normal circumstances.
[1350:1350:1121/135732.097468:ERROR:gpu_interface_provider.cc(87)] Not implemented reached in virtual void content::GpuInterfaceProvider::RegisterOzoneGpuInterfaces(service_manager::BinderRegistry *)
[1350:1356:1121/135732.431799:WARNING:embedded_test_server.cc(239)] Request not handled. Returning 404: /favicon.ico
../../chrome/browser/previews/previews_lite_page_browsertest.cc:1024: Failure
Expected: (max_penalty) >= (kTimeoutMs), actual: 210 vs 250
Stack trace:
#0 0x55b48525c7fc testing::internal::UnitTestImpl::CurrentOsStackTraceExceptTop()
#1 0x55b48525c1d9 testing::internal::AssertHelper::operator=()
#2 0x55b483dec582 PreviewsLitePageServerTimeoutBrowserTest_LitePagePreviewsTimeout_Test::RunTestOnMainThread()
#3 0x55b487970a4d content::BrowserTestBase::ProxyRunTestOnMainThreadLoop()
#4 0x55b487530ba5 ChromeBrowserMainParts::PreMainMessageLoopRunImpl()
#5 0x55b48752fb42 ChromeBrowserMainParts::PreMainMessageLoopRun()
#6 0x55b484dfcc82 chromeos::ChromeBrowserMainPartsChromeos::PreMainMessageLoopRun()
#7 0x55b485df7795 content::BrowserMainLoop::PreMainMessageLoopRun()
#8 0x55b486185c27 content::StartupTaskRunner::RunAllTasksNow()
#9 0x55b485df65a0 content::BrowserMainLoop::CreateStartupTasks()
#10 0x55b485df9718 content::BrowserMainRunnerImpl::Initialize()
#11 0x55b485df49a2 content::BrowserMain()
#12 0x55b48707d459 content::ContentMainRunnerImpl::Run()
#13 0x55b4891727dc service_manager::Main()
#14 0x55b48707b934 content::ContentMain()
#15 0x55b487970681 content::BrowserTestBase::SetUp()
#16 0x55b4874e0754 InProcessBrowserTest::SetUp()

 
Cc: robertogden@chromium.org
robertogden, is this failure related to your recent change 
https://chromium-review.googlesource.com/c/chromium/src/+/1334944 ?
Owner: robertogden@chromium.org
Status: Assigned (was: Available)
assigning to robertogden, PreviewsLitePageServerTimeoutBrowserTest.LitePagePreviewsTimeout is falky.

Status: Started (was: Assigned)
Oops, yeah. I can fix this quick though.
 Issue 907690  has been merged into this issue.
Labels: -Sheriff-Chromium
Chromium Sheriff here.
It looks an owner is assigned. Let me remove Sheriff-Chromium label.
Project Member

Comment 6 by bugdroid1@chromium.org, Nov 22

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

commit ad12edee42df137c0c78cad5b36db9b9e2d63e59
Author: Robert Ogden <robertogden@chromium.org>
Date: Thu Nov 22 01:57:41 2018

Fix timeout flake from last CL

Bug:  907665 
Change-Id: I07737f4e4dee8c28741dc63016eb376604c21723
Reviewed-on: https://chromium-review.googlesource.com/c/1346996
Reviewed-by: Gayane Petrosyan <gayane@chromium.org>
Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Commit-Queue: Robert Ogden <robertogden@chromium.org>
Commit-Queue: Tarun Bansal <tbansal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#610279}
[modify] https://crrev.com/ad12edee42df137c0c78cad5b36db9b9e2d63e59/chrome/browser/previews/previews_lite_page_browsertest.cc

Issue 907776 has been merged into this issue.
Cc: tbansal@chromium.org csharrison@chromium.org
 Issue 909824  has been merged into this issue.

Sign in to add a comment