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

chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappActivity.java

Index: chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappActivity.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappActivity.java b/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappActivity.java
index 945a9645acdd4855e7534dbe4fc7006adff272eb..fc3d295d7f60903299d6e9f4a1038e7f8a07f4bc 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappActivity.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappActivity.java
@@ -52,6 +52,7 @@ public class WebappActivity extends FullScreenActivity {
 
     private final WebappInfo mWebappInfo;
     private AsyncTask<WebappActivity, Void, Void> mCleanupTask;
+    private boolean mOldWebappCleanupStarted;
 
     private WebContentsObserver mWebContentsObserver;
 
@@ -165,6 +166,13 @@ public class WebappActivity extends FullScreenActivity {
     public void postInflationStartup() {
         initializeSplashScreen();
 
+        // Kick off the old web app cleanup (if we haven't alread) now that we have queued the

[gone, 2015/09/24 10:27:11]

nit: alread?

I'd move this further down in the pipeline (maybe onStart or onResume).  Even if
it's an asynctask, it still takes cycles to kick it off.

[Lalit Maganti, 2015/09/24 12:35:44]

Moved to onResume as this point, UI work should have almost been done.

+        // current web app's storage to be opened.
+        if (!mOldWebappCleanupStarted) {
+            WebappRegistry.unregisterOldWebapps(this);
+            mOldWebappCleanupStarted = true;
+        }
+
         super.postInflationStartup();
         WebappControlContainer controlContainer =
                 (WebappControlContainer) findViewById(R.id.control_container);