Commit 49fed9b9 authored by qinmin@chromium.org's avatar qinmin@chromium.org

Revert of Temporarily disable WebView fullscreen video tests until we land...

Revert of Temporarily disable WebView fullscreen video tests until we land fullscreen IPC changes (https://codereview.chromium.org/364043002/)

Reason for revert:
revert this as all related CLs are submitted

Original issue's description:
> Temporarily disable WebView fullscreen video tests until we land fullscreen IPC changes
> 
> there are 2 changes needs to be submitted before we reenable the tests:
> 1. https://codereview.chromium.org/290643005/ and
> 2. https://codereview.chromium.org/292573004/
> 
> the blink change will break all the tests, so we need to reenable the tests after the chromium change lands
> 
> BUG=367346
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=281165

TBR=boliu@chromium.org
BUG=367346

Review URL: https://codereview.chromium.org/383433002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@282158 0039d316-1c4b-4281-b951-d872f2087c98
parent 89aca929
......@@ -12,7 +12,6 @@ import junit.framework.Assert;
import org.chromium.android_webview.test.util.VideoTestWebServer;
import org.chromium.base.CommandLine;
import org.chromium.base.test.util.DisabledTest;
import org.chromium.base.test.util.Feature;
import org.chromium.content.browser.ContentViewCore;
import org.chromium.content.browser.test.util.DOMUtils;
......@@ -46,12 +45,8 @@ public class AwContentsClientFullScreenVideoTest extends AwTestBase {
if (webServer != null) webServer.getTestWebServer().shutdown();
}
/*
@MediumTest
@Feature({"AndroidWebView"})
crbug.com/367346
*/
@DisabledTest
public void testOnShowAndHideCustomViewWithCallback() throws Throwable {
doOnShowAndHideCustomViewTest(new Runnable() {
@Override
......@@ -61,12 +56,8 @@ public class AwContentsClientFullScreenVideoTest extends AwTestBase {
});
}
/*
@MediumTest
@Feature({"AndroidWebView"})
crbug.com/367346
*/
@DisabledTest
public void testOnShowAndHideCustomViewWithBackKeyLegacy() throws Throwable {
// When html controls are enabled we skip this test because pressing the back key
// moves away from the current activity instead of exiting fullscreen mode.
......@@ -85,12 +76,8 @@ public class AwContentsClientFullScreenVideoTest extends AwTestBase {
});
}
/*
@MediumTest
@Feature({"AndroidWebView"})
crbug.com/367346
*/
@DisabledTest
public void testOnShowAndHideCustomViewWithJavascript() throws Throwable {
doOnShowAndHideCustomViewTest(new Runnable() {
@Override
......@@ -100,12 +87,8 @@ public class AwContentsClientFullScreenVideoTest extends AwTestBase {
});
}
/*
@MediumTest
@Feature({"AndroidWebView"})
crbug.com/367346
*/
@DisabledTest
public void testOnShowCustomViewAndPlayWithHtmlControl() throws Throwable {
if (!areHtmlControlsEnabled())
return;
......
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