Store standalone web app data in WebappDataStorage as well as the homescreen intent.

chrome/android/java/src/org/chromium/chrome/browser/ShortcutHelper.java

 // 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.
 
 package org.chromium.chrome.browser;
 
 import android.app.ActivityManager;
 import android.content.Context;
 import android.content.Intent;
 import android.content.pm.PackageManager;
@@ skipping 40 matching lines @@
  */
 public class ShortcutHelper {
     public static final String EXTRA_ICON = "org.chromium.chrome.browser.webapp_icon";
     public static final String EXTRA_ID = "org.chromium.chrome.browser.webapp_id";
     public static final String EXTRA_MAC = "org.chromium.chrome.browser.webapp_mac";
     // EXTRA_TITLE is present for backward compatibility reasons
     public static final String EXTRA_TITLE = "org.chromium.chrome.browser.webapp_title";
     public static final String EXTRA_NAME = "org.chromium.chrome.browser.webapp_name";
     public static final String EXTRA_SHORT_NAME = "org.chromium.chrome.browser.webapp_short_name";
     public static final String EXTRA_URL = "org.chromium.chrome.browser.webapp_url";
+    public static final String EXTRA_SCOPE = "org.chromium.chrome.browser.webapp_scope";
     public static final String EXTRA_ORIENTATION = ScreenOrientationConstants.EXTRA_ORIENTATION;
     public static final String EXTRA_SOURCE = "org.chromium.chrome.browser.webapp_source";
     public static final String EXTRA_THEME_COLOR = "org.chromium.chrome.browser.theme_color";
     public static final String EXTRA_BACKGROUND_COLOR =
             "org.chromium.chrome.browser.background_color";
     public static final String EXTRA_IS_ICON_GENERATED =
             "org.chromium.chrome.browser.is_icon_generated";
     public static final String REUSE_URL_MATCHING_TAB_ELSE_NEW_TAB =
             "REUSE_URL_MATCHING_TAB_ELSE_NEW_TAB";
 
@@ skipping 46 matching lines @@
     /**
      * Called when we have to fire an Intent to add a shortcut to the home screen.
      * If the webpage indicated that it was capable of functioning as a webapp, it is added as a
      * shortcut to a webapp Activity rather than as a general bookmark. User is sent to the
      * home screen as soon as the shortcut is created.
      */
     @SuppressWarnings("unused")
     @CalledByNative
     private static void addShortcut(Context context, String id, String url, final String userTitle,
             String name, String shortName, Bitmap icon, boolean isWebappCapable, int orientation,
-            int source, long themeColor, long backgroundColor, boolean isIconGenerated) {
+            int source, long themeColor, long backgroundColor, boolean isIconGenerated,
+            final long callbackPointer) {
         Intent shortcutIntent;
         if (isWebappCapable) {
-            // Encode the icon as a base64 string (Launcher drops Bitmaps in the Intent).
-            String encodedIcon = encodeBitmapAsString(icon);
-
-            // Add the shortcut as a launcher icon for a full-screen Activity.
-            shortcutIntent = new Intent();
-            shortcutIntent.setAction(sDelegate.getFullscreenAction())
-                    .putExtra(EXTRA_ICON, encodedIcon)
-                    .putExtra(EXTRA_ID, id)
-                    .putExtra(EXTRA_NAME, name)
-                    .putExtra(EXTRA_SHORT_NAME, shortName)
-                    .putExtra(EXTRA_URL, url)
-                    .putExtra(EXTRA_ORIENTATION, orientation)
-                    .putExtra(EXTRA_MAC, getEncodedMac(context, url))
-                    .putExtra(EXTRA_THEME_COLOR, themeColor)
-                    .putExtra(EXTRA_BACKGROUND_COLOR, backgroundColor)
-                    .putExtra(EXTRA_IS_ICON_GENERATED, isIconGenerated);
+            shortcutIntent = createWebappShortcutIntent(id, sDelegate.getFullscreenAction(), url,
+                    name, shortName, icon, orientation, themeColor, backgroundColor,
+                    isIconGenerated);
+            shortcutIntent.putExtra(EXTRA_MAC, getEncodedMac(context, url));
         } else {
             // Add the shortcut as a launcher icon to open in the browser Activity.
             shortcutIntent = createShortcutIntent(url);
         }
 
         // Always attach a source (one of add to home screen menu item, app banner, or unknown) to
         // the intent. This allows us to distinguish where a shortcut was added from in metrics.
         shortcutIntent.putExtra(EXTRA_SOURCE, source);
         shortcutIntent.setPackage(context.getPackageName());
         sDelegate.sendBroadcast(
                 context, createAddToHomeIntent(url, userTitle, icon, shortcutIntent));
 
+        if (isWebappCapable) {
+            // Store the webapp data so that we can access it.
+            WebappRegistry.registerWebapp(context, id).updateFromShortcutIntent(shortcutIntent,
+                    new Runnable() {
+                        @Override
+                        public void run() {
+                            nativeOnWebappDataStored(callbackPointer, true);
+                        }
+                    });
+        } else {
+            nativeOnWebappDataStored(callbackPointer, false);
+        }
+
         // Alert the user about adding the shortcut.
         Handler handler = new Handler(Looper.getMainLooper());
         handler.post(new Runnable() {
             @Override
             public void run() {
                 Context applicationContext = ApplicationStatus.getApplicationContext();
                 String toastText =
                         applicationContext.getString(R.string.added_to_homescreen, userTitle);
                 Toast toast = Toast.makeText(applicationContext, toastText, Toast.LENGTH_SHORT);
                 toast.show();
             }
         });
     }
 
     /**
      * Creates a storage location and stores the data for a web app using {@link WebappDataStorage}.
      * @param context     Context to open the WebappDataStorage with.
      * @param id          ID of the webapp which is storing data.
-     * @param scope       scope of the webapp which is storing data.
      * @param splashImage Image which should be displayed on the splash screen of
      *                    the webapp. This can be null of there is no image to show.
      */
     @SuppressWarnings("unused")
     @CalledByNative
-    private static void storeWebappData(Context context, String id, String scope,
-            Bitmap splashImage) {
-        WebappRegistry.registerWebapp(context, id, scope);
-        WebappDataStorage.open(context, id).updateSplashScreenImage(splashImage);
+    private static void storeWebappSplashImage(final Context context, final String id,
+            final Bitmap splashImage) {
+        WebappRegistry.getWebappDataStorage(context, id,
+                new WebappRegistry.FetchWebappDataStorageCallback() {
+                    @Override
+                    public void onWebappDataStorageRetrieved(WebappDataStorage storage) {
+                        if (storage == null) return;
+
+                        storage.updateSplashScreenImage(splashImage);
+                    }
+
+                }
+        );
     }
 
     /**
      * Creates an intent that will add a shortcut to the home screen.
      * @param shortcutIntent Intent to fire when the shortcut is activated.
      * @param url URL of the shortcut.
      * @param title Title of the shortcut.
      * @param icon Image that represents the shortcut.
      * @return Intent for the shortcut.
      */
@@ skipping 12 matching lines @@
      * @param title Title of the shortcut.
      * @param icon Image that represents the shortcut.
      * @return Intent for the shortcut.
      */
     public static Intent createAddToHomeIntent(String url, String title, Bitmap icon) {
         Intent shortcutIntent = createShortcutIntent(url);
         return createAddToHomeIntent(url, title, icon, shortcutIntent);
     }
 
     /**
+     * Webapp-capable shortcut intent for icon on home screen.
+     * @param action Intent action to open a full screen activity.
+     * @param id  Id of the webapp.

[gone, 2016/04/01 23:44:48]

nit: Align these maybe?  It's kind of blobby atm.

[dominickn, 2016/04/04 07:26:23]

Done.

+     * @param url Url of the webapp.
+     * @param name Name of the webapp.
+     * @param shortName Short name of the webapp. @param icon Icon of the webapp.
+     * @param orientation Orientation of the webapp.
+     * @param themeColor Theme color of the webapp.
+     * @param backgroundColor Background color of the webapp.
+     * @param isIconGenerated True if the icon is generated by Chromium.
+     * @return Intent for onclick action of the shortcut.
+     */
+    public static Intent createWebappShortcutIntent(String id, String action, String url,
+            String name, String shortName, Bitmap icon, int orientation, long themeColor,
+            long backgroundColor, boolean isIconGenerated) {
+        // Encode the icon as a base64 string (Launcher drops Bitmaps in the Intent).
+        String encodedIcon = encodeBitmapAsString(icon);
+
+        // Create an intent as a launcher icon for a full-screen Activity.
+        // The scope is set to the URL until the manifest scope member is available.
+        Intent shortcutIntent = new Intent();
+        shortcutIntent.setAction(action)
+                .putExtra(EXTRA_ID, id)
+                .putExtra(EXTRA_URL, url)
+                .putExtra(EXTRA_SCOPE, url)
+                .putExtra(EXTRA_NAME, name)
+                .putExtra(EXTRA_SHORT_NAME, shortName)
+                .putExtra(EXTRA_ICON, encodedIcon)
+                .putExtra(EXTRA_ORIENTATION, orientation)
+                .putExtra(EXTRA_THEME_COLOR, themeColor)
+                .putExtra(EXTRA_BACKGROUND_COLOR, backgroundColor)
+                .putExtra(EXTRA_IS_ICON_GENERATED, isIconGenerated);
+        return shortcutIntent;
+    }
+
+    /**
      * Shortcut intent for icon on home screen.
      * @param url Url of the shortcut.
      * @return Intent for onclick action of the shortcut.
      */
     public static Intent createShortcutIntent(String url) {
         Intent shortcutIntent = new Intent(Intent.ACTION_VIEW, Uri.parse(url));
         shortcutIntent.putExtra(REUSE_URL_MATCHING_TAB_ELSE_NEW_TAB, true);
         return shortcutIntent;
     }
 
@@ skipping 226 matching lines @@
         Drawable drawable = ApiCompatibilityUtils.getDrawableForDensity(
                 context.getResources(), id, density);
 
         if (drawable instanceof BitmapDrawable) {
             BitmapDrawable bd = (BitmapDrawable) drawable;
             return bd.getBitmap();
         }
         assert false : "The drawable was not a bitmap drawable as expected";
         return null;
     }
+
+    private static native void nativeOnWebappDataStored(long callbackPointer, boolean wasStored);
 }