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

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

Issue 2951883003: android: Stricter android-app:// referrer handling. (Closed)
Patch Set: . Created 3 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
« no previous file with comments | « chrome/android/javatests/src/org/chromium/chrome/browser/IntentHandlerTest.java ('k') | 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/TabsOpenedFromExternalAppTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/TabsOpenedFromExternalAppTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/TabsOpenedFromExternalAppTest.java
index c52de3d6711a3f75e6ba6932c591dadece914bdd..b80b6ac0e71331a5c1ccdc916761d5bde0961e58 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/TabsOpenedFromExternalAppTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/TabsOpenedFromExternalAppTest.java
@@ -260,7 +260,26 @@ public class TabsOpenedFromExternalAppTest {
}
/**
- * Tests that URLs opened from external apps can set an android-app scheme referrer.
+ * Tests that URLs opened from external apps cannot set an invalid android-app referrer.
+ * @throws InterruptedException
+ */
+ @Test
+ @LargeTest
+ @Feature({"Navigation"})
+ public void testInvalidAndroidAppReferrer() throws InterruptedException {
+ String invalidReferrer = "android-app:///note.the.extra.leading/";
+ String url = mTestServer.getURL("/chrome/test/data/android/about.html");
+ mActivityTestRule.startMainActivityFromLauncher();
+ Bundle extras = new Bundle();
+ extras.putParcelable(Intent.EXTRA_REFERRER, Uri.parse(invalidReferrer));
+ launchUrlFromExternalApp(url, url, EXTERNAL_APP_1_ID, true, extras);
+ CriteriaHelper.pollInstrumentationThread(
+ new ReferrerCriteria(mActivityTestRule.getActivity().getActivityTab(), ""), 2000,
+ 200);
+ }
+
+ /**
+ * Tests that URLs opened from external apps cannot set an arbitrary referrer scheme.
* @throws InterruptedException
*/
@Test
« no previous file with comments | « chrome/android/javatests/src/org/chromium/chrome/browser/IntentHandlerTest.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698