webapps: Add cleanup task when opening up WebappActivity to clean old web apps

chrome/android/junit/src/org/chromium/chrome/browser/webapps/WebappRegistryTest.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 166 matching lines @@
                 .putLong(WebappDataStorage.KEY_LAST_USED, 100L)
                 .commit();
 
         WebappRegistry.unregisterAllWebapps(Robolectric.application, null);
         BackgroundShadowAsyncTask.runBackgroundTasks();
 
         Map<String, ?> actual = webAppPrefs.getAll();
         assertTrue(actual.isEmpty());
     }
 
+    @Test
+    @Feature({"Webapp"})
+    public void testCleanupDoesNotRunTooOften() throws Exception {
+        addWebappsToRegistry("oldWebapp");
+        SharedPreferences webAppPrefs = Robolectric.application.getSharedPreferences(
+                WebappDataStorage.SHARED_PREFS_FILE_PREFIX + "oldWebapp", Context.MODE_PRIVATE);
+        webAppPrefs.edit()
+                .putLong(WebappDataStorage.KEY_LAST_USED, Long.MIN_VALUE)
+                .commit();
+
+        // Time just before the task should run.
+        WebappRegistry.unregisterOldWebapps(Robolectric.application,
+                WebappRegistry.FULL_CLEANUP_DURATION - 1);
+        BackgroundShadowAsyncTask.runBackgroundTasks();
+
+        Set<String> actual = mSharedPreferences.getStringSet(
+                WebappRegistry.KEY_WEBAPP_SET, Collections.<String>emptySet());
+        assertEquals(new HashSet<String>(Arrays.asList("test")), actual);

[gone, 2015/09/24 12:45:10]

teeeeeeeeeeeeeeeeest?

[Lalit Maganti, 2015/09/24 12:49:25]

Done.

+
+        long actualLastUsed = webAppPrefs.getLong(WebappDataStorage.KEY_LAST_USED,
+                WebappDataStorage.INVALID_LAST_USED);
+        assertEquals(Long.MIN_VALUE, actualLastUsed);
+    }
+
+    @Test
+    @Feature({"Webapp"})
+    public void testCleanupDoesNotRemoveRecentApps() throws Exception {
+        long currentTime = WebappRegistry.FULL_CLEANUP_DURATION;
+
+        addWebappsToRegistry("recentWebapp");
+        SharedPreferences webAppPrefs = Robolectric.application.getSharedPreferences(
+                WebappDataStorage.SHARED_PREFS_FILE_PREFIX + "recentWebapp", Context.MODE_PRIVATE);
+
+        // Put the last used time just inside the no-cleanup window.
+        long lastUsed = currentTime - WebappRegistry.WEBAPP_UNOPENED_CLEANUP_DURATION + 1;
+        webAppPrefs.edit()
+                .putLong(WebappDataStorage.KEY_LAST_USED, lastUsed)
+                .commit();
+
+        WebappRegistry.unregisterOldWebapps(Robolectric.application, currentTime);
+        BackgroundShadowAsyncTask.runBackgroundTasks();
+
+        Set<String> actual = mSharedPreferences.getStringSet(
+                WebappRegistry.KEY_WEBAPP_SET, Collections.<String>emptySet());
+        assertEquals(new HashSet<String>(Arrays.asList("test")), actual);

[gone, 2015/09/24 12:45:10]

"test"?

[Lalit Maganti, 2015/09/24 12:49:25]

Done.

+
+        long actualLastUsed = webAppPrefs.getLong(WebappDataStorage.KEY_LAST_USED,
+                WebappDataStorage.INVALID_LAST_USED);
+        assertEquals(lastUsed, actualLastUsed);
+    }
+
+    @Test
+    @Feature({"Webapp"})
+    public void testCleanupRemovesOldApps() throws Exception {
+        long currentTime = WebappRegistry.FULL_CLEANUP_DURATION;
+
+        addWebappsToRegistry("oldWebapp");
+        SharedPreferences webAppPrefs = Robolectric.application.getSharedPreferences(
+                WebappDataStorage.SHARED_PREFS_FILE_PREFIX + "oldWebapp", Context.MODE_PRIVATE);
+
+        // Put the last used time just outside the no-cleanup window.
+        long lastUsed = currentTime - WebappRegistry.WEBAPP_UNOPENED_CLEANUP_DURATION;

[gone, 2015/09/24 12:45:10]

any reason why this doesn't +1 but the one above does?

[Lalit Maganti, 2015/09/24 12:49:25]

The one above is just outside the cleanup window and this just is just inside -
they are testing different things.

+        webAppPrefs.edit()
+                .putLong(WebappDataStorage.KEY_LAST_USED, lastUsed)
+                .commit();
+
+        WebappRegistry.unregisterOldWebapps(Robolectric.application, currentTime);
+        BackgroundShadowAsyncTask.runBackgroundTasks();
+
+        Set<String> actual = mSharedPreferences.getStringSet(
+                WebappRegistry.KEY_WEBAPP_SET, Collections.<String>emptySet());
+        assertTrue(actual.isEmpty());
+
+        long actualLastUsed = webAppPrefs.getLong(WebappDataStorage.KEY_LAST_USED,
+                WebappDataStorage.INVALID_LAST_USED);
+        assertEquals(WebappDataStorage.INVALID_LAST_USED, actualLastUsed);
+    }
+
     private Set<String> addWebappsToRegistry(String... webapps) {
         final Set<String> expected = new HashSet<String>(Arrays.asList(webapps));
         mSharedPreferences.edit()
                 .putStringSet(WebappRegistry.KEY_WEBAPP_SET, expected)
                 .commit();
         return expected;
     }
 
     private Set<String> getRegisteredWebapps() {
         return mSharedPreferences.getStringSet(
                 WebappRegistry.KEY_WEBAPP_SET, Collections.<String>emptySet());
     }
 }