PaymentRequest: complete() method should take PaymentComplete enum value.

chrome/android/java/src/org/chromium/chrome/browser/payments/ui/PaymentRequestUI.java

 // Copyright 2016 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.payments.ui;
 
 import android.animation.Animator;
 import android.animation.AnimatorListenerAdapter;
 import android.animation.AnimatorSet;
 import android.animation.ObjectAnimator;
@@ skipping 380 matching lines @@
      *  <li>Successfully processing the payment.</li>
      *  <li>Failure to process the payment.</li>
      *  <li>The JavaScript calling the abort() method in PaymentRequest API.</li>
      *  <li>The PaymentRequest JavaScript object being destroyed.</li>
      * </ul>
      *
      * Does not call Client.onDismissed().
      *
      * Should not be called multiple times.
      *
-     * @param paymentSuccess Whether the payment (if any) was successful.
+     * @param immediateClose Whether dismissing the dialog immediately without describing error.

[palmer, 2016/06/16 22:00:55]

This comment is not a complete English sentence. How about:

  @param shouldCloseImmediately If true, this function will immediately dismiss
the dialog without describing the error.

      * @param callback The callback to notify of finished animations.
      */
-    public void close(boolean paymentSuccess, final Runnable callback) {
+    public void close(boolean immediateClose, final Runnable callback) {
         mIsClientClosing = true;
-        mResultView.update(paymentSuccess, new Runnable() {
+        mResultView.update(immediateClose, new Runnable() {
             @Override
             public void run() {
                 dismissDialog(false);
                 if (callback != null) callback.run();
             }
         });
         if (sObserverForTest != null) sObserverForTest.onPaymentRequestResultReady(this);
     }
 
     /**
@@ skipping 517 matching lines @@
             sObserverForTest.onPaymentRequestReadyToPay(this);
         }
     }
 
     private void notifyReadyToClose() {
         if (sObserverForTest != null && isAcceptingCloseButton()) {
             sObserverForTest.onPaymentRequestReadyToClose(this);
         }
     }
 }