New issue
Advanced search Search tips

Issue 748697 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 1
Type: Bug-Regression



Sign in to add a comment

MSE_Widevine_Mojo_EncryptedMediaTest.PolicyCheck_0 : Fails in Win, Win64 and Linux trunk builders

Project Member Reported by ligim...@chromium.org, Jul 25 2017

Issue description

Below mentioned unit_test fails in Win, Win64 and Linux official.desktop trunk builders.

MSE_Widevine_Mojo/EncryptedMediaTest.PolicyCheck/0

Link to the Builder
===================
https://uberchromegw.corp.google.com/i/official.desktop.continuous/builders/linux64%20trunk/builds/2274

Link to the log file
====================
https://uberchromegw.corp.google.com/i/official.desktop.continuous/builders/linux64%20trunk/builds/2274/steps/browser_tests/logs/MSE_Widevine_Mojo_EncryptedMediaTest.PolicyCheck_0

Error Log
=========
MSE_Widevine_Mojo/EncryptedMediaTest.PolicyCheck/0 (run #1):
[ RUN      ] MSE_Widevine_Mojo/EncryptedMediaTest.PolicyCheck/0
Xlib:  extension "RANDR" missing on display ":9".
[22859:22859:0725/132435.648788:WARNING:password_store_factory.cc(255)] Using basic (unencrypted) store for password storage. See https://chromium.googlesource.com/chromium/src/+/master/docs/linux_password_storage.md for more information about password storage options.
[22859:22859:0725/132435.692489:WARNING:browser_main_loop.cc(276)] <unknown>: AT-SPI: Could not obtain desktop path or name

The test is flaky, hence unable to get the regression information.

Flaky Information
==================
Build ranges checked for flakiness: 2225 : 2274
Number of times failed: 13 times out of 48 builds.Suspected CL

First Bad Build: https://uberchromegw.corp.google.com/i/official.desktop.continuous/builders/linux64%20trunk/builds/2260
Last Good Build: https://uberchromegw.corp.google.com/i/official.desktop.continuous/builders/linux64%20trunk/builds/2259

This is the closest patch which could have caused the failure.

https://chromium.googlesource.com/chromium/src/+/ada25fea214db0d07efb6d99f8a97e5d39c9960e

Assigning to the CL owner as well as the sheriff to either disable of fix the test.
 

Comment 1 by xhw...@chromium.org, Jul 25 2017

Status: Started (was: Assigned)
It's caused by https://chromium.googlesource.com/chromium/src/+/57fc53b6e72d22dbc04bacafc117e5a58dcef19d

Looking now.

Comment 2 by xhw...@chromium.org, Jul 26 2017

Fix is under review at https://chromium-review.googlesource.com/c/585771/
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 26 2017

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

commit 18e2ca23369139e604c5a5751ccc29eb510972cd
Author: Xiaohan Wang <xhwang@chromium.org>
Date: Wed Jul 26 06:32:09 2017

media: Fix CdmAdapter on GetStatusForPolicy() on CDM_8

CdmAdapter should check the return value and reject the promise as
needed per CdmWrapper API. Also mark the API as WARN_UNUSED_RESULT to
prevent this from happening again.

TBR=jrummell@chromium.org
BUG= 748697 
TEST=Failing test is now passing.

Change-Id: Ic656e968b03f5853a194d1b5a4202ef7decb7626
Reviewed-on: https://chromium-review.googlesource.com/585771
Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
Reviewed-by: Xiaohan Wang <xhwang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#489566}
[modify] https://crrev.com/18e2ca23369139e604c5a5751ccc29eb510972cd/media/cdm/cdm_adapter.cc
[modify] https://crrev.com/18e2ca23369139e604c5a5751ccc29eb510972cd/media/cdm/cdm_wrapper.h

Comment 4 by xhw...@chromium.org, Jul 26 2017

Status: Fixed (was: Started)
Fix landed and bots are green now.

Sign in to add a comment