New issue
Advanced search Search tips

Issue 893757 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 10
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Enable WebAPK to handle non HTTP/HTTPS deep links

Project Member Reported by pkotw...@chromium.org, Oct 9

Issue description

Enable WebAPK to handle non HTTP/HTTPS deep links

Context:
http://b/79392573

 
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 9

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6e5ea77d7d2d05b15d5894de016a962ae63d38d9

commit 6e5ea77d7d2d05b15d5894de016a962ae63d38d9
Author: Peter Kotwicz <pkotwicz@chromium.org>
Date: Tue Oct 09 22:17:59 2018

[Android WebAPK] Enable WebAPKs to handle non HTTP/HTTPS deep links

This CL changes the sanity checks in HostBrowserLauncherParams to
enable WebAPKs to handle non HTTP/HTTPs deep links as long as
the URL is rewritten to a HTTP/HTTPS URL.

BUG= 893757 
TEST=MainActivityTest.testIntetnUrlOutOfScopeBecauseOfScheme

Change-Id: I1c88326024129f95cdc7abc12ea7d042f6ad24e9
Reviewed-on: https://chromium-review.googlesource.com/c/1271938
Commit-Queue: Peter Kotwicz <pkotwicz@chromium.org>
Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#598107}
[modify] https://crrev.com/6e5ea77d7d2d05b15d5894de016a962ae63d38d9/chrome/android/webapk/shell_apk/current_version/current_version.gni
[modify] https://crrev.com/6e5ea77d7d2d05b15d5894de016a962ae63d38d9/chrome/android/webapk/shell_apk/junit/src/org/chromium/webapk/shell_apk/MainActivityTest.java
[modify] https://crrev.com/6e5ea77d7d2d05b15d5894de016a962ae63d38d9/chrome/android/webapk/shell_apk/src/org/chromium/webapk/shell_apk/HostBrowserLauncherParams.java

Status: Fixed (was: Started)

Sign in to add a comment