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

chrome/browser/android/banners/app_banner_data_fetcher_android.cc

Index: chrome/browser/android/banners/app_banner_data_fetcher_android.cc
diff --git a/chrome/browser/android/banners/app_banner_data_fetcher_android.cc b/chrome/browser/android/banners/app_banner_data_fetcher_android.cc
index 10164088aa5ddfb2cfc92468681e243ffb5e2a2e..43f378915144b035e6b6ef2dfafe1ff978b0b6f9 100644
--- a/chrome/browser/android/banners/app_banner_data_fetcher_android.cc
+++ b/chrome/browser/android/banners/app_banner_data_fetcher_android.cc
@@ -73,6 +73,12 @@ base::Closure AppBannerDataFetcherAndroid::FetchWebappSplashScreenImageCallback(
       minimum_splash_image_size_in_dp_, webapp_id);
 }
 
+bool AppBannerDataFetcherAndroid::IsWebAppInstalled(
+    content::BrowserContext* browser_context,
+    const GURL& start_url) {

[dominickn, 2016/07/07 00:50:58]

You should explicitly document here that this method will return false if
WebAPKs are not enabled, and return false if a web app is installed, but not as
a WebAPK.

[pkotwicz, 2016/07/07 18:12:44]

I have added some comments. Let me know if my comments can be improved.

+  return ShortcutHelper::IsWebApkInstalled(start_url);
+}
+
 void AppBannerDataFetcherAndroid::ShowBanner(const GURL& icon_url,
                                              const SkBitmap* icon,
                                              const base::string16& title,