Commit 55b7fff6 authored by tguilbert's avatar tguilbert Committed by Commit bot

Enable PlatformMediaCodecTest

testCanPlayePlatformMediaCodecs was disabled due to flakiness. It seems
that the test is no longer flaky and can be re-enabled (since we are no
longer using WMPA, or because of some unknown changes in test
reliability).

NOTE: If ever the screen is off when the test runs, the test fails.

BUG=620890
TEST=100x run on Nexus 5 and Nexus 7

Review-Url: https://codereview.chromium.org/2570483005
Cr-Commit-Position: refs/heads/master@{#437984}
parent b8467997
...@@ -4,7 +4,9 @@ ...@@ -4,7 +4,9 @@
package org.chromium.android_webview.test; package org.chromium.android_webview.test;
import org.chromium.base.test.util.DisabledTest; import android.test.suitebuilder.annotation.MediumTest;
import org.chromium.base.test.util.Feature;
import org.chromium.content.browser.ContentViewCore; import org.chromium.content.browser.ContentViewCore;
import org.chromium.content.browser.test.util.DOMUtils; import org.chromium.content.browser.test.util.DOMUtils;
import org.chromium.content_public.browser.WebContents; import org.chromium.content_public.browser.WebContents;
...@@ -28,11 +30,8 @@ public class PlatformMediaCodecTest extends AwTestBase { ...@@ -28,11 +30,8 @@ public class PlatformMediaCodecTest extends AwTestBase {
enableJavaScriptOnUiThread(mTestContainerView.getAwContents()); enableJavaScriptOnUiThread(mTestContainerView.getAwContents());
} }
/*
@MediumTest @MediumTest
@Feature({"AndroidWebView"}) @Feature({"AndroidWebView"})
*/
@DisabledTest(message = "crbug.com/620890")
public void testCanPlayPlatformMediaCodecs() throws Throwable { public void testCanPlayPlatformMediaCodecs() throws Throwable {
loadUrlSync(mTestContainerView.getAwContents(), mContentsClient.getOnPageFinishedHelper(), loadUrlSync(mTestContainerView.getAwContents(), mContentsClient.getOnPageFinishedHelper(),
"file:///android_asset/platform-media-codec-test.html"); "file:///android_asset/platform-media-codec-test.html");
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment