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

Issue 607350 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
not available
Closed: Jun 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 608979



Sign in to add a comment

testEMEPermission failure on chromium.android WebView L and M bot

Project Member Reported by yolandyan@chromium.org, Apr 27 2016

Issue description

The failure is only on AOSP build Android because AOSP Android doesn't support the DRM scheme (widevine) offered by Google and used by Chrome (OEM partners need to have a contract with Google to get the widevine source)

Therefore, the upstream bot fails on testEMEPermission. Not sure whether I should delete the test case or add DisableIf annotation condition to it?
 
Blocking: 608979

Comment 2 by torne@chromium.org, May 4 2016

Cc: mlamouri@chromium.org
Please don't leave failing tests on teh waterfall for a week - when you found the test was broken you should have either reverted the change or disabled the test right away. The bot has since started failing *other* tests as well, but nobody has paid any attention because it was already broken by your change :/
Project Member

Comment 3 by bugdroid1@chromium.org, May 4 2016

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

commit 8b8866014d743f8ee4fd9a5229f7ec28334b46b7
Author: timvolodine <timvolodine@chromium.org>
Date: Wed May 04 14:13:01 2016

[WebViewLayoutTests] Disable failing testEMEPermission layout test.

Disable the EME test which is currently failing on AOSP bots.

BUG= 607350 

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

[modify] https://crrev.com/8b8866014d743f8ee4fd9a5229f7ec28334b46b7/android_webview/tools/system_webview_shell/layout_tests/src/org/chromium/webview_shell/test/WebViewLayoutTest.java

+torne, sorry, that was my mistake. Will do in the future
Status: Fixed (was: Untriaged)

Sign in to add a comment