Add a chrome://webapks page.

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 "base/android/jni_android.h"
 #include "base/android/jni_array.h"
@@ skipping 264 matching lines @@
 
 GURL ShortcutHelper::GetScopeFromURL(const GURL& url) {
   JNIEnv* env = base::android::AttachCurrentThread();
   ScopedJavaLocalRef<jstring> java_url =
       base::android::ConvertUTF8ToJavaString(env, url.spec());
   ScopedJavaLocalRef<jstring> java_scope_url =
       Java_ShortcutHelper_getScopeFromUrl(env, java_url);
   return GURL(base::android::ConvertJavaStringToUTF16(env, java_scope_url));
 }
 
+void ShortcutHelper::ListWebApks(JNIEnv* env,

[pkotwicz, 2017/01/18 02:08:21]

Nit: Remove the JNIEnv* parameter and call AttachCurrentThread() inside this
function

[gonzalon, 2017/01/18 15:29:32]

Done.

+    const base::Callback<void(std::vector<WebApkInfo*>)>& callback) {
+  uintptr_t callback_pointer = reinterpret_cast<uintptr_t>(
+      new base::Callback<void(std::vector<WebApkInfo*>)>(callback));
+  Java_ShortcutHelper_listWebApks(env, callback_pointer);
+}
+
 // Callback used by Java when the shortcut has been created.
 // |splash_image_callback| is a pointer to a base::Closure allocated in
 // AddShortcutWithSkBitmap, so reinterpret_cast it back and run it.
 //
 // This callback should only ever be called when the shortcut was for a
 // webapp-capable site; otherwise, |splash_image_callback| will have never been
 // allocated and doesn't need to be run or deleted.
 void OnWebappDataStored(JNIEnv* env,
                         const JavaParamRef<jclass>& clazz,
                         jlong jsplash_image_callback) {
   DCHECK(jsplash_image_callback);
   base::Closure* splash_image_callback =
       reinterpret_cast<base::Closure*>(jsplash_image_callback);
   splash_image_callback->Run();
   delete splash_image_callback;
 }
 
+void OnWebApksFound(JNIEnv* env,
+                    const JavaParamRef<jclass>& clazz,
+                    const jlong jcallback_pointer,
+                    const JavaParamRef<jobjectArray>& jshort_names,
+                    const JavaParamRef<jobjectArray>& jpackage_names,
+                    const JavaParamRef<jintArray>& jshell_apk_versions,
+                    const JavaParamRef<jintArray>& jversion_codes) {
+  DCHECK(jcallback_pointer);
+  std::vector<std::string> short_names;
+  base::android::AppendJavaStringArrayToStringVector(env, jshort_names,
+                                                     &short_names);
+  std::vector<std::string> package_names;
+  base::android::AppendJavaStringArrayToStringVector(env, jpackage_names,
+                                                     &package_names);
+  std::vector<int> shell_apk_versions;
+  base::android::JavaIntArrayToIntVector(env, jshell_apk_versions,
+                                         &shell_apk_versions);
+  std::vector<int> version_codes;
+  base::android::JavaIntArrayToIntVector(env, jversion_codes, &version_codes);
+
+  std::vector<WebApkInfo*> webapks_list;

[pkotwicz, 2017/01/18 02:08:21]

Can this be std::vector<WebApkInfo> instead?

[gonzalon, 2017/01/18 15:29:32]

Done.

+  for (size_t i = 0; i < short_names.size(); ++i) {
+    WebApkInfo* webapk_info =
+        new WebApkInfo(short_names[i], package_names[i], shell_apk_versions[i],
+                       version_codes[i]);
+    webapks_list.push_back(webapk_info);
+  }
+
+  base::Callback<void(std::vector<WebApkInfo*>)>* webapks_list_callback =
+      reinterpret_cast<base::Callback<void(std::vector<WebApkInfo*>)>*>(
+          jcallback_pointer);
+  webapks_list_callback->Run(webapks_list);
+  delete webapks_list_callback;
+}
+
 bool ShortcutHelper::RegisterShortcutHelper(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }