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

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

Issue 1856273002: Change all FlakyTest import to use org.chromium.base.test.util.FlakyTest (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 static org.chromium.base.test.util.Restriction.RESTRICTION_TYPE_NON_LOW_E ND_DEVICE; 7 import static org.chromium.base.test.util.Restriction.RESTRICTION_TYPE_NON_LOW_E ND_DEVICE;
8 8
9 import android.os.Environment; 9 import android.os.Environment;
10 import android.test.FlakyTest;
11 import android.test.MoreAsserts; 10 import android.test.MoreAsserts;
12 import android.test.suitebuilder.annotation.LargeTest; 11 import android.test.suitebuilder.annotation.LargeTest;
13 12
14 import org.chromium.base.ThreadUtils; 13 import org.chromium.base.ThreadUtils;
15 import org.chromium.base.test.util.DisabledTest; 14 import org.chromium.base.test.util.DisabledTest;
16 import org.chromium.base.test.util.Feature; 15 import org.chromium.base.test.util.Feature;
16 import org.chromium.base.test.util.FlakyTest;
17 import org.chromium.base.test.util.Restriction; 17 import org.chromium.base.test.util.Restriction;
18 import org.chromium.chrome.browser.prerender.ExternalPrerenderHandler; 18 import org.chromium.chrome.browser.prerender.ExternalPrerenderHandler;
19 import org.chromium.chrome.browser.tab.Tab; 19 import org.chromium.chrome.browser.tab.Tab;
20 import org.chromium.chrome.test.ChromeTabbedActivityTestBase; 20 import org.chromium.chrome.test.ChromeTabbedActivityTestBase;
21 import org.chromium.chrome.test.util.PrerenderTestHelper; 21 import org.chromium.chrome.test.util.PrerenderTestHelper;
22 import org.chromium.chrome.test.util.browser.TabTitleObserver; 22 import org.chromium.chrome.test.util.browser.TabTitleObserver;
23 import org.chromium.net.test.EmbeddedTestServer; 23 import org.chromium.net.test.EmbeddedTestServer;
24 import org.chromium.ui.base.PageTransition; 24 import org.chromium.ui.base.PageTransition;
25 25
26 import java.util.concurrent.TimeoutException; 26 import java.util.concurrent.TimeoutException;
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
66 typeInOmnibox(testUrl, true); 66 typeInOmnibox(testUrl, true);
67 67
68 assertFalse("URL should not have been prerendered.", 68 assertFalse("URL should not have been prerendered.",
69 PrerenderTestHelper.waitForPrerenderUrl(tab, testUrl, true)); 69 PrerenderTestHelper.waitForPrerenderUrl(tab, testUrl, true));
70 // Navigate should not use the prerendered version. 70 // Navigate should not use the prerendered version.
71 assertEquals(TabLoadStatus.DEFAULT_PAGE_LOAD, 71 assertEquals(TabLoadStatus.DEFAULT_PAGE_LOAD,
72 loadUrlInTab(testUrl, PageTransition.TYPED | PageTransition.FROM _ADDRESS_BAR, tab)); 72 loadUrlInTab(testUrl, PageTransition.TYPED | PageTransition.FROM _ADDRESS_BAR, tab));
73 } 73 }
74 74
75 /* 75 /*
76 crbug.com/339668
77 @LargeTest 76 @LargeTest
78 @Restriction({RESTRICTION_TYPE_NON_LOW_END_DEVICE}) 77 @Restriction({RESTRICTION_TYPE_NON_LOW_END_DEVICE})
79 @Feature({"TabContents"}) 78 @Feature({"TabContents"})
80 */ 79 */
81 @FlakyTest 80 @FlakyTest(message = "crbug.com/339668")
82 public void testPrerenderNotDead() throws InterruptedException, TimeoutExcep tion { 81 public void testPrerenderNotDead() throws InterruptedException, TimeoutExcep tion {
83 String testUrl = mTestServer.getURL( 82 String testUrl = mTestServer.getURL(
84 "/chrome/test/data/android/prerender/google.html"); 83 "/chrome/test/data/android/prerender/google.html");
85 final Tab tab = getActivity().getActivityTab(); 84 final Tab tab = getActivity().getActivityTab();
86 PrerenderTestHelper.prerenderUrl(testUrl, tab); 85 PrerenderTestHelper.prerenderUrl(testUrl, tab);
87 // Navigate should use the prerendered version. 86 // Navigate should use the prerendered version.
88 assertEquals(TabLoadStatus.FULL_PRERENDERED_PAGE_LOAD, loadUrl(testUrl)) ; 87 assertEquals(TabLoadStatus.FULL_PRERENDERED_PAGE_LOAD, loadUrl(testUrl)) ;
89 88
90 // Prerender again with new text; make sure we get something different. 89 // Prerender again with new text; make sure we get something different.
91 String newTitle = "Welcome to the YouTube"; 90 String newTitle = "Welcome to the YouTube";
(...skipping 51 matching lines...) Expand 10 before | Expand all | Expand 10 after
143 handler.cancelCurrentPrerender(); 142 handler.cancelCurrentPrerender();
144 } 143 }
145 }); 144 });
146 } 145 }
147 146
148 @Override 147 @Override
149 public void startMainActivity() throws InterruptedException { 148 public void startMainActivity() throws InterruptedException {
150 startMainActivityOnBlankPage(); 149 startMainActivityOnBlankPage();
151 } 150 }
152 } 151 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698