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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadTest.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/download/DownloadTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadTest.java
index 869fbf4012641203f92d1a9f3c723190ebe996f7..17258481ab42479afcf5af3671499aa57c2ef9fb 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadTest.java
@@ -12,6 +12,7 @@ import org.chromium.base.Log;
import org.chromium.base.test.util.DisabledTest;
import org.chromium.base.test.util.Feature;
import org.chromium.base.test.util.FlakyTest;
+import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.chrome.R;
import org.chromium.chrome.browser.compositor.CompositorViewHolder;
import org.chromium.chrome.browser.compositor.layouts.LayoutManager;
@@ -74,6 +75,7 @@ public class DownloadTest extends DownloadTestBase {
@MediumTest
@Feature({"Downloads"})
+ @RetryOnFailure
public void testHttpGetDownload() throws Exception {
loadUrl(mTestServer.getURL(TEST_DOWNLOAD_DIRECTORY + "get.html"));
waitForFocus();
@@ -90,6 +92,7 @@ public class DownloadTest extends DownloadTestBase {
@MediumTest
@Feature({"Downloads"})
+ @RetryOnFailure
public void testDangerousDownload() throws Exception {
loadUrl(mTestServer.getURL(TEST_DOWNLOAD_DIRECTORY + "dangerous.html"));
waitForFocus();
@@ -107,6 +110,7 @@ public class DownloadTest extends DownloadTestBase {
@MediumTest
@Feature({"Downloads"})
+ @RetryOnFailure
public void testDangerousDownloadCancel() throws Exception {
loadUrl(mTestServer.getURL(TEST_DOWNLOAD_DIRECTORY + "dangerous.html"));
waitForFocus();
@@ -136,6 +140,7 @@ public class DownloadTest extends DownloadTestBase {
@MediumTest
@Feature({"Downloads"})
+ @RetryOnFailure
public void testHttpPostDownload() throws Exception {
loadUrl(mTestServer.getURL(TEST_DOWNLOAD_DIRECTORY + "post.html"));
waitForFocus();
@@ -176,6 +181,7 @@ public class DownloadTest extends DownloadTestBase {
@MediumTest
@Feature({"Downloads"})
+ @RetryOnFailure
public void testDuplicateHttpPostDownload_Overwrite() throws Exception {
// Snackbar overlaps the infobar which is clicked in this test.
getActivity().getSnackbarManager().disableForTesting();
@@ -401,6 +407,7 @@ public class DownloadTest extends DownloadTestBase {
@MediumTest
@Feature({"Downloads"})
+ @RetryOnFailure
public void testUrlEscaping() throws Exception {
loadUrl(mTestServer.getURL(TEST_DOWNLOAD_DIRECTORY + "urlescaping.html"));
waitForFocus();

Powered by Google App Engine
This is Rietveld 408576698