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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/customtabs/CustomTabActivityTest.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
Index: chrome/android/javatests/src/org/chromium/chrome/browser/customtabs/CustomTabActivityTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/customtabs/CustomTabActivityTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/customtabs/CustomTabActivityTest.java
index a9b27f338e908fdbf15d0ad8822c3df12b5045e1..25361c603046edc3b82154364245c9900d05c819 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/customtabs/CustomTabActivityTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/customtabs/CustomTabActivityTest.java
@@ -250,7 +250,7 @@ public class CustomTabActivityTest extends CustomTabActivityTestBase {
return bundle;
}
- private void openAppMenuAndAssertMenuShown() throws InterruptedException {
+ private void openAppMenuAndAssertMenuShown() {
ThreadUtils.runOnUiThread(new Runnable() {
@Override
public void run() {
@@ -1796,16 +1796,12 @@ public class CustomTabActivityTest extends CustomTabActivityTestBase {
sessionReference.set(client.newSession(callback));
}
});
- try {
- CriteriaHelper.pollInstrumentationThread(new Criteria() {
- @Override
- public boolean isSatisfied() {
- return sessionReference.get() != null;
- }
- });
- } catch (InterruptedException e) {
- fail();
- }
+ CriteriaHelper.pollInstrumentationThread(new Criteria() {
+ @Override
+ public boolean isSatisfied() {
+ return sessionReference.get() != null;
+ }
+ });
return sessionReference.get();
}

Powered by Google App Engine
This is Rietveld 408576698