New issue
Advanced search Search tips

Issue 817971 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Widevine browser_tests failing

Project Member Reported by jrumm...@chromium.org, Mar 1 2018

Issue description

Running ToT locally on Linux, the Widevine browser_tests are failing.

[253711:253711:0301/112556.771118:INFO:test_license_server.cc(51)] Killing license server.
[253711:253711:0301/112556.771180:FATAL:thread_restrictions.cc(105)] Check failed: !g_base_sync_primitives_disallowed.Get().Get(). Waiting on a //base sync primitive is not allowed on this thread to prevent jank and deadlock. If waiting on a //base sync primitive is unavoidable, do it within the scope of a ScopedAllowBaseSyncPrimitives. If in a test, use ScopedAllowBaseSyncPrimitivesForTesting.
#0 0x7fe18ece56ad base::debug::StackTrace::StackTrace()
#1 0x7fe18ece3b9c base::debug::StackTrace::StackTrace()
#2 0x7fe18ed6bbca logging::LogMessage::~LogMessage()
#3 0x7fe18ef0abdb base::internal::AssertBaseSyncPrimitivesAllowed()
#4 0x7fe18ee35385 base::Process::WaitForExitWithTimeout()
#5 0x7fe18ee351da base::Process::Terminate()
#6 0x00000220cf0a TestLicenseServer::Stop()
#7 0x00000220cd3c TestLicenseServer::~TestLicenseServer()
#8 0x0000021be5b1 EncryptedMediaTestBase::~EncryptedMediaTestBase()
#9 0x0000021be461 EncryptedMediaTest::~EncryptedMediaTest()
#10 0x0000021baca5 EncryptedMediaTest_PolicyCheck_Test::~EncryptedMediaTest_PolicyCheck_Test()
 
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 2 2018

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

commit 5dcf67cb7f275bcd288968c96832debf76ad14df
Author: John Rummell <jrummell@chromium.org>
Date: Fri Mar 02 00:10:17 2018

Enable Sync primitives when shutting down test license server

Recent changes to the Process code trigger this, so allow it inside the
test license server.

BUG= 817971 
TEST=Widevine browser_tests pass locally

Change-Id: Ice5f89164413e7aab3095e9a767e411ff7e940d7
Reviewed-on: https://chromium-review.googlesource.com/944265
Commit-Queue: John Rummell <jrummell@chromium.org>
Reviewed-by: Xiaohan Wang <xhwang@chromium.org>
Reviewed-by: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540360}
[modify] https://crrev.com/5dcf67cb7f275bcd288968c96832debf76ad14df/chrome/browser/media/test_license_server.cc

Status: Fixed (was: Started)

Sign in to add a comment