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

Unified Diff: chrome/test/android/javatests/src/org/chromium/chrome/test/util/browser/sync/SyncTestUtil.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/test/android/javatests/src/org/chromium/chrome/test/util/browser/sync/SyncTestUtil.java
diff --git a/chrome/test/android/javatests/src/org/chromium/chrome/test/util/browser/sync/SyncTestUtil.java b/chrome/test/android/javatests/src/org/chromium/chrome/test/util/browser/sync/SyncTestUtil.java
index 9660c2f7fece160c231bb32fb50f4a92f166e4e2..7de8f3e950f5b64b0cf70c55951dfb982d205676 100644
--- a/chrome/test/android/javatests/src/org/chromium/chrome/test/util/browser/sync/SyncTestUtil.java
+++ b/chrome/test/android/javatests/src/org/chromium/chrome/test/util/browser/sync/SyncTestUtil.java
@@ -67,7 +67,7 @@ public final class SyncTestUtil {
/**
* Waits for sync to become active.
*/
- public static void waitForSyncActive() throws InterruptedException {
+ public static void waitForSyncActive() {
CriteriaHelper.pollUiThread(new Criteria("Timed out waiting for sync to become active.") {
@Override
public boolean isSatisfied() {
@@ -79,7 +79,7 @@ public final class SyncTestUtil {
/**
* Waits for sync's engine to be initialized.
*/
- public static void waitForEngineInitialized() throws InterruptedException {
+ public static void waitForEngineInitialized() {
CriteriaHelper.pollUiThread(
new Criteria("Timed out waiting for sync's engine to initialize.") {
@Override
@@ -110,7 +110,7 @@ public final class SyncTestUtil {
* one that it triggered. Therefore this method should only be used where it can result in
* false positives (e.g. checking that something doesn't sync), not false negatives.
*/
- public static void triggerSyncAndWaitForCompletion() throws InterruptedException {
+ public static void triggerSyncAndWaitForCompletion() {
final long oldSyncTime = getCurrentSyncTime();
triggerSync();
CriteriaHelper.pollInstrumentationThread(new Criteria(

Powered by Google App Engine
This is Rietveld 408576698