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

Issue 695375 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 1
Type: ----



Sign in to add a comment

content_unittests failing on 3 builders

Project Member Reported by hajimehoshi@google.com, Feb 23 2017

Issue description

Cc: maxmorin@chromium.org

Comment 2 by treib@chromium.org, Feb 23 2017

Suspected culprit reverted in https://codereview.chromium.org/2717453002/
Let's see if that fixes things.

Comment 3 by treib@chromium.org, Feb 23 2017

Labels: OS-Windows Pri-1
Owner: treib@chromium.org
Status: Started (was: Available)
The revert didn't help; CL was relanded.

I'm disabling the test on Windows.

Comment 4 by treib@chromium.org, Feb 23 2017

Cc: fsam...@chromium.org clamy@chromium.org samans@chromium.org
Components: Tests>Disabled
Labels: -Sheriff-Chromium
Owner: alex...@chromium.org
Status: Assigned (was: Started)
CL to disable the test is in the CQ: https://codereview.chromium.org/2712713006

Assigning to an owner; CCing other owner and people who recently touched code in this area.
Project Member

Comment 5 by bugdroid1@chromium.org, Feb 23 2017

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

commit 8d90ac250bb2ac5aaa876b04300c8edce0a7b824
Author: treib <treib@chromium.org>
Date: Thu Feb 23 11:49:38 2017

Disable RenderWidgetHostViewGuestTest.VisibilityTest on Windows

BUG= 695375 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
TBR=alexmos@chromium.org

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

[modify] https://crrev.com/8d90ac250bb2ac5aaa876b04300c8edce0a7b824/content/browser/frame_host/render_widget_host_view_guest_unittest.cc

Comment 6 by tzik@chromium.org, Feb 23 2017

I think this is due to missing TaskScheduler instance used by base::PostTaskWithTraits in RenderWidgetHostImpl ctor.
I don't know why it has not failed before, but adding a ScopedTaskScheduler should fix the failure.
Cc: alex...@chromium.org
Owner: tzik@chromium.org
tzik@ has a CL in progress to fix this.
Project Member

Comment 8 by bugdroid1@chromium.org, Feb 24 2017

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

commit 33237fadcbda24bd31d2a166ba9a79c994671dda
Author: tzik <tzik@chromium.org>
Date: Fri Feb 24 00:52:10 2017

Keep a TaskScheduler instance in RenderWidgetHostViewGuestTest

RenderWidgetHostImpl on Windows uses TaskScheduler in its constructor,
and that requires a TaskScheduler instance.
This CL add one to RenderWidgetHostViewGuestTest, so that it unbreak
the test failure on Windows bots.

BUG= 695375 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

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

[modify] https://crrev.com/33237fadcbda24bd31d2a166ba9a79c994671dda/content/browser/frame_host/render_widget_host_view_guest_unittest.cc

Comment 9 by tzik@chromium.org, Feb 24 2017

Status: Fixed (was: Assigned)
Components: Tests>Disabled
Labels: Test-Disabled

Sign in to add a comment