Upstream: Launch WebApkActivity from WebAPK.

chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappInfo.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.chrome.browser.webapps;
 
 import android.content.Intent;
 import android.graphics.Bitmap;
 import android.net.Uri;
 import android.util.Log;
 
 import org.chromium.chrome.browser.ShortcutHelper;
 import org.chromium.chrome.browser.ShortcutSource;
 import org.chromium.chrome.browser.util.IntentUtils;
 import org.chromium.content_public.common.ScreenOrientationValues;
 
 /**
  * Stores info about a web app.
  */
 public class WebappInfo {
     private boolean mIsInitialized;
     private String mId;
     private String mEncodedIcon;
     private Bitmap mDecodedIcon;
     private Uri mUri;
+    private Uri mScope;

[pkotwicz, 2016/05/18 23:00:17]

I don't think that this member is needed at all. I will write a CL which removes
it completely.

[Xi Han, 2016/05/19 18:31:49]

Since WebAPK will have two scopes, the "launch scope" can be determine by intent
filters, we still need a "remain in" scope, so I keep it here. What I am not
sure is whether we need an array for the "remain in" scope.

I can remove it for now, and add it back when we need.

     private String mName;
     private String mShortName;
     private int mOrientation;
     private int mSource;
     private long mThemeColor;
     private long mBackgroundColor;
     private boolean mIsIconGenerated;
+    private String mPackageName;
 
     public static WebappInfo createEmpty() {
         return new WebappInfo();
     }
 
+    public static String scopeFromIntent(Intent intent) {
+        String scope = IntentUtils.safeGetStringExtra(intent, ShortcutHelper.EXTRA_SCOPE);
+        return scope == null ? "" : scope;
+    }
+
     private static String titleFromIntent(Intent intent) {
         // The reference to title has been kept for reasons of backward compatibility. For intents
         // and shortcuts which were created before we utilized the concept of name and shortName,
         // we set the name and shortName to be the title.
         String title = IntentUtils.safeGetStringExtra(intent, ShortcutHelper.EXTRA_TITLE);
         return title == null ? "" : title;
     }
 
     public static String nameFromIntent(Intent intent) {
         String name = IntentUtils.safeGetStringExtra(intent, ShortcutHelper.EXTRA_NAME);
         return name == null ? titleFromIntent(intent) : name;
     }
 
     public static String shortNameFromIntent(Intent intent) {
         String shortName = IntentUtils.safeGetStringExtra(intent, ShortcutHelper.EXTRA_SHORT_NAME);
         return shortName == null ? titleFromIntent(intent) : shortName;
     }
 
+    public static String packageNameFromIntent(Intent intent) {
+        String packageName = IntentUtils.safeGetStringExtra(
+                intent, ShortcutHelper.EXTRA_WEBAPK_PACKAGE_NAME);
+        return packageName == null ? "" : packageName;
+    }
+
     /**
      * Construct a WebappInfo.
      * @param intent Intent containing info about the app.
      */
     public static WebappInfo create(Intent intent) {
         String id = IntentUtils.safeGetStringExtra(intent, ShortcutHelper.EXTRA_ID);
         String icon = IntentUtils.safeGetStringExtra(intent, ShortcutHelper.EXTRA_ICON);
         String url = IntentUtils.safeGetStringExtra(intent, ShortcutHelper.EXTRA_URL);
+        String scope = scopeFromIntent(intent);
         int orientation = IntentUtils.safeGetIntExtra(intent,
                 ShortcutHelper.EXTRA_ORIENTATION, ScreenOrientationValues.DEFAULT);
         int source = IntentUtils.safeGetIntExtra(intent,
                 ShortcutHelper.EXTRA_SOURCE, ShortcutSource.UNKNOWN);
         long themeColor = IntentUtils.safeGetLongExtra(intent,
                 ShortcutHelper.EXTRA_THEME_COLOR,
                 ShortcutHelper.MANIFEST_COLOR_INVALID_OR_MISSING);
         long backgroundColor = IntentUtils.safeGetLongExtra(intent,
                 ShortcutHelper.EXTRA_BACKGROUND_COLOR,
                 ShortcutHelper.MANIFEST_COLOR_INVALID_OR_MISSING);
         boolean isIconGenerated = IntentUtils.safeGetBooleanExtra(intent,
                 ShortcutHelper.EXTRA_IS_ICON_GENERATED, false);
 
         String name = nameFromIntent(intent);
         String shortName = shortNameFromIntent(intent);
+        String packageName = packageNameFromIntent(intent);
 
-        return create(id, url, icon, name, shortName, orientation, source,
-                themeColor, backgroundColor, isIconGenerated);
+        return create(id, url, scope, icon, name, shortName, orientation, source,
+                themeColor, backgroundColor, isIconGenerated, packageName);
     }
 
     /**
      * Construct a WebappInfo.
      * @param id ID for the webapp.
      * @param url URL for the webapp.
+     * @param scope Scope of URLs that the WebAPK can navigate to.
      * @param icon Icon to show for the webapp.
      * @param name Name of the webapp.
      * @param shortName The short name of the webapp.
      * @param orientation Orientation of the webapp.
      * @param source Source where the webapp was added from.
      * @param themeColor The theme color of the webapp.
      * @param isIconGenerated Whether the |icon| was generated by Chromium.
+     * @param packageName The package name of the WebAPK that intents to start the WebApp
+     * activity.
      */
-    public static WebappInfo create(String id, String url, String icon, String name,
-            String shortName, int orientation, int source, long themeColor,
-            long backgroundColor, boolean isIconGenerated) {
+    public static WebappInfo create(String id, String url, String scope, String icon, String name,
+            String shortName, int orientation, int source, long themeColor, long backgroundColor,
+            boolean isIconGenerated, String packageName) {
         if (id == null || url == null) {
             Log.e("WebappInfo", "Data passed in was incomplete: " + id + ", " + url);
             return null;
         }
 
         Uri uri = Uri.parse(url);
-        return new WebappInfo(id, uri, icon, name, shortName, orientation, source,
-                themeColor, backgroundColor, isIconGenerated);
+        Uri scope_uri = scope == null ? Uri.parse("") : Uri.parse(scope);
+        return new WebappInfo(id, uri, scope_uri, icon, name, shortName, orientation, source,
+                themeColor, backgroundColor, isIconGenerated, packageName);
     }
 
-    private WebappInfo(String id, Uri uri, String encodedIcon, String name,
-            String shortName, int orientation, int source, long themeColor,
-            long backgroundColor, boolean isIconGenerated) {
+    private WebappInfo(String id, Uri uri, Uri scope, String encodedIcon, String name,
+            String shortName, int orientation, int source, long themeColor, long backgroundColor,
+            boolean isIconGenerated, String packageName) {
         mEncodedIcon = encodedIcon;
         mId = id;
         mName = name;
         mShortName = shortName;
         mUri = uri;
+        mScope = scope;
         mOrientation = orientation;
         mSource = source;
         mThemeColor = themeColor;
         mBackgroundColor = backgroundColor;
         mIsIconGenerated = isIconGenerated;
         mIsInitialized = mUri != null;
+        mPackageName = packageName;
     }
 
     private WebappInfo() {
     }
 
-    /**
-     * Copies all the fields from the given WebappInfo into this instance.
-     * @param newInfo Information about the new webapp.
-     */
-    void copy(WebappInfo newInfo) {
-        mIsInitialized = newInfo.mIsInitialized;
-        mEncodedIcon = newInfo.mEncodedIcon;
-        mDecodedIcon = newInfo.mDecodedIcon;
-        mId = newInfo.mId;
-        mUri = newInfo.mUri;
-        mName = newInfo.mName;
-        mShortName = newInfo.mShortName;
-        mOrientation = newInfo.mOrientation;
-        mSource = newInfo.mSource;
-        mThemeColor = newInfo.mThemeColor;
-        mBackgroundColor = newInfo.mBackgroundColor;
-        mIsIconGenerated = newInfo.mIsIconGenerated;
-    }
-
     public boolean isInitialized() {
         return mIsInitialized;
     }
 
     public String id() {
         return mId;
     }
 
     public Uri uri() {
         return mUri;
     }
 
+    public Uri scope() {
+        return mScope;
+    }
+
     public String name() {
         return mName;
     }
 
     public String shortName() {
         return mShortName;
     }
 

[pkotwicz, 2016/05/18 23:00:17]

Can we rename this function to getWebApkPackage() (and rename the member name to
mWebApkPackage) ? This matches the name of the extra.

[Xi Han, 2016/05/19 18:31:50]

Change to WebApkPackageName.

+    public String packageName() {
+        return mPackageName;
+    }
+
     public int orientation() {
         return mOrientation;
     }
 
     public int source() {
         return mSource;
     }
 
     /**
      * Theme color is actually a 32 bit unsigned integer which encodes a color
@@ skipping 59 matching lines @@
         return mIsIconGenerated;
     }
 
     /**
      * Sets extras on an Intent that will launch a WebappActivity.
      * @param intent Intent that will be used to launch a WebappActivity.
      */
     public void setWebappIntentExtras(Intent intent) {
         intent.putExtra(ShortcutHelper.EXTRA_ID, id());
         intent.putExtra(ShortcutHelper.EXTRA_URL, uri().toString());
-        intent.putExtra(ShortcutHelper.EXTRA_SCOPE,
-                ShortcutHelper.getScopeFromUrl(uri().toString()));
+        intent.putExtra(ShortcutHelper.EXTRA_SCOPE, scope().toString());
         intent.putExtra(ShortcutHelper.EXTRA_ICON, encodedIcon());
         intent.putExtra(ShortcutHelper.EXTRA_VERSION, ShortcutHelper.WEBAPP_SHORTCUT_VERSION);
         intent.putExtra(ShortcutHelper.EXTRA_NAME, name());
         intent.putExtra(ShortcutHelper.EXTRA_SHORT_NAME, shortName());
         intent.putExtra(ShortcutHelper.EXTRA_ORIENTATION, orientation());
         intent.putExtra(ShortcutHelper.EXTRA_SOURCE, source());
         intent.putExtra(ShortcutHelper.EXTRA_THEME_COLOR, themeColor());
         intent.putExtra(ShortcutHelper.EXTRA_BACKGROUND_COLOR, backgroundColor());
         intent.putExtra(ShortcutHelper.EXTRA_IS_ICON_GENERATED, isIconGenerated());
+        intent.putExtra(ShortcutHelper.EXTRA_WEBAPK_PACKAGE_NAME, packageName());
     }
 
     /**
      * Returns true if the WebappInfo was created for an Intent fired from a launcher shortcut (as
      * opposed to an intent from a push notification or other internal source).
      */
     public boolean isLaunchedFromHomescreen() {
         return source() != ShortcutSource.NOTIFICATION;
     }
 }