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

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

Issue 2833053002: Disable the flaky test CopylessPasteTest#testValid (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;
(...skipping 118 matching lines...) Expand 10 before | Expand all | Expand 10 after
129 loadUrl(mTestServer.getURL(NODATA_PAGE)); 129 loadUrl(mTestServer.getURL(NODATA_PAGE));
130 mCallbackHelper.waitForCallback(0); 130 mCallbackHelper.waitForCallback(0);
131 assertNull(mCallbackHelper.getWebPage()); 131 assertNull(mCallbackHelper.getWebPage());
132 } 132 }
133 133
134 /** 134 /**
135 * Tests that CopylessPaste works end-to-end. 135 * Tests that CopylessPaste works end-to-end.
136 */ 136 */
137 @LargeTest 137 @LargeTest
138 @Feature({"CopylessPaste"}) 138 @Feature({"CopylessPaste"})
139 @DisabledTest(message = "Flaky: crbug.com/713172")
139 public void testValid() throws InterruptedException, TimeoutException { 140 public void testValid() throws InterruptedException, TimeoutException {
140 loadUrl(mTestServer.getURL(DATA_PAGE)); 141 loadUrl(mTestServer.getURL(DATA_PAGE));
141 mCallbackHelper.waitForCallback(0); 142 mCallbackHelper.waitForCallback(0);
142 WebPage extracted = mCallbackHelper.getWebPage(); 143 WebPage extracted = mCallbackHelper.getWebPage();
143 144
144 WebPage expected = new WebPage(); 145 WebPage expected = new WebPage();
145 expected.url = new Url(); 146 expected.url = new Url();
146 expected.url.url = mTestServer.getURL(DATA_PAGE); 147 expected.url.url = mTestServer.getURL(DATA_PAGE);
147 expected.title = "JSON-LD for AppIndexing Test"; 148 expected.title = "JSON-LD for AppIndexing Test";
148 Entity e = new Entity(); 149 Entity e = new Entity();
(...skipping 28 matching lines...) Expand all
177 loadUrl(mTestServer.getURL(DATA_PAGE + uniqueTag)); 178 loadUrl(mTestServer.getURL(DATA_PAGE + uniqueTag));
178 mCallbackHelper.waitForCallback(1); 179 mCallbackHelper.waitForCallback(1);
179 180
180 // Cache hit without entities. Shouldn't parse again. 181 // Cache hit without entities. Shouldn't parse again.
181 loadUrl(mTestServer.getURL(NODATA_PAGE + uniqueTag)); 182 loadUrl(mTestServer.getURL(NODATA_PAGE + uniqueTag));
182 // Cache hit with entities. Shouldn't parse again. 183 // Cache hit with entities. Shouldn't parse again.
183 loadUrl(mTestServer.getURL(DATA_PAGE + uniqueTag)); 184 loadUrl(mTestServer.getURL(DATA_PAGE + uniqueTag));
184 assertEquals(2, mCallbackHelper.getCallCount()); 185 assertEquals(2, mCallbackHelper.getCallCount());
185 } 186 }
186 } 187 }
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