WebAPK: Move MainActivity#onCreate() to runtime library

chrome/android/webapk/libs/runtime_library/src/org/chromium/webapk/lib/runtime_library/HostBrowserLauncher.java

 // Copyright 2015 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.
 
-package org.chromium.webapk.shell_apk;
+package org.chromium.webapk.lib.runtime_library;
 
-import android.app.Activity;
+import android.content.ActivityNotFoundException;
 import android.content.ComponentName;
+import android.content.Context;
 import android.content.Intent;
 import android.content.pm.ApplicationInfo;
 import android.content.pm.PackageManager;
 import android.content.pm.PackageManager.NameNotFoundException;
 import android.graphics.Bitmap;
 import android.graphics.BitmapFactory;
 import android.os.Bundle;
 import android.text.TextUtils;
 import android.util.Base64;
 import android.util.Log;
 
 import org.chromium.webapk.lib.common.WebApkConstants;
 
 import java.io.ByteArrayOutputStream;
 
 /**
- * WebAPK's main Activity.
+ * Launches Chrome in WebAPK mode.
  */
-public class MainActivity extends Activity {
+public class HostBrowserLauncher {
     // These EXTRA_* values must stay in sync with
     // {@link org.chromium.chrome.browser.ShortcutHelper}.
     private static final String EXTRA_ID = "org.chromium.chrome.browser.webapp_id";
     private static final String EXTRA_ICON = "org.chromium.chrome.browser.webapp_icon";
     private static final String EXTRA_SHORT_NAME = "org.chromium.chrome.browser.webapp_short_name";
     private static final String EXTRA_NAME = "org.chromium.chrome.browser.webapp_name";
     private static final String EXTRA_URL = "org.chromium.chrome.browser.webapp_url";
     private static final String EXTRA_SOURCE = "org.chromium.chrome.browser.webapp_source";
     private static final String EXTRA_THEME_COLOR = "org.chromium.chrome.browser.theme_color";
     private static final String EXTRA_BACKGROUND_COLOR =
             "org.chromium.chrome.browser.background_color";
     private static final String EXTRA_IS_ICON_GENERATED =
             "org.chromium.chrome.browser.is_icon_generated";
     private static final String EXTRA_WEBAPK_PACKAGE_NAME =
             "org.chromium.chrome.browser.webapk_package_name";
 
     private static final String META_DATA_RUNTIME_HOST = "runtimeHost";
     private static final String META_DATA_START_URL = "startUrl";
     private static final String META_DATA_NAME = "name";
     private static final String META_DATA_DISPLAY_MODE = "displayMode";
     private static final String META_DATA_ORIENTATION = "orientation";
     private static final String META_DATA_THEME_COLOR = "themeColor";
     private static final String META_DATA_BACKGROUND_COLOR = "backgroundColor";
     private static final String META_DATA_ICON_URL = "iconUrl";
 
-    private static final String TAG = "cr_MainActivity";
+    /**
+     * Key for passing app icon id in Bundle to {@link #launch()}.
+     */
+    private static final String KEY_APP_ICON_ID = "app_icon_id";

[Xi Han, 2016/06/27 18:27:08]

How about to put it in the WebApkConstants? So we will only keep one copy of the
 KEY_APP_ICON_ID, and will run into sync issue.

[pkotwicz, 2016/06/27 20:44:28]

I want to keep libs/common as small as possible. libs/common is hard to think
about. 

This scenario is possible:
ShellAPK uses an old version of libs/common (Request for new ShellAPK has been
sent to server)
runtime library uses a new version of libs/common

[Xi Han, 2016/06/28 13:53:39]

It makes sense.

 
-    @Override
-    protected void onCreate(Bundle savedInstanceState) {
-        super.onCreate(savedInstanceState);
+    private static final String TAG = "cr_HostBrowserLauncher";
 
-        String packageName = getPackageName();
+    /**
+     * Launches Chrome in WebAPK mode.
+     * @param context Application context.
+     * @param intent Intent used to launch WebAPK.
+     * @param bundle Contains extra parameters.
+     */
+    public void launch(Context context, Intent intent, Bundle bundle) {
+        int appIconId = bundle.getInt(KEY_APP_ICON_ID);
+
+        String packageName = context.getPackageName();
+        ApplicationInfo appInfo;
         try {
-            ApplicationInfo appInfo = getPackageManager().getApplicationInfo(
+            appInfo = context.getPackageManager().getApplicationInfo(
                     packageName, PackageManager.GET_META_DATA);
-            Bundle bundle = appInfo.metaData;
-            String url = bundle.getString(META_DATA_START_URL);
+        } catch (NameNotFoundException e) {
+            return;
+        }
 
-            Intent intent = getIntent();
-            String overrideUrl = intent.getDataString();
-            // TODO(pkotwicz): Use same logic as {@code IntentHandler#shouldIgnoreIntent()}
-            if (overrideUrl != null && overrideUrl.startsWith("https:")) {
-                url = overrideUrl;
-            }
-            int source = intent.getIntExtra(EXTRA_SOURCE, 0);
+        Bundle metaBundle = appInfo.metaData;
+        String url = metaBundle.getString(META_DATA_START_URL);
 
-            String webappId = WebApkConstants.WEBAPK_ID_PREFIX + packageName;
-            String runtimeHost = bundle.getString(META_DATA_RUNTIME_HOST);
-            String shortName = (String) getPackageManager().getApplicationLabel(appInfo);
-            // TODO(hanxi): find a neat solution to avoid encode/decode each time launch the
-            // activity.
-            Bitmap icon = BitmapFactory.decodeResource(getResources(), R.drawable.app_icon);
-            String encodedIcon = encodeBitmapAsString(icon);
-            String name = bundle.getString(META_DATA_NAME);
-            String displayMode = bundle.getString(META_DATA_DISPLAY_MODE);
-            String orientation = bundle.getString(META_DATA_ORIENTATION);
-            long themeColor = getLongFromBundle(bundle, META_DATA_THEME_COLOR);
-            long backgroundColor = getLongFromBundle(bundle, META_DATA_BACKGROUND_COLOR);
-            boolean isIconGenerated = TextUtils.isEmpty(bundle.getString(META_DATA_ICON_URL));
-            Log.w(TAG, "Url of the WebAPK: " + url);
-            Log.w(TAG, "WebappId of the WebAPK: " + webappId);
-            Log.w(TAG, "Name of the WebAPK:" + name);
-            Log.w(TAG, "Package name of the WebAPK:" + packageName);
+        String overrideUrl = intent.getDataString();
+        // TODO(pkotwicz): Use same logic as {@code IntentHandler#shouldIgnoreIntent()}
+        if (overrideUrl != null && overrideUrl.startsWith("https:")) {
+            url = overrideUrl;
+        }
+        int source = intent.getIntExtra(EXTRA_SOURCE, 0);
 
-            Intent newIntent = new Intent();
-            newIntent.setComponent(new ComponentName(runtimeHost,
-                    "org.chromium.chrome.browser.webapps.WebappLauncherActivity"));
-            newIntent.putExtra(EXTRA_ID, webappId)
-                     .putExtra(EXTRA_SHORT_NAME, shortName)
-                     .putExtra(EXTRA_NAME, name)
-                     .putExtra(EXTRA_URL, url)
-                     .putExtra(EXTRA_ICON, encodedIcon)
-                     .putExtra(EXTRA_SOURCE, source)
-                     .putExtra(EXTRA_THEME_COLOR, themeColor)
-                     .putExtra(EXTRA_BACKGROUND_COLOR, backgroundColor)
-                     .putExtra(EXTRA_IS_ICON_GENERATED, isIconGenerated)
-                     .putExtra(EXTRA_WEBAPK_PACKAGE_NAME, packageName)
-                     .putExtra(WebApkConstants.EXTRA_DISPLAY_MODE, displayMode)
-                     .putExtra(WebApkConstants.EXTRA_ORIENTATION, orientation);
-            startActivity(newIntent);
-            finish();
-        } catch (NameNotFoundException e) {
+        String webappId = WebApkConstants.WEBAPK_ID_PREFIX + packageName;
+        String runtimeHost = metaBundle.getString(META_DATA_RUNTIME_HOST);
+        String shortName = (String) context.getPackageManager().getApplicationLabel(appInfo);
+        // TODO(hanxi): find a neat solution to avoid encode/decode each time launch the
+        // activity.
+        Bitmap icon = BitmapFactory.decodeResource(context.getResources(), appIconId);
+        String encodedIcon = encodeBitmapAsString(icon);
+        String name = metaBundle.getString(META_DATA_NAME);
+        String displayMode = metaBundle.getString(META_DATA_DISPLAY_MODE);
+        String orientation = metaBundle.getString(META_DATA_ORIENTATION);
+        long themeColor = getLongFromBundle(metaBundle, META_DATA_THEME_COLOR);
+        long backgroundColor = getLongFromBundle(metaBundle, META_DATA_BACKGROUND_COLOR);
+        boolean isIconGenerated = TextUtils.isEmpty(metaBundle.getString(META_DATA_ICON_URL));
+        Log.w(TAG, "Url of the WebAPK: " + url);
+        Log.w(TAG, "WebappId of the WebAPK: " + webappId);
+        Log.w(TAG, "Name of the WebAPK:" + name);
+        Log.w(TAG, "Package name of the WebAPK:" + packageName);
+
+        Intent newIntent = new Intent();
+        newIntent.setComponent(new ComponentName(
+                runtimeHost, "org.chromium.chrome.browser.webapps.WebappLauncherActivity"));
+        newIntent.putExtra(EXTRA_ID, webappId)
+                .putExtra(EXTRA_SHORT_NAME, shortName)
+                .putExtra(EXTRA_NAME, name)
+                .putExtra(EXTRA_URL, url)
+                .putExtra(EXTRA_ICON, encodedIcon)
+                .putExtra(EXTRA_SOURCE, source)
+                .putExtra(EXTRA_THEME_COLOR, themeColor)
+                .putExtra(EXTRA_BACKGROUND_COLOR, backgroundColor)
+                .putExtra(EXTRA_IS_ICON_GENERATED, isIconGenerated)
+                .putExtra(EXTRA_WEBAPK_PACKAGE_NAME, packageName)
+                .putExtra(WebApkConstants.EXTRA_DISPLAY_MODE, displayMode)
+                .putExtra(WebApkConstants.EXTRA_ORIENTATION, orientation);
+
+        try {
+            context.startActivity(newIntent);
+        } catch (ActivityNotFoundException e) {
             e.printStackTrace();
         }
     }
 
     /**
      * Compresses a bitmap into a PNG and converts into a Base64 encoded string.
      * The encoded string can be decoded using {@link decodeBitmapFromString(String)}.
      * @param bitmap The Bitmap to compress and encode.
      * @return the String encoding the Bitmap.
      */
@@ skipping 13 matching lines @@
         if (value == null || !value.endsWith("L")) {
             return 0;
         }
         try {
             return Long.parseLong(value.substring(0, value.length() - 1));
         } catch (NumberFormatException e) {
         }
         return 0;
     }
 }