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

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

Issue 2855293002: More tests for UrlUtilities#isValidForIntentFallbackNavigation. (Closed)
Patch Set: Initial patch Created 3 years, 8 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/javatests/src/org/chromium/chrome/browser/util/UrlUtilitiesTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/util/UrlUtilitiesTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/util/UrlUtilitiesTest.java
index b15b78319030f0a701b65dd1909d30ee37c4156f..fcedb0569df2f26f87109ad73b0f96aab1b4d682 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/util/UrlUtilitiesTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/util/UrlUtilitiesTest.java
@@ -83,6 +83,10 @@ public class UrlUtilitiesTest {
UrlUtilities.isValidForIntentFallbackNavigation("http://awesome.example.com/"));
Assert.assertFalse(UrlUtilities.isValidForIntentFallbackNavigation("inline:skates.co.uk"));
Assert.assertFalse(UrlUtilities.isValidForIntentFallbackNavigation("javascript:alert(1)"));
+ Assert.assertFalse(
+ UrlUtilities.isValidForIntentFallbackNavigation("file://hostname/path/to/file"));
+ Assert.assertFalse(UrlUtilities.isValidForIntentFallbackNavigation("data:data"));
+ Assert.assertFalse(UrlUtilities.isValidForIntentFallbackNavigation("about:awesome"));
Assert.assertFalse(UrlUtilities.isValidForIntentFallbackNavigation(""));
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698