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

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

Issue 2229353003: Clean up document-mode specific UI. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 4 years, 4 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/externalnav/ExternalNavigationHandlerTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/externalnav/ExternalNavigationHandlerTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/externalnav/ExternalNavigationHandlerTest.java
index 69394748bbf43d514f80a325db90c02126782e6b..4a2928b62358f5d677df5d38773dc78ec71409a5 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/externalnav/ExternalNavigationHandlerTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/externalnav/ExternalNavigationHandlerTest.java
@@ -5,7 +5,6 @@
package org.chromium.chrome.browser.externalnav;
import android.annotation.SuppressLint;
-import android.content.Context;
import android.content.Intent;
import android.content.pm.ActivityInfo;
import android.content.pm.PackageManager;
@@ -26,7 +25,6 @@ import org.chromium.chrome.browser.IntentHandler;
import org.chromium.chrome.browser.externalnav.ExternalNavigationHandler.OverrideUrlLoadingResult;
import org.chromium.chrome.browser.tab.Tab;
import org.chromium.chrome.browser.tab.TabRedirectHandler;
-import org.chromium.chrome.browser.util.FeatureUtilities;
import org.chromium.ui.base.PageTransition;
import org.chromium.webapk.lib.common.WebApkConstants;
@@ -109,7 +107,6 @@ public class ExternalNavigationHandlerTest extends InstrumentationTestCase {
@Override
protected void setUp() throws Exception {
super.setUp();
- mDelegate.setContext(getInstrumentation().getTargetContext());
CommandLine.init(new String[0]);
RecordHistogram.disableForTests();
mDelegate.mQueryIntentOverride = null;
@@ -944,12 +941,6 @@ public class ExternalNavigationHandlerTest extends InstrumentationTestCase {
}
private static class TestExternalNavigationDelegate implements ExternalNavigationDelegate {
- private Context mContext;
-
- public void setContext(Context context) {
- mContext = context;
- }
-
@Override
public List<ResolveInfo> queryIntentActivities(Intent intent) {
List<ResolveInfo> list = new ArrayList<>();
@@ -1013,7 +1004,6 @@ public class ExternalNavigationHandlerTest extends InstrumentationTestCase {
if (infos == null) {
return result;
}
- int count = 0;
for (ResolveInfo info : infos) {
String packageName = info.activityInfo.packageName;
if (packageName.equals("youtube") || packageName.equals("calendar")
@@ -1099,11 +1089,6 @@ public class ExternalNavigationHandlerTest extends InstrumentationTestCase {
}
@Override
- public boolean isDocumentMode() {
- return FeatureUtilities.isDocumentMode(mContext);
- }
-
- @Override
public String getDefaultSmsPackageName() {
return defaultSmsPackageName;
}
@@ -1215,11 +1200,6 @@ public class ExternalNavigationHandlerTest extends InstrumentationTestCase {
return this;
}
- public ExternalNavigationTestParams withHasUserGesture(boolean hasUserGesture) {
- mHasUserGesture = hasUserGesture;
- return this;
- }
-
public ExternalNavigationTestParams withRedirectHandler(TabRedirectHandler handler) {
mRedirectHandler = handler;
return this;

Powered by Google App Engine
This is Rietveld 408576698