New issue
Advanced search Search tips

Issue 622135 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Do not show app banner if WebAPK is already installed for site

Project Member Reported by pkotw...@chromium.org, Jun 22 2016

Issue description

Do not show app banner if WebAPK is already installed for site

 
Blocking: -524670
Owner: pkotw...@chromium.org
Status: Started (was: Untriaged)
Issue 608229 has been merged into this issue.
Project Member

Comment 4 by bugdroid1@chromium.org, Jul 8 2016

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

commit cda82fec90f5262fbd162bb2433c5059bca1e7ba
Author: pkotwicz <pkotwicz@chromium.org>
Date: Fri Jul 08 18:56:54 2016

Do not show "Add to Home Screen" info bar if WebAPK is installed

Non-WebAPK web apps rely on a property in site preferences to determine
whether the web app has been installed. If a user clears Chrome's data
via Settings->Apps->Chrome->Storage->"Clear Data" site preferences are
cleared.
This CL adds a way for determining whether a WebAPK is installed which
works even after a user clears Chrome's data.

BUG= 622135 

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

[modify] https://crrev.com/cda82fec90f5262fbd162bb2433c5059bca1e7ba/chrome/android/java/src/org/chromium/chrome/browser/ShortcutHelper.java
[modify] https://crrev.com/cda82fec90f5262fbd162bb2433c5059bca1e7ba/chrome/browser/android/banners/app_banner_data_fetcher_android.cc
[modify] https://crrev.com/cda82fec90f5262fbd162bb2433c5059bca1e7ba/chrome/browser/android/banners/app_banner_data_fetcher_android.h
[modify] https://crrev.com/cda82fec90f5262fbd162bb2433c5059bca1e7ba/chrome/browser/android/shortcut_helper.cc
[modify] https://crrev.com/cda82fec90f5262fbd162bb2433c5059bca1e7ba/chrome/browser/android/shortcut_helper.h
[modify] https://crrev.com/cda82fec90f5262fbd162bb2433c5059bca1e7ba/chrome/browser/banners/app_banner_data_fetcher.h

Status: Fixed (was: Started)

Sign in to add a comment