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

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

Issue 2096073002: Record how often we end up in Chrome from external navigations. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add null check for activity info. Created 4 years, 6 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/ExternalNavigationDelegateImplTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/externalnav/ExternalNavigationDelegateImplTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/externalnav/ExternalNavigationDelegateImplTest.java
index 34e320c4ec78fad6f3fc09e2042d13d87d3c9cb0..fd51d4a214b71b80cb3e4c4bda3ccc836a65845d 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/externalnav/ExternalNavigationDelegateImplTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/externalnav/ExternalNavigationDelegateImplTest.java
@@ -34,8 +34,8 @@ public class ExternalNavigationDelegateImplTest extends ChromeActivityTestCaseBa
public void testIsPackageSpecializedHandler_NoResolveInfo() {
String packageName = "";
List<ResolveInfo> resolveInfos = new ArrayList<ResolveInfo>();
- assertEquals(0, ExternalNavigationDelegateImpl.countSpecializedHandlersWithFilter(
- resolveInfos, packageName));
+ assertEquals(0, ExternalNavigationDelegateImpl.getSpecializedHandlersWithFilter(
+ resolveInfos, packageName).size());
}
@SmallTest
@@ -44,8 +44,8 @@ public class ExternalNavigationDelegateImplTest extends ChromeActivityTestCaseBa
ResolveInfo info = new ResolveInfo();
info.filter = new IntentFilter();
List<ResolveInfo> resolveInfos = makeResolveInfos(info);
- assertEquals(0, ExternalNavigationDelegateImpl.countSpecializedHandlersWithFilter(
- resolveInfos, packageName));
+ assertEquals(0, ExternalNavigationDelegateImpl.getSpecializedHandlersWithFilter(
+ resolveInfos, packageName).size());
}
@SmallTest
@@ -55,8 +55,8 @@ public class ExternalNavigationDelegateImplTest extends ChromeActivityTestCaseBa
info.filter = new IntentFilter();
info.filter.addDataPath("somepath", 2);
List<ResolveInfo> resolveInfos = makeResolveInfos(info);
- assertEquals(1, ExternalNavigationDelegateImpl.countSpecializedHandlersWithFilter(
- resolveInfos, packageName));
+ assertEquals(1, ExternalNavigationDelegateImpl.getSpecializedHandlersWithFilter(
+ resolveInfos, packageName).size());
}
@SmallTest
@@ -66,8 +66,8 @@ public class ExternalNavigationDelegateImplTest extends ChromeActivityTestCaseBa
info.filter = new IntentFilter();
info.filter.addDataAuthority("http://www.google.com", "80");
List<ResolveInfo> resolveInfos = makeResolveInfos(info);
- assertEquals(1, ExternalNavigationDelegateImpl.countSpecializedHandlersWithFilter(
- resolveInfos, packageName));
+ assertEquals(1, ExternalNavigationDelegateImpl.getSpecializedHandlersWithFilter(
+ resolveInfos, packageName).size());
}
@SmallTest
@@ -79,8 +79,8 @@ public class ExternalNavigationDelegateImplTest extends ChromeActivityTestCaseBa
info.activityInfo = new ActivityInfo();
info.activityInfo.packageName = packageName;
List<ResolveInfo> resolveInfos = makeResolveInfos(info);
- assertEquals(1, ExternalNavigationDelegateImpl.countSpecializedHandlersWithFilter(
- resolveInfos, packageName));
+ assertEquals(1, ExternalNavigationDelegateImpl.getSpecializedHandlersWithFilter(
+ resolveInfos, packageName).size());
}
@SmallTest
@@ -92,8 +92,8 @@ public class ExternalNavigationDelegateImplTest extends ChromeActivityTestCaseBa
info.activityInfo = new ActivityInfo();
info.activityInfo.packageName = "com.foo.bar";
List<ResolveInfo> resolveInfos = makeResolveInfos(info);
- assertEquals(0, ExternalNavigationDelegateImpl.countSpecializedHandlersWithFilter(
- resolveInfos, packageName));
+ assertEquals(0, ExternalNavigationDelegateImpl.getSpecializedHandlersWithFilter(
+ resolveInfos, packageName).size());
}
@SmallTest

Powered by Google App Engine
This is Rietveld 408576698