Remove untrusted sources installation flow.

chrome/browser/android/shortcut_helper.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/android/shortcut_helper.h"
 
 #include <jni.h>
 #include <utility>
 
 #include "base/android/jni_android.h"
@@ skipping 143 matching lines @@
 }
 
 // static
 void ShortcutHelper::InstallWebApkWithSkBitmap(
     content::WebContents* web_contents,
     const ShortcutInfo& info,
     const SkBitmap& icon_bitmap,
     const WebApkInstallService::FinishCallback& callback) {
   WebApkInstallService::Get(web_contents->GetBrowserContext())
       ->InstallAsync(info, icon_bitmap, callback);
-  // Don't record metric for users who install WebAPKs via "unsigned sources"
-  // flow.
   if (ChromeWebApkHost::GetGooglePlayInstallState() ==

[Yaron, 2017/04/04 16:42:02]

the comment was unnecessary, but I'm truthfully unsure about the conditional. We
can probably assume it's SUPPORTED if we get here?

[pkotwicz, 2017/04/04 19:06:07]

We can. The caller only calls ShortcutHelper::InstallWebApkWithSkBitmap() if
ChromeWebApkHost::CanInstallWebApk()

See AppBannerInfoBarDelegateAndroid::Create()

[Yaron, 2017/04/04 19:49:08]

Done.

       GooglePlayInstallState::SUPPORTED) {
     webapk::TrackGooglePlayInstallState(GooglePlayInstallState::SUPPORTED);
   }
 }
 
 void ShortcutHelper::ShowWebApkInstallInProgressToast() {
   Java_ShortcutHelper_showWebApkInstallInProgressToast(
       base::android::AttachCurrentThread());
 }
 
@@ skipping 236 matching lines @@
 
   ShortcutHelper::WebApkInfoCallback* webapk_list_callback =
       reinterpret_cast<ShortcutHelper::WebApkInfoCallback*>(jcallback_pointer);
   webapk_list_callback->Run(webapk_list);
   delete webapk_list_callback;
 }
 
 bool ShortcutHelper::RegisterShortcutHelper(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }