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

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

Issue 2431753003: Merge EmbeddedTestServer#createAndStartDefaultServer() and createAndStartFileServer() (Closed)
Patch Set: Merge branch 'master' into webapk_embedded_test_server Created 4 years, 2 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
Index: chrome/android/javatests/src/org/chromium/chrome/browser/TabsTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/TabsTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/TabsTest.java
index c99d743aa6eba598154b041bb9f581b833f3dd3e..ea17e7d27d85fb4576c5187d0b1aeda16b933348 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/TabsTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/TabsTest.java
@@ -9,7 +9,6 @@ import static org.chromium.base.test.util.Restriction.RESTRICTION_TYPE_NON_LOW_E
import android.content.DialogInterface;
import android.content.pm.ActivityInfo;
import android.os.Debug;
-import android.os.Environment;
import android.os.SystemClock;
import android.test.suitebuilder.annotation.LargeTest;
import android.test.suitebuilder.annotation.MediumTest;
@@ -142,8 +141,7 @@ public class TabsTest extends ChromeTabbedActivityTestBase {
@CommandLineFlags.Add(ContentSwitches.DISABLE_POPUP_BLOCKING)
@RetryOnFailure
public void testSpawnPopupOnBackgroundTab() throws InterruptedException, TimeoutException {
- mTestServer = EmbeddedTestServer.createAndStartFileServer(
- getInstrumentation().getContext(), Environment.getExternalStorageDirectory());
+ mTestServer = EmbeddedTestServer.createAndStartServer(getInstrumentation().getContext());
loadUrl(mTestServer.getURL(TEST_FILE_PATH));
final Tab tab = getActivity().getActivityTab();
@@ -171,8 +169,7 @@ public class TabsTest extends ChromeTabbedActivityTestBase {
@MediumTest
@RetryOnFailure
public void testAlertDialogDoesNotChangeActiveModel() throws InterruptedException {
- mTestServer = EmbeddedTestServer.createAndStartFileServer(
- getInstrumentation().getContext(), Environment.getExternalStorageDirectory());
+ mTestServer = EmbeddedTestServer.createAndStartServer(getInstrumentation().getContext());
newIncognitoTabFromMenu();
loadUrl(mTestServer.getURL(TEST_FILE_PATH));
final Tab tab = getActivity().getActivityTab();
@@ -230,8 +227,7 @@ public class TabsTest extends ChromeTabbedActivityTestBase {
*/
@DisabledTest
public void testOpenAndCloseNewTabButton() throws InterruptedException {
- mTestServer = EmbeddedTestServer.createAndStartFileServer(
- getInstrumentation().getContext(), Environment.getExternalStorageDirectory());
+ mTestServer = EmbeddedTestServer.createAndStartServer(getInstrumentation().getContext());
startMainActivityWithURL(mTestServer.getURL(TEST_FILE_PATH));
getInstrumentation().runOnMainSync(new Runnable() {
@Override
@@ -478,8 +474,7 @@ public class TabsTest extends ChromeTabbedActivityTestBase {
*/
@FlakyTest
public void testOpenManyTabsInBursts() throws InterruptedException {
- mTestServer = EmbeddedTestServer.createAndStartFileServer(
- getInstrumentation().getContext(), Environment.getExternalStorageDirectory());
+ mTestServer = EmbeddedTestServer.createAndStartServer(getInstrumentation().getContext());
final int burstSize = 5;
final String url = mTestServer.getURL(TEST_PAGE_FILE_PATH);
final int startCount = getActivity().getCurrentTabModel().getCount();
@@ -507,8 +502,7 @@ public class TabsTest extends ChromeTabbedActivityTestBase {
* tab loads when selected.
*/
private void openAndVerifyManyTestTabs(final int num) throws InterruptedException {
- mTestServer = EmbeddedTestServer.createAndStartFileServer(
- getInstrumentation().getContext(), Environment.getExternalStorageDirectory());
+ mTestServer = EmbeddedTestServer.createAndStartServer(getInstrumentation().getContext());
final String url = mTestServer.getURL(TEST_PAGE_FILE_PATH);
int startCount = getActivity().getCurrentTabModel().getCount();
loadUrlInManyNewTabs(url, num);
@@ -966,8 +960,7 @@ public class TabsTest extends ChromeTabbedActivityTestBase {
@Feature({"Android-TabSwitcher", "Main"})
@RetryOnFailure
public void testCloseTabPortrait() throws InterruptedException {
- mTestServer = EmbeddedTestServer.createAndStartFileServer(
- getInstrumentation().getContext(), Environment.getExternalStorageDirectory());
+ mTestServer = EmbeddedTestServer.createAndStartServer(getInstrumentation().getContext());
startMainActivityWithURL(mTestServer.getURL("/chrome/test/data/android/test.html"));
getActivity().setRequestedOrientation(ActivityInfo.SCREEN_ORIENTATION_PORTRAIT);
@@ -992,8 +985,7 @@ public class TabsTest extends ChromeTabbedActivityTestBase {
@Restriction({ChromeRestriction.RESTRICTION_TYPE_PHONE, RESTRICTION_TYPE_NON_LOW_END_DEVICE})
@RetryOnFailure
public void testCloseTabLandscape() throws InterruptedException {
- mTestServer = EmbeddedTestServer.createAndStartFileServer(
- getInstrumentation().getContext(), Environment.getExternalStorageDirectory());
+ mTestServer = EmbeddedTestServer.createAndStartServer(getInstrumentation().getContext());
startMainActivityWithURL(mTestServer.getURL("/chrome/test/data/android/test.html"));
getActivity().setRequestedOrientation(ActivityInfo.SCREEN_ORIENTATION_LANDSCAPE);
@@ -1170,8 +1162,7 @@ public class TabsTest extends ChromeTabbedActivityTestBase {
@Feature({"Android-TabSwitcher"})
@RetryOnFailure
public void testCloseTabDuringFling() throws InterruptedException {
- mTestServer = EmbeddedTestServer.createAndStartFileServer(
- getInstrumentation().getContext(), Environment.getExternalStorageDirectory());
+ mTestServer = EmbeddedTestServer.createAndStartServer(getInstrumentation().getContext());
loadUrlInNewTab(mTestServer.getURL(
"/chrome/test/data/android/tabstest/text_page.html"));
getInstrumentation().runOnMainSync(new Runnable() {
@@ -1192,8 +1183,7 @@ public class TabsTest extends ChromeTabbedActivityTestBase {
*/
@FlakyTest
public void testQuickSwitchBetweenTabAndSwitcherMode() throws InterruptedException {
- mTestServer = EmbeddedTestServer.createAndStartFileServer(
- getInstrumentation().getContext(), Environment.getExternalStorageDirectory());
+ mTestServer = EmbeddedTestServer.createAndStartServer(getInstrumentation().getContext());
final String[] urls = {
mTestServer.getURL("/chrome/test/data/android/navigate/one.html"),
mTestServer.getURL("/chrome/test/data/android/navigate/two.html"),

Powered by Google App Engine
This is Rietveld 408576698