Issue metadata
Sign in to add a comment
|
MSE_Widevine_Mojo_EncryptedMediaTest.PolicyCheck_0 : Fails in Win, Win64 and Linux trunk builders |
||||||||||||||||||||
Issue descriptionBelow 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.
,
Jul 26 2017
Fix is under review at https://chromium-review.googlesource.com/c/585771/
,
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
,
Jul 26 2017
Fix landed and bots are green now. |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by xhw...@chromium.org
, Jul 25 2017