diff options
author | qinmin@chromium.org <qinmin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-07-09 23:00:29 +0000 |
---|---|---|
committer | qinmin@chromium.org <qinmin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-07-09 23:00:29 +0000 |
commit | 49fed9b90a097bd010bd2a0c16114bbe33fedd16 (patch) | |
tree | e3de289ec4fed1c393894097018f6ba950c574a2 | |
parent | 89aca92982d695849508f1906c17b225431b7a4f (diff) | |
download | chromium_src-49fed9b90a097bd010bd2a0c16114bbe33fedd16.zip chromium_src-49fed9b90a097bd010bd2a0c16114bbe33fedd16.tar.gz chromium_src-49fed9b90a097bd010bd2a0c16114bbe33fedd16.tar.bz2 |
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
-rw-r--r-- | android_webview/javatests/src/org/chromium/android_webview/test/AwContentsClientFullScreenVideoTest.java | 17 |
1 files changed, 0 insertions, 17 deletions
diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/AwContentsClientFullScreenVideoTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/AwContentsClientFullScreenVideoTest.java index 543358d..fb29ea5 100644 --- a/android_webview/javatests/src/org/chromium/android_webview/test/AwContentsClientFullScreenVideoTest.java +++ b/android_webview/javatests/src/org/chromium/android_webview/test/AwContentsClientFullScreenVideoTest.java @@ -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; |