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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/externalnav/ExternalNavigationDelegateImplTest.java

Issue 2331533002: Mark flaky chrome_public tests with @RetryOnFailure Batch 2 (Closed)
Patch Set: Created 4 years, 3 months 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/externalnav/ExternalNavigationDelegateImplTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/externalnav/ExternalNavigationDelegateImplTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/externalnav/ExternalNavigationDelegateImplTest.java
index fd51d4a214b71b80cb3e4c4bda3ccc836a65845d..1fb1f393ef1a6535d5849ffb0bfb024860c000d9 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/externalnav/ExternalNavigationDelegateImplTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/externalnav/ExternalNavigationDelegateImplTest.java
@@ -10,6 +10,7 @@ import android.content.pm.ResolveInfo;
import android.test.suitebuilder.annotation.SmallTest;
import org.chromium.base.test.util.CommandLineFlags;
+import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.chrome.browser.ChromeActivity;
import org.chromium.chrome.test.ChromeActivityTestCaseBase;
@@ -31,6 +32,7 @@ public class ExternalNavigationDelegateImplTest extends ChromeActivityTestCaseBa
}
@SmallTest
+ @RetryOnFailure
public void testIsPackageSpecializedHandler_NoResolveInfo() {
String packageName = "";
List<ResolveInfo> resolveInfos = new ArrayList<ResolveInfo>();
@@ -39,6 +41,7 @@ public class ExternalNavigationDelegateImplTest extends ChromeActivityTestCaseBa
}
@SmallTest
+ @RetryOnFailure
public void testIsPackageSpecializedHandler_NoPathOrAuthority() {
String packageName = "";
ResolveInfo info = new ResolveInfo();
@@ -49,6 +52,7 @@ public class ExternalNavigationDelegateImplTest extends ChromeActivityTestCaseBa
}
@SmallTest
+ @RetryOnFailure
public void testIsPackageSpecializedHandler_WithPath() {
String packageName = "";
ResolveInfo info = new ResolveInfo();
@@ -60,6 +64,7 @@ public class ExternalNavigationDelegateImplTest extends ChromeActivityTestCaseBa
}
@SmallTest
+ @RetryOnFailure
public void testIsPackageSpecializedHandler_WithAuthority() {
String packageName = "";
ResolveInfo info = new ResolveInfo();
@@ -71,6 +76,7 @@ public class ExternalNavigationDelegateImplTest extends ChromeActivityTestCaseBa
}
@SmallTest
+ @RetryOnFailure
public void testIsPackageSpecializedHandler_WithTargetPackage_Matching() {
String packageName = "com.android.chrome";
ResolveInfo info = new ResolveInfo();
@@ -84,6 +90,7 @@ public class ExternalNavigationDelegateImplTest extends ChromeActivityTestCaseBa
}
@SmallTest
+ @RetryOnFailure
public void testIsPackageSpecializedHandler_WithTargetPackage_NotMatching() {
String packageName = "com.android.chrome";
ResolveInfo info = new ResolveInfo();
@@ -98,6 +105,7 @@ public class ExternalNavigationDelegateImplTest extends ChromeActivityTestCaseBa
@SmallTest
@CommandLineFlags.Add({"disable-features=SystemDownloadManager"})
+ @RetryOnFailure
public void testIsDownload_noSystemDownloadManager() throws Exception {
ExternalNavigationDelegateImpl delegate = new ExternalNavigationDelegateImpl(
getActivity().getActivityTab());

Powered by Google App Engine
This is Rietveld 408576698