[Payments] Implement openWindow for service worker based payment handler

chrome/android/java/src/org/chromium/chrome/browser/ServiceTabLauncher.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;
 
 import android.content.Intent;
 import android.os.AsyncTask;
+import android.support.annotation.Nullable;
+import android.text.TextUtils;
 
 import org.chromium.base.ContextUtils;
+import org.chromium.base.ThreadUtils;
 import org.chromium.base.annotations.CalledByNative;
 import org.chromium.chrome.browser.tab.Tab;
 import org.chromium.chrome.browser.tabmodel.TabModel.TabLaunchType;
 import org.chromium.chrome.browser.tabmodel.document.AsyncTabCreationParams;
 import org.chromium.chrome.browser.tabmodel.document.TabDelegate;
 import org.chromium.chrome.browser.webapps.ChromeWebApkHost;
 import org.chromium.chrome.browser.webapps.WebappDataStorage;
 import org.chromium.chrome.browser.webapps.WebappRegistry;
 import org.chromium.content_public.browser.LoadUrlParams;
 import org.chromium.content_public.browser.WebContents;
 import org.chromium.content_public.common.Referrer;
 import org.chromium.content_public.common.ResourceRequestBody;
 import org.chromium.ui.base.PageTransition;
+import org.chromium.ui.mojom.WindowOpenDisposition;
 import org.chromium.webapk.lib.client.WebApkNavigationClient;
 import org.chromium.webapk.lib.client.WebApkValidator;
 
 /**
  * Tab Launcher to be used to launch new tabs from background Android Services,
  * when it is not known whether an activity is available. It will send an intent to launch the
  * activity.
  *
  * URLs within the scope of a recently launched standalone-capable web app on the Android home
  * screen are launched in the standalone web app frame.
  */
 public class ServiceTabLauncher {
     // Name of the extra containing the Id of a tab launch request id.
     public static final String LAUNCH_REQUEST_ID_EXTRA =
             "org.chromium.chrome.browser.ServiceTabLauncher.LAUNCH_REQUEST_ID";
 
     /**
      * Launches the browser activity and launches a tab for |url|.
-     *  @param requestId Id of the request for launching this tab.
-     * @param incognito Whether the tab should be launched in incognito mode.
-     * @param url The URL which should be launched in a tab.
-     * @param disposition The disposition requested by the navigation source.
-     * @param referrerUrl URL of the referrer which is opening the page.
+     * @param requestId      Id of the request for launching this tab.
+     * @param incognito      Whether the tab should be launched in incognito mode.
+     * @param redirectUrl    The last redirect URL that occured before |url|.
+     * @param url            The URL which should be launched in a tab.
+     * @param disposition    The disposition requested by the navigation source.
+     * @param referrerUrl    URL of the referrer which is opening the page.
      * @param referrerPolicy The referrer policy to consider when applying the referrer.
-     * @param extraHeaders Extra headers to apply when requesting the tab's URL.
-     * @param postData Post-data to include in the tab URL's request body.
+     * @param extraHeaders   Extra headers to apply when requesting the tab's URL.
+     * @param postData       Post-data to include in the tab URL's request body.
      */
     @CalledByNative
-    public static void launchTab(final int requestId, final boolean incognito, final String url,
-            final int disposition, final String referrerUrl, final int referrerPolicy,
-            final String extraHeaders, final ResourceRequestBody postData) {
+    public static void launchTab(final int requestId, final boolean incognito,
+            final String redirectUrl, final String url, final int disposition,
+            final String referrerUrl, final int referrerPolicy, final String extraHeaders,
+            final ResourceRequestBody postData) {
         final TabDelegate tabDelegate = new TabDelegate(incognito);
 
+        // Open the url in popup custom tab over the tab showing the redirect url.
+        if (disposition == WindowOpenDisposition.NEW_POPUP && !TextUtils.isEmpty(redirectUrl)) {

[please use gerrit instead, 2017/06/01 17:50:37]

I recommend to use an enum for the reason why the tab is launching. That would
be more clear than checking !TextUtils.isEmpty(redirectUrl).

Example enum values:

final static int LAUNCH_TAB_FOR_PAYMENT_HANDLER = 0;
final static int LAUNCH_TAB_FOR_OTHER = 1;

[gogerald1, 2017/06/02 03:33:11]

Acknowledged.

+            if (!tabDelegate.createPopupCustomTab(requestId, redirectUrl, url)) {
+                ThreadUtils.postOnUiThread(new Runnable() {
+                    @Override
+                    public void run() {
+                        onWebContentsForRequestAvailable(requestId, null);
+                    }
+                });
+            }
+            return;
+        }
+
         // 1. Launch WebAPK if one matches the target URL.
         if (ChromeWebApkHost.isEnabled()) {
             String webApkPackageName =
                     WebApkValidator.queryWebApkPackage(ContextUtils.getApplicationContext(), url);
             if (webApkPackageName != null) {
                 Intent intent = WebApkNavigationClient.createLaunchWebApkIntent(
                         webApkPackageName, url, true /* forceNavigation */);
                 if (intent != null) {
                     intent.putExtra(ShortcutHelper.EXTRA_SOURCE, ShortcutSource.NOTIFICATION);
                     ContextUtils.getApplicationContext().startActivity(intent);
@@ skipping 48 matching lines @@
                     tabDelegate.createNewStandaloneFrame(intent);
                 }
             }.executeOnExecutor(AsyncTask.THREAD_POOL_EXECUTOR);
         }
     }
 
     /**
      * To be called by the activity when the WebContents for |requestId| has been created, or has
      * been recycled from previous use. The |webContents| must not yet have started provisional
      * load for the main frame.
+     * The |webContents| could be null if the request is failed.
      *
      * @param requestId Id of the tab launching request which has been fulfilled.
      * @param webContents The WebContents instance associated with this request.
      */
-    public static void onWebContentsForRequestAvailable(int requestId, WebContents webContents) {
+    public static void onWebContentsForRequestAvailable(
+            int requestId, @Nullable WebContents webContents) {
         nativeOnWebContentsForRequestAvailable(requestId, webContents);
     }
 
     private static native void nativeOnWebContentsForRequestAvailable(
             int requestId, WebContents webContents);
 }