Initial CL for talking to the WebAPK server to generate WebAPK

chrome/browser/android/shortcut_helper.cc

Index: chrome/browser/android/shortcut_helper.cc
diff --git a/chrome/browser/android/shortcut_helper.cc b/chrome/browser/android/shortcut_helper.cc
index 7efff3a02ecbed13dbdc82ed3021318b38e59a08..30bf93dddc40bade32fa01fa7d3bef37377e47ef 100644
--- a/chrome/browser/android/shortcut_helper.cc
+++ b/chrome/browser/android/shortcut_helper.cc
@@ -14,6 +14,7 @@
 #include "base/command_line.h"
 #include "base/strings/string16.h"
 #include "base/strings/utf_string_conversions.h"
+#include "chrome/browser/android/webapk/webapk_installer.h"
 #include "chrome/browser/manifest/manifest_icon_downloader.h"
 #include "chrome/common/chrome_switches.h"
 #include "content/public/browser/browser_thread.h"
@@ -62,6 +63,7 @@ void GetHomescreenIconAndSplashImageSizes() {
 
 // static
 void ShortcutHelper::AddToLauncherInBackgroundWithSkBitmap(
+    content::BrowserContext* browser_context,
     const ShortcutInfo& info,
     const std::string& webapp_id,
     const SkBitmap& icon_bitmap,
@@ -72,7 +74,7 @@ void ShortcutHelper::AddToLauncherInBackgroundWithSkBitmap(
       info.display == blink::WebDisplayModeFullscreen) {
     if (base::CommandLine::ForCurrentProcess()->HasSwitch(
             switches::kEnableWebApk)) {
-      InstallWebApkInBackgroundWithSkBitmap(info, webapp_id, icon_bitmap);
+      InstallWebApkInBackgroundWithSkBitmap(browser_context, info, icon_bitmap);
       return;
     }
     AddWebappInBackgroundWithSkBitmap(info, webapp_id, icon_bitmap,
@@ -84,43 +86,14 @@ void ShortcutHelper::AddToLauncherInBackgroundWithSkBitmap(
 
 // static
 void ShortcutHelper::InstallWebApkInBackgroundWithSkBitmap(
+    content::BrowserContext* browser_context,
     const ShortcutInfo& info,
-    const std::string& webapp_id,
     const SkBitmap& icon_bitmap) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::IO);
-
-  // TODO(pkotwicz): Send request to WebAPK server to generate WebAPK.
-
-  JNIEnv* env = base::android::AttachCurrentThread();
-  ScopedJavaLocalRef<jstring> java_url =
-      base::android::ConvertUTF8ToJavaString(env, info.url.spec());
-  ScopedJavaLocalRef<jstring> java_scope_url =
-      base::android::ConvertUTF8ToJavaString(env, info.scope.spec());
-  ScopedJavaLocalRef<jstring> java_name =
-      base::android::ConvertUTF16ToJavaString(env, info.name);
-  ScopedJavaLocalRef<jstring> java_short_name =
-      base::android::ConvertUTF16ToJavaString(env, info.short_name);
-  ScopedJavaLocalRef<jstring> java_icon_url =
-      base::android::ConvertUTF8ToJavaString(env, info.icon_url.spec());
-  ScopedJavaLocalRef<jobject> java_bitmap;
-  if (icon_bitmap.getSize())
-    java_bitmap = gfx::ConvertToJavaBitmap(&icon_bitmap);
-  ScopedJavaLocalRef<jstring> java_manifest_url =
-      base::android::ConvertUTF8ToJavaString(env, info.manifest_url.spec());
-
-  Java_ShortcutHelper_installWebApk(
-      env,
-      java_url.obj(),
-      java_scope_url.obj(),
-      java_name.obj(),
-      java_short_name.obj(),
-      java_icon_url.obj(),
-      java_bitmap.obj(),
-      info.display,
-      info.orientation,
-      info.theme_color,
-      info.background_color,
-      java_manifest_url.obj());
+  // WebApkInstaller destroys itself when it is done.
+  WebApkInstaller* installer = new WebApkInstaller(info, icon_bitmap);
+  installer->InstallAsync(browser_context,
+                          base::Bind(&ShortcutHelper::OnBuiltWebApk));
 }
 
 // static
@@ -192,6 +165,16 @@ void ShortcutHelper::AddShortcutInBackgroundWithSkBitmap(
                                   java_bitmap.obj(), info.source);
 }
 
+void ShortcutHelper::OnBuiltWebApk(bool success) {
+  if (success) {
+    LOG(ERROR) << "Sent request to install WebAPK. Seems to have worked.";

[gone, 2016/07/29 21:20:28]

ERROR doesn't seem like the right LOG level to use here.

[pkotwicz, 2016/08/02 04:09:37]

Changed to LOG(INFO). The reason for the log is so that this function is
non-empty. Once this function is non-empty I will move the logging to
webapk_installer.cc

+  } else {
+    LOG(ERROR) << "WebAPK install failed.";
+  }
+  // TODO(pkotwicz): Figure out what to do when installing WebAPK fails.
+  // (crbug.com/626950)
+}
+
 int ShortcutHelper::GetIdealHomescreenIconSizeInDp() {
   if (kIdealHomescreenIconSize == -1)
     GetHomescreenIconAndSplashImageSizes();