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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/TabModelMergingTest.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/tabmodel/TabModelMergingTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/TabModelMergingTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/TabModelMergingTest.java
index fcd849d19a13f3f9373e053d29b96c7cd74aa50a..86546f5ab37e938e99dff8be18855ce15b011788 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/TabModelMergingTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/TabModelMergingTest.java
@@ -155,7 +155,7 @@ public class TabModelMergingTest extends ChromeTabbedActivityTestBase {
}
private void mergeTabsAndAssert(final ChromeTabbedActivity activity,
- final String[] expectedTabUrls) throws InterruptedException {
+ final String[] expectedTabUrls) {
String selectedTabUrl = activity.getTabModelSelector().getCurrentTab().getUrl();
mergeTabsAndAssert(activity, expectedTabUrls, expectedTabUrls.length, selectedTabUrl);
}
@@ -169,7 +169,7 @@ public class TabModelMergingTest extends ChromeTabbedActivityTestBase {
*/
private void mergeTabsAndAssert(final ChromeTabbedActivity activity,
final String[] expectedTabUrls, final int expectedNumberOfTabs,
- String expectedSelectedTabUrl) throws InterruptedException {
+ String expectedSelectedTabUrl) {
// Merge tabs into the activity.
ThreadUtils.runOnUiThreadBlocking(new Runnable() {
@Override
@@ -208,8 +208,7 @@ public class TabModelMergingTest extends ChromeTabbedActivityTestBase {
* @return The newly created ChromeTabbedActivity.
*/
private ChromeTabbedActivity startNewChromeTabbedActivityAndAssert(Intent intent,
- String expectedSelectedTabUrl, String[] expectedTabUrls)
- throws InterruptedException {
+ String expectedSelectedTabUrl, String[] expectedTabUrls) {
final ChromeTabbedActivity newActivity =
(ChromeTabbedActivity) getInstrumentation().startActivitySync(intent);

Powered by Google App Engine
This is Rietveld 408576698