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

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

Issue 785163002: [Android] Add a check function for intent fallback navigation (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years 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/UrlUtilitiesTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/UrlUtilitiesTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/UrlUtilitiesTest.java
index 9133ab4592ac1a1ad56750dfe08dd39f634787f9..5995cc229f5d361d6527c3676856ade3b7b53be1 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/UrlUtilitiesTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/UrlUtilitiesTest.java
@@ -62,6 +62,16 @@ public class UrlUtilitiesTest extends InstrumentationTestCase {
}
@SmallTest
+ public void testIsValidForIntentFallbackUrl() {
+ assertTrue(UrlUtilities.isValidForIntentFallbackNavigation(
+ "https://user:pass@:awesome.com:9000/bad-scheme:#fake:"));
+ assertTrue(UrlUtilities.isValidForIntentFallbackNavigation("http://awesome.example.com/"));
+ assertFalse(UrlUtilities.isValidForIntentFallbackNavigation("inline:skates.co.uk"));
+ assertFalse(UrlUtilities.isValidForIntentFallbackNavigation("javascript:alert(1)"));
+ assertFalse(UrlUtilities.isValidForIntentFallbackNavigation(""));
+ }
+
+ @SmallTest
public void testFixUrl() throws URISyntaxException {
try {
URI uri;

Powered by Google App Engine
This is Rietveld 408576698