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

Unified Diff: chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncTest.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/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncTest.java
diff --git a/chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncTest.java b/chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncTest.java
index 33a1b3bba3ebc77258369f5a4cf89f171bac79c6..48ae00f189765604ae669984f90ce30de843bcd3 100644
--- a/chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncTest.java
+++ b/chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncTest.java
@@ -67,7 +67,7 @@ public class SyncTest extends SyncTestBase {
@LargeTest
@Feature({"Sync"})
- public void testStopAndClear() throws InterruptedException {
+ public void testStopAndClear() {
setUpTestAccountAndSignIn();
CriteriaHelper.pollUiThread(
new Criteria("Timed out checking that isSignedInOnNative() == true") {
@@ -99,7 +99,7 @@ public class SyncTest extends SyncTestBase {
* @Feature({"Sync"})
*/
@DisabledTest(message = "crbug.com/588050,crbug.com/595893")
- public void testRename() throws InterruptedException {
+ public void testRename() {
// The two accounts object that would represent the account rename.
final Account oldAccount = setUpTestAccountAndSignIn();
final Account newAccount = SigninTestUtil.addTestAccount("test2@gmail.com");
@@ -146,7 +146,7 @@ public class SyncTest extends SyncTestBase {
@LargeTest
@Feature({"Sync"})
- public void testStopAndStartSync() throws InterruptedException {
+ public void testStopAndStartSync() {
Account account = setUpTestAccountAndSignIn();
stopSync();
@@ -161,7 +161,7 @@ public class SyncTest extends SyncTestBase {
* @Feature({"Sync"})
*/
@FlakyTest(message = "crbug.com/594558")
- public void testStopAndStartSyncThroughAndroid() throws InterruptedException {
+ public void testStopAndStartSyncThroughAndroid() {
Account account = setUpTestAccountAndSignIn();
String authority = AndroidSyncSettings.getContractAuthority(mContext);
@@ -198,7 +198,7 @@ public class SyncTest extends SyncTestBase {
@LargeTest
@Feature({"Sync"})
- public void testMasterSyncBlocksSyncStart() throws InterruptedException {
+ public void testMasterSyncBlocksSyncStart() {
setUpTestAccountAndSignIn();
stopSync();
assertFalse(SyncTestUtil.isSyncRequested());

Powered by Google App Engine
This is Rietveld 408576698