Upstream: Launch WebApkActivity from WebAPK.

chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappActivity.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.Intent;
 import android.graphics.Bitmap;
 import android.graphics.Color;
 import android.graphics.drawable.Drawable;
@@ skipping 11 matching lines @@
 
 import org.chromium.base.ActivityState;
 import org.chromium.base.ApiCompatibilityUtils;
 import org.chromium.base.ApplicationStatus;
 import org.chromium.base.Log;
 import org.chromium.base.StreamUtil;
 import org.chromium.base.VisibleForTesting;
 import org.chromium.base.metrics.RecordHistogram;
 import org.chromium.blink_public.platform.WebDisplayMode;
 import org.chromium.chrome.R;
+import org.chromium.chrome.browser.ShortcutHelper;
 import org.chromium.chrome.browser.TabState;
 import org.chromium.chrome.browser.document.DocumentUtils;
 import org.chromium.chrome.browser.fullscreen.ChromeFullscreenManager;
 import org.chromium.chrome.browser.metrics.WebappUma;
 import org.chromium.chrome.browser.tab.EmptyTabObserver;
 import org.chromium.chrome.browser.tab.Tab;
 import org.chromium.chrome.browser.tab.TabDelegateFactory;
 import org.chromium.chrome.browser.tab.TabObserver;
 import org.chromium.chrome.browser.tab.TopControlsVisibilityDelegate;
 import org.chromium.chrome.browser.util.ColorUtils;
@@ skipping 16 matching lines @@
  */
 public class WebappActivity extends FullScreenActivity {
     public static final String WEBAPP_SCHEME = "webapp";
 
     private static final String TAG = "WebappActivity";
     private static final long MS_BEFORE_NAVIGATING_BACK_FROM_INTERSTITIAL = 1000;
 
     private final WebappDirectoryManager mDirectoryManager;
 
     private WebappInfo mWebappInfo;
+    private boolean mIsWebApk;

[gone, 2016/05/21 00:07:28]

Do you gain anything by caching this boolean?  mWebappInfo should be the source
of truth and isWebApk() looks cheap enough to call.  I'm afraid of a disconnect
between these two.

[Xi Han, 2016/05/25 14:37:58]

We don't need this function after introducing WebApkActivity!

 
     private boolean mOldWebappCleanupStarted;
 
     private ViewGroup mSplashScreen;
     private WebappUrlBar mUrlBar;
 
     private boolean mIsInitialized;
     private Integer mBrandColor;
 
     private WebappUma mWebappUma;
@@ skipping 13 matching lines @@
 
     @Override
     protected void onNewIntent(Intent intent) {
         if (intent == null) return;
         super.onNewIntent(intent);
 
         WebappInfo newWebappInfo = WebappInfo.create(intent);
         if (newWebappInfo == null) {
             Log.e(TAG, "Failed to parse new Intent: " + intent);
             finish();
+            return;
         } else if (!TextUtils.equals(mWebappInfo.id(), newWebappInfo.id())) {
             mWebappInfo = newWebappInfo;
+            mIsWebApk = isWebApk(mWebappInfo);
             resetSavedInstanceState();
             if (mIsInitialized) initializeUI(null);
             // TODO(dominickn): send the web app into fullscreen if mDisplayMode is
             // WebDisplayMode.Fullscreen. See crbug.com/581522
         }
+        if (mIsWebApk) {
+            // We could bring a WebAPK hosted WebappActivity to foreground and navigate it to a
+            // different URL. For example, WebAPK "foo" is launched and navigates to
+            // "www.foo.com/foo". In Chrome, user clicks a link "www.foo.com/bar" in Google search
+            // results. After clicking the link, WebAPK "foo" is brought to foreground, and
+            // loads the page of "www.foo.com/bar" at the same time.
+            // The extra {@link ShortcutHelper.EXTRA_URL} provides the URL that the WebAPK will
+            // navigate to.
+            String overrideUrl = intent.getStringExtra(ShortcutHelper.EXTRA_URL);
+            if (overrideUrl != null && mIsInitialized
+                    && !overrideUrl.equals(getActivityTab().getUrl())) {
+                getActivityTab().loadUrl(
+                        new LoadUrlParams(overrideUrl, PageTransition.AUTO_TOPLEVEL));
+            }
+        }
     }
 
     private void initializeUI(Bundle savedInstanceState) {
         // We do not load URL when restoring from saved instance states.
         if (savedInstanceState == null && mWebappInfo.isInitialized()) {
             if (TextUtils.isEmpty(getActivityTab().getUrl())) {
                 getActivityTab().loadUrl(new LoadUrlParams(
                         mWebappInfo.uri().toString(), PageTransition.AUTO_TOPLEVEL));
             }
         } else {
             if (NetworkChangeNotifier.isOnline()) getActivityTab().reloadIgnoringCache();
         }
 
         getActivityTab().addObserver(createTabObserver());
         getActivityTab().getTabWebContentsDelegateAndroid().setDisplayMode(
                 WebDisplayMode.Standalone);
         // TODO(dominickn): send the web app into fullscreen if mDisplayMode is
         // WebDisplayMode.Fullscreen. See crbug.com/581522
     }
 
     @Override
     public void preInflationStartup() {
         WebappInfo info = WebappInfo.create(getIntent());
-        if (info != null) mWebappInfo = info;
+        if (info != null) {
+            mWebappInfo = info;
+            mIsWebApk = isWebApk(info);
+        }
 
         ScreenOrientationProvider.lockOrientation((byte) mWebappInfo.orientation(), this);
         super.preInflationStartup();
     }
 
     @Override
     public void finishNativeInitialization() {
         if (!mWebappInfo.isInitialized()) finish();
         super.finishNativeInitialization();
         initializeUI(getSavedInstanceState());
@@ skipping 110 matching lines @@
         contentView.addView(mSplashScreen);
 
         mWebappUma.splashscreenVisible();
         mWebappUma.recordSplashscreenBackgroundColor(mWebappInfo.hasValidBackgroundColor()
                 ? WebappUma.SPLASHSCREEN_COLOR_STATUS_CUSTOM
                 : WebappUma.SPLASHSCREEN_COLOR_STATUS_DEFAULT);
         mWebappUma.recordSplashscreenThemeColor(mWebappInfo.hasValidThemeColor()
                 ? WebappUma.SPLASHSCREEN_COLOR_STATUS_CUSTOM
                 : WebappUma.SPLASHSCREEN_COLOR_STATUS_DEFAULT);
 
+        if (mIsWebApk) {
+            // TODO(hanxi): Removes this check when WebAPKs are registered in WebappRegistry.
+            initializeSplashScreenWidgets(backgroundColor, null);
+            return;
+        }
+
         final Intent intent = getIntent();
         WebappRegistry.getWebappDataStorage(this, mWebappInfo.id(),
                 new WebappRegistry.FetchWebappDataStorageCallback() {
                     @Override
                     public void onWebappDataStorageRetrieved(WebappDataStorage storage) {
                         if (storage == null) return;
 
                         // The information in the WebappDataStorage may have been purged by the
                         // user clearing their history or not launching the web app recently.
                         // Restore the data if necessary from the intent.
@@ skipping 328 matching lines @@
                 || securityLevel == ConnectionSecurityLevel.SECURITY_WARNING;
         return visible;
     }
 
     // We're temporarily disable CS on webapp since there are some issues. (http://crbug.com/471950)
     // TODO(changwan): re-enable it once the issues are resolved.
     @Override
     protected boolean isContextualSearchAllowed() {
         return false;
     }
+
+    /**
+     * Returns whether {@link info} refers to a WebAPK.
+     * Assumes that WebappLauncherActivity does not create WebappActivities for APKs whose package
+     * starts with WEBAPK_PACKAGE_PREFIX but which are not WebAPKs (due to an incorrect signature).
+     * @param info
+     */
+    private static boolean isWebApk(WebappInfo info) {
+        return info.webApkPackageName() != null;
+    }
 }