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

Side by Side Diff: chrome/android/javatests/src/org/chromium/chrome/browser/CopylessPasteTest.java

Issue 2830673003: Disable the flaky test CopylessPasteTest#testCache (Closed)
Patch Set: Created 3 years, 8 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 package org.chromium.chrome.browser; 5 package org.chromium.chrome.browser;
6 6
7 import android.support.test.filters.LargeTest; 7 import android.support.test.filters.LargeTest;
8 8
9 import org.chromium.base.Callback; 9 import org.chromium.base.Callback;
10 import org.chromium.base.ThreadUtils; 10 import org.chromium.base.ThreadUtils;
11 import org.chromium.base.test.util.CallbackHelper; 11 import org.chromium.base.test.util.CallbackHelper;
12 import org.chromium.base.test.util.CommandLineFlags; 12 import org.chromium.base.test.util.CommandLineFlags;
13 import org.chromium.base.test.util.DisabledTest;
13 import org.chromium.base.test.util.Feature; 14 import org.chromium.base.test.util.Feature;
14 import org.chromium.base.test.util.Restriction; 15 import org.chromium.base.test.util.Restriction;
15 import org.chromium.blink.mojom.document_metadata.Entity; 16 import org.chromium.blink.mojom.document_metadata.Entity;
16 import org.chromium.blink.mojom.document_metadata.Property; 17 import org.chromium.blink.mojom.document_metadata.Property;
17 import org.chromium.blink.mojom.document_metadata.Values; 18 import org.chromium.blink.mojom.document_metadata.Values;
18 import org.chromium.blink.mojom.document_metadata.WebPage; 19 import org.chromium.blink.mojom.document_metadata.WebPage;
19 import org.chromium.chrome.browser.firstrun.FirstRunStatus; 20 import org.chromium.chrome.browser.firstrun.FirstRunStatus;
20 import org.chromium.chrome.test.ChromeTabbedActivityTestBase; 21 import org.chromium.chrome.test.ChromeTabbedActivityTestBase;
21 import org.chromium.chrome.test.util.ChromeTabUtils; 22 import org.chromium.chrome.test.util.ChromeTabUtils;
22 import org.chromium.net.test.EmbeddedTestServer; 23 import org.chromium.net.test.EmbeddedTestServer;
(...skipping 135 matching lines...) Expand 10 before | Expand all | Expand 10 after
158 expected.entities = new Entity[1]; 159 expected.entities = new Entity[1];
159 expected.entities[0] = e; 160 expected.entities[0] = e;
160 assertEquals(expected, extracted); 161 assertEquals(expected, extracted);
161 } 162 }
162 163
163 /** 164 /**
164 * Tests that CopylessPaste skips parsing visited pages. 165 * Tests that CopylessPaste skips parsing visited pages.
165 */ 166 */
166 @LargeTest 167 @LargeTest
167 @Feature({"CopylessPaste"}) 168 @Feature({"CopylessPaste"})
169 @DisabledTest(message = "Flaky: crbug.com/713172")
168 public void testCache() throws InterruptedException, TimeoutException { 170 public void testCache() throws InterruptedException, TimeoutException {
169 // The URLs used here should be unique in CopylessPasteTest. 171 // The URLs used here should be unique in CopylessPasteTest.
170 String uniqueTag = "#123"; 172 String uniqueTag = "#123";
171 // NODATA_PAGE doesn't contain desired metadata. 173 // NODATA_PAGE doesn't contain desired metadata.
172 loadUrl(mTestServer.getURL(NODATA_PAGE + uniqueTag)); 174 loadUrl(mTestServer.getURL(NODATA_PAGE + uniqueTag));
173 mCallbackHelper.waitForCallback(0); 175 mCallbackHelper.waitForCallback(0);
174 // DATA_PAGE contains desired metadata. 176 // DATA_PAGE contains desired metadata.
175 loadUrl(mTestServer.getURL(DATA_PAGE + uniqueTag)); 177 loadUrl(mTestServer.getURL(DATA_PAGE + uniqueTag));
176 mCallbackHelper.waitForCallback(1); 178 mCallbackHelper.waitForCallback(1);
177 179
178 // Cache hit without entities. Shouldn't parse again. 180 // Cache hit without entities. Shouldn't parse again.
179 loadUrl(mTestServer.getURL(NODATA_PAGE + uniqueTag)); 181 loadUrl(mTestServer.getURL(NODATA_PAGE + uniqueTag));
180 // Cache hit with entities. Shouldn't parse again. 182 // Cache hit with entities. Shouldn't parse again.
181 loadUrl(mTestServer.getURL(DATA_PAGE + uniqueTag)); 183 loadUrl(mTestServer.getURL(DATA_PAGE + uniqueTag));
182 assertEquals(2, mCallbackHelper.getCallCount()); 184 assertEquals(2, mCallbackHelper.getCallCount());
183 } 185 }
184 } 186 }
OLDNEW
« 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