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

chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappRegistry.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.Context;
 import android.content.SharedPreferences;
 import android.os.AsyncTask;
 
@@ skipping 20 matching lines @@
     static final String KEY_WEBAPP_SET = "webapp_set";
     static final String KEY_LAST_CLEANUP = "last_cleanup";
 
     /** Represents a period of 4 weeks in milliseconds */
     static final long FULL_CLEANUP_DURATION = TimeUnit.DAYS.toMillis(4L * 7L);
 
     /** Represents a period of 13 weeks in milliseconds */
     static final long WEBAPP_UNOPENED_CLEANUP_DURATION = TimeUnit.DAYS.toMillis(13L * 7L);
 
     /**
-     * Called when a retrieval of the stored web apps occurs.
+     * Called when a retrieval of the set of stored web app ids occurs.

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

nit: IDs

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

Done.

      */
     public interface FetchCallback {
         public void onWebappIdsRetrieved(Set<String> readObject);
     }
 
     /**
-     * Registers the existence of a web app and creates the SharedPreference for it.
+     * Called when a retrieval of the stored WebappDataStorage occurs. The storage parameter will
+     * be null if the web app queried for was not in the registry.
+     */
+    public interface FetchWebappDataStorageCallback {
+        public void onWebappDataStorageRetrieved(WebappDataStorage storage);
+    }
+
+    /**
+     * Registers the existence of a web app, creates the SharedPreference for it, and returns the
+     * resulting WebappDataStorage object.
      * @param context  Context to open the registry with.
      * @param webappId The id of the web app to register.
+     * @return The storage object for the web app.
      */
-    public static void registerWebapp(final Context context, final String webappId,
-            final String scope) {
+    public static WebappDataStorage registerWebapp(final Context context, final String webappId) {
+        final WebappDataStorage storage = new WebappDataStorage(context, webappId);
         new AsyncTask<Void, Void, Void>() {
             @Override
             protected final Void doInBackground(Void... nothing) {
                 SharedPreferences preferences = openSharedPreferences(context);
+                // The set returned by getRegisteredWebappIds must be treated as immutable, so we
+                // make a copy to edit and save.
                 Set<String> webapps = new HashSet<String>(getRegisteredWebappIds(preferences));
                 boolean added = webapps.add(webappId);
                 assert added;
 
+                preferences.edit().putStringSet(KEY_WEBAPP_SET, webapps).apply();
+
                 // Update the last used time, so we can guarantee that a web app which appears in
                 // the registry will have a last used time != WebappDataStorage.LAST_USED_INVALID.
-                WebappDataStorage storage = new WebappDataStorage(context, webappId);
-                storage.setScope(scope);
                 storage.updateLastUsedTime();
-                preferences.edit().putStringSet(KEY_WEBAPP_SET, webapps).apply();
                 return null;
             }
         }.execute();
+
+        return storage;
+    }
+
+    /**
+     * Runs the callback, supplying the WebappDataStorage object for webappId, or null if the web
+     * app has not been registered.
+     * @param context  Context to open the registry with.
+     * @param webappId The id of the web app to register.
+     * @return The storage object for the web app, or null if webappId is not registered.
+     */
+    public static void getWebappDataStorage(final Context context, final String webappId,
+            final FetchWebappDataStorageCallback callback) {
+        new AsyncTask<Void, Void, Boolean>() {
+            @Override
+            protected final Boolean doInBackground(Void... nothing) {
+                SharedPreferences preferences = openSharedPreferences(context);
+                return getRegisteredWebappIds(preferences).contains(webappId);
+            }
+
+            protected final void onPostExecute(Boolean exists) {
+                WebappDataStorage storage = null;
+                if (exists) {
+                    storage = WebappDataStorage.open(context, webappId);
+                }
+                callback.onWebappDataStorageRetrieved(storage);
+            }
+        }.execute();
     }
 
     /**
      * Asynchronously retrieves the list of web app IDs which this registry is aware of.
      * @param context  Context to open the registry with.
      * @param callback Called when the set has been retrieved. The set may be empty.
      */
     @VisibleForTesting
     public static void getRegisteredWebappIds(final Context context, final FetchCallback callback) {
         new AsyncTask<Void, Void, Set<String>>() {
@@ skipping 119 matching lines @@
     private static Set<String> getRegisteredWebappIds(SharedPreferences preferences) {
         return preferences.getStringSet(KEY_WEBAPP_SET, Collections.<String>emptySet());
     }
 
     private WebappRegistry() {
     }
 
     private static native void nativeOnWebappsUnregistered(long callbackPointer);
     private static native void nativeOnClearedWebappHistory(long callbackPointer);
 }