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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/UrlManagerTest.java

Issue 2652703002: Mark UrlManagerTest#testAddUrlGarbageCollectsForSize flaky (Closed)
Patch Set: Created 3 years, 11 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/UrlManagerTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/UrlManagerTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/UrlManagerTest.java
index db518c57db22f8cfb42f43aa011d8359e84dca3e..f5326fd8b085d79888958ce26be33685a2652c78 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/UrlManagerTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/UrlManagerTest.java
@@ -9,6 +9,7 @@ import android.support.test.filters.SmallTest;
import android.test.InstrumentationTestCase;
import org.chromium.base.ContextUtils;
+import org.chromium.base.test.util.DisabledTest
import org.chromium.base.test.util.FlakyTest;
import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.chrome.test.util.browser.notifications.MockNotificationManagerProxy;
@@ -224,8 +225,11 @@ public class UrlManagerTest extends InstrumentationTestCase {
assertEquals(0, mMockNotificationManagerProxy.getNotifications().size());
}
- @SmallTest
- @RetryOnFailure
+ /*
+ * @SmallTest
+ * Bug=crbug.com/683923
+ */
+ @DisabledTest
public void testAddUrlGarbageCollectsForSize() throws Exception {
// Add and remove 101 URLs, making sure one is clearly slightly older than the others.
addEmptyPwsResult();
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698