New issue
Advanced search Search tips

Issue 708274 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Remove "untrusted sources" installation for webapks

Project Member Reported by yfried...@chromium.org, Apr 4 2017

Issue description

We should now always use the play installation flow. The other flow was just for testing purposes and is unsafe
 
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 6 2017

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

commit a8d6359dff31bf5db91295dadac0aff2bc02182d
Author: yfriedman <yfriedman@chromium.org>
Date: Thu Apr 06 14:26:39 2017

Remove untrusted sources installation flow.

This model of installation involved a user having to lower the security level
of their device to allow apps to install APKs. It was only used for developer
and preview purposes. It's no longer necessary so clean things up.

BUG= 708274 

Review-Url: https://codereview.chromium.org/2791983003
Cr-Commit-Position: refs/heads/master@{#462462}

[modify] https://crrev.com/a8d6359dff31bf5db91295dadac0aff2bc02182d/chrome/android/java/src/org/chromium/chrome/browser/webapps/ChromeWebApkHost.java
[modify] https://crrev.com/a8d6359dff31bf5db91295dadac0aff2bc02182d/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkInstaller.java
[modify] https://crrev.com/a8d6359dff31bf5db91295dadac0aff2bc02182d/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkUpdateManager.java
[modify] https://crrev.com/a8d6359dff31bf5db91295dadac0aff2bc02182d/chrome/android/junit/src/org/chromium/chrome/browser/webapps/WebApkUpdateManagerTest.java
[modify] https://crrev.com/a8d6359dff31bf5db91295dadac0aff2bc02182d/chrome/browser/android/shortcut_helper.cc
[modify] https://crrev.com/a8d6359dff31bf5db91295dadac0aff2bc02182d/chrome/browser/android/webapk/chrome_webapk_host.cc
[modify] https://crrev.com/a8d6359dff31bf5db91295dadac0aff2bc02182d/chrome/browser/android/webapk/chrome_webapk_host.h
[modify] https://crrev.com/a8d6359dff31bf5db91295dadac0aff2bc02182d/chrome/browser/android/webapk/webapk_install_service.h
[modify] https://crrev.com/a8d6359dff31bf5db91295dadac0aff2bc02182d/chrome/browser/android/webapk/webapk_installer.cc
[modify] https://crrev.com/a8d6359dff31bf5db91295dadac0aff2bc02182d/chrome/browser/android/webapk/webapk_installer.h
[modify] https://crrev.com/a8d6359dff31bf5db91295dadac0aff2bc02182d/chrome/browser/android/webapk/webapk_installer_unittest.cc

Project Member

Comment 2 by bugdroid1@chromium.org, Apr 6 2017

Status: Fixed (was: Started)

Sign in to add a comment