Store URLs in WebappDataStorage, and purge them when history is cleared.

chrome/android/junit/src/org/chromium/chrome/browser/webapps/WebappDataStorageTest.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 static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertTrue;
 
 import android.content.Context;
@@ skipping 31 matching lines @@
 
         mCallbackCalled = false;
     }
 
     @Test
     @Feature({"Webapp"})
     public void testBackwardCompat() {
         assertEquals("webapp_", WebappDataStorage.SHARED_PREFS_FILE_PREFIX);
         assertEquals("splash_icon", WebappDataStorage.KEY_SPLASH_ICON);
         assertEquals("last_used", WebappDataStorage.KEY_LAST_USED);
+        assertEquals("origin_url", WebappDataStorage.KEY_ORIGIN_URL);
     }
 
     @Test
     @Feature({"Webapp"})
     public void testLastUsedRetrieval() throws Exception {
         mSharedPreferences.edit()
                 .putLong(WebappDataStorage.KEY_LAST_USED, 100L)
                 .commit();
 
         WebappDataStorage.getLastUsedTime(Robolectric.application, "test",
@@ skipping 74 matching lines @@
             for (int j = 0; j < actual.getHeight(); j++) {
                 if (actual.getPixel(i, j) != 0) return false;
             }
         }
         return true;
     }
 
     private static Bitmap createBitmap() {
         return Bitmap.createBitmap(1, 1, Bitmap.Config.ARGB_4444);
     }
-}
+
+    @Test
+    @Feature({"Webapp"})
+    public void testOriginUrlRetrieval() throws Exception {
+        final String origin = "http://drive.google.com";
+        mSharedPreferences.edit()
+                .putString(WebappDataStorage.KEY_ORIGIN_URL, origin)
+                .commit();
+
+        WebappDataStorage.getOriginUrl(Robolectric.application, "test",

[gone, 2016/03/08 23:03:52]

Maybe it'd make sense to create and declare the FetchCallback outside of the
function invocation.  It'd help with readability.

[dominickn, 2016/03/09 08:18:32]

Done.

+                new WebappDataStorage.FetchCallback<String>() {
+                    @Override
+                    public void onDataRetrieved(String readObject) {
+                        mCallbackCalled = true;
+                        assertEquals(origin, readObject);
+                    }
+                });
+        BackgroundShadowAsyncTask.runBackgroundTasks();
+        Robolectric.runUiThreadTasks();
+
+        assertTrue(mCallbackCalled);
+    }
+
+    @Test
+    @Feature({"Webapp"})
+    public void testOriginUrlUpdate() throws Exception {
+        final String origin = "http://maps.google.com";
+
+        WebappDataStorage.updateOriginUrl(Robolectric.application, "test", origin);
+        BackgroundShadowAsyncTask.runBackgroundTasks();
+        Robolectric.runUiThreadTasks();
+
+        assertEquals(origin, mSharedPreferences.getString(WebappDataStorage.KEY_ORIGIN_URL, null));
+    }
+}