Index: chrome/android/javatests/src/org/chromium/chrome/browser/toolbar/ToolbarTest.java |
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/toolbar/ToolbarTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/toolbar/ToolbarTest.java |
index 73edd89b4d1b7c96874e17737584e02332606920..42838fd0021d33993ddc5739e73b9bb64d17bf23 100644 |
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/toolbar/ToolbarTest.java |
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/toolbar/ToolbarTest.java |
@@ -4,28 +4,18 @@ |
package org.chromium.chrome.browser.toolbar; |
-import android.support.test.InstrumentationRegistry; |
import android.support.test.filters.MediumTest; |
-import org.junit.Assert; |
-import org.junit.Before; |
-import org.junit.Rule; |
-import org.junit.Test; |
-import org.junit.runner.RunWith; |
- |
import org.chromium.base.ThreadUtils; |
-import org.chromium.base.test.util.CommandLineFlags; |
import org.chromium.base.test.util.Feature; |
import org.chromium.base.test.util.Restriction; |
import org.chromium.chrome.R; |
import org.chromium.chrome.browser.ChromeActivity; |
-import org.chromium.chrome.browser.ChromeSwitches; |
import org.chromium.chrome.browser.UrlConstants; |
import org.chromium.chrome.browser.omnibox.UrlBar; |
import org.chromium.chrome.browser.tab.Tab; |
import org.chromium.chrome.browser.widget.findinpage.FindToolbar; |
-import org.chromium.chrome.test.ChromeActivityTestRule; |
-import org.chromium.chrome.test.ChromeJUnit4ClassRunner; |
+import org.chromium.chrome.test.ChromeActivityTestCaseBase; |
import org.chromium.chrome.test.util.ChromeRestriction; |
import org.chromium.chrome.test.util.MenuUtils; |
import org.chromium.chrome.test.util.OmniboxTestUtils; |
@@ -37,24 +27,21 @@ |
/** |
* Tests for toolbar manager behavior. |
*/ |
-@RunWith(ChromeJUnit4ClassRunner.class) |
-@CommandLineFlags.Add({ChromeSwitches.DISABLE_FIRST_RUN_EXPERIENCE, |
- ChromeActivityTestRule.DISABLE_NETWORK_PREDICTION_FLAG}) |
-public class ToolbarTest { |
- @Rule |
- public ChromeActivityTestRule<ChromeActivity> mActivityTestRule = |
- new ChromeActivityTestRule<>(ChromeActivity.class); |
- |
+public class ToolbarTest extends ChromeActivityTestCaseBase<ChromeActivity> { |
private static final String TEST_PAGE = "/chrome/test/data/android/test.html"; |
- @Before |
- public void setUp() throws InterruptedException { |
- mActivityTestRule.startMainActivityOnBlankPage(); |
+ public ToolbarTest() { |
+ super(ChromeActivity.class); |
+ } |
+ |
+ @Override |
+ public void startMainActivity() throws InterruptedException { |
+ startMainActivityOnBlankPage(); |
} |
private void findInPageFromMenu() { |
- MenuUtils.invokeCustomMenuActionSync(InstrumentationRegistry.getInstrumentation(), |
- mActivityTestRule.getActivity(), R.id.find_in_page_id); |
+ MenuUtils.invokeCustomMenuActionSync(getInstrumentation(), |
+ getActivity(), R.id.find_in_page_id); |
waitForFindInPageVisibility(true); |
} |
@@ -63,9 +50,8 @@ |
CriteriaHelper.pollUiThread(new Criteria() { |
@Override |
public boolean isSatisfied() { |
- FindToolbar findToolbar = |
- (FindToolbar) mActivityTestRule.getActivity().findViewById( |
- R.id.find_toolbar); |
+ FindToolbar findToolbar = (FindToolbar) getActivity().findViewById( |
+ R.id.find_toolbar); |
boolean isVisible = findToolbar != null && findToolbar.isShown(); |
return (visible == isVisible) && !findToolbar.isAnimating(); |
@@ -84,19 +70,18 @@ |
return isShowingError[0]; |
} |
- @Test |
@MediumTest |
public void testNTPNavigatesToErrorPageOnDisconnectedNetwork() throws Exception { |
- EmbeddedTestServer testServer = EmbeddedTestServer.createAndStartServer( |
- InstrumentationRegistry.getInstrumentation().getContext()); |
+ EmbeddedTestServer testServer = |
+ EmbeddedTestServer.createAndStartServer(getInstrumentation().getContext()); |
String testUrl = testServer.getURL(TEST_PAGE); |
- Tab tab = mActivityTestRule.getActivity().getActivityTab(); |
+ Tab tab = getActivity().getActivityTab(); |
// Load new tab page. |
- mActivityTestRule.loadUrl(UrlConstants.NTP_URL); |
- Assert.assertEquals(UrlConstants.NTP_URL, tab.getUrl()); |
- Assert.assertFalse(isErrorPage(tab)); |
+ loadUrl(UrlConstants.NTP_URL); |
+ assertEquals(UrlConstants.NTP_URL, tab.getUrl()); |
+ assertFalse(isErrorPage(tab)); |
// Stop the server and also disconnect the network. |
testServer.stopAndDestroyServer(); |
@@ -107,19 +92,18 @@ |
} |
}); |
- mActivityTestRule.loadUrl(testUrl); |
- Assert.assertEquals(testUrl, tab.getUrl()); |
- Assert.assertTrue(isErrorPage(tab)); |
+ loadUrl(testUrl); |
+ assertEquals(testUrl, tab.getUrl()); |
+ assertTrue(isErrorPage(tab)); |
} |
- @Test |
@MediumTest |
@Restriction(ChromeRestriction.RESTRICTION_TYPE_TABLET) |
@Feature({"Omnibox"}) |
public void testFindInPageDismissedOnOmniboxFocus() { |
findInPageFromMenu(); |
- UrlBar urlBar = (UrlBar) mActivityTestRule.getActivity().findViewById(R.id.url_bar); |
+ UrlBar urlBar = (UrlBar) getActivity().findViewById(R.id.url_bar); |
OmniboxTestUtils.toggleUrlBarFocus(urlBar, true); |
OmniboxTestUtils.waitForFocusAndKeyboardActive(urlBar, true); |