Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1487)

Unified Diff: android_webview/javatests/src/org/chromium/android_webview/test/AwContentsClientFullScreenTest.java

Issue 2573263002: Catch InterruptedException in CriteriaHelper (Closed)
Patch Set: Change OverviewModeBehaviorWatcher back to try/finally Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | android_webview/javatests/src/org/chromium/android_webview/test/AwContentsGarbageCollectionTest.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: android_webview/javatests/src/org/chromium/android_webview/test/AwContentsClientFullScreenTest.java
diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/AwContentsClientFullScreenTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/AwContentsClientFullScreenTest.java
index 540d02dadd6fa3ef1c41db05f8a4c7b2ceb8e3b8..d62b7bf4a42618210ea8c96dee48feddd7ffd648 100644
--- a/android_webview/javatests/src/org/chromium/android_webview/test/AwContentsClientFullScreenTest.java
+++ b/android_webview/javatests/src/org/chromium/android_webview/test/AwContentsClientFullScreenTest.java
@@ -363,8 +363,7 @@ public class AwContentsClientFullScreenTest extends AwTestBase {
* Asserts that the keep screen on property in the given {@code view} is active as
* {@code expected}. It also verifies that it is only active when the video is playing.
*/
- private void assertWaitForKeepScreenOnActive(final View view, final boolean expected)
- throws InterruptedException {
+ private void assertWaitForKeepScreenOnActive(final View view, final boolean expected) {
// We need to poll because it takes time to synchronize the state between the android
// views and Javascript.
CriteriaHelper.pollInstrumentationThread(new Criteria() {
@@ -416,7 +415,7 @@ public class AwContentsClientFullScreenTest extends AwTestBase {
return view.getKeepScreenOn();
}
- private void assertWaitForIsFullscreen() throws InterruptedException {
+ private void assertWaitForIsFullscreen() {
// We need to poll because the Javascript state is updated asynchronously
CriteriaHelper.pollInstrumentationThread(new Criteria() {
@Override
@@ -431,7 +430,7 @@ public class AwContentsClientFullScreenTest extends AwTestBase {
});
}
- private void assertWaitForIsEmbedded() throws InterruptedException {
+ private void assertWaitForIsEmbedded() {
// We need to poll because the Javascript state is updated asynchronously
CriteriaHelper.pollInstrumentationThread(new Criteria() {
@Override
« no previous file with comments | « no previous file | android_webview/javatests/src/org/chromium/android_webview/test/AwContentsGarbageCollectionTest.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698