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

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

Index: chrome/android/java/src/org/chromium/chrome/browser/payments/ui/PaymentResultUIManager.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/payments/ui/PaymentResultUIManager.java b/chrome/android/java/src/org/chromium/chrome/browser/payments/ui/PaymentResultUIManager.java
index ebb171dbc3a5bad56937078755d84acf1325447c..08912f08b1a2c8047769fd1423faef6d6a069104 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/payments/ui/PaymentResultUIManager.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/payments/ui/PaymentResultUIManager.java
@@ -16,6 +16,7 @@ import android.widget.TextView;
 
 import org.chromium.base.ApiCompatibilityUtils;
 import org.chromium.chrome.R;
+import org.chromium.mojom.payments.PaymentComplete;

[please use gerrit instead, 2016/06/10 19:15:35]

UI should not interact with mojo. Continue using a boolean.

 
 /**
  * Displays the status of a payment request to the user.
@@ -63,11 +64,12 @@ public class PaymentResultUIManager {
     /**
      * Updates the UI to display whether or not the payment request was successful.
      *
-     * @param paymentSuccess Whether or not the payment request was successful.
-     * @param callback       Callback to run upon dismissal.
+     * @param result   Whether or not the payment request was successful.
+     * @param callback Callback to run upon dismissal.
      */
-    public void update(boolean paymentSuccess, final Runnable callback) {
-        if (mResultLayout.getParent() == null || paymentSuccess) {
+    public void update(int result, final Runnable callback) {
+        if (mResultLayout.getParent() == null || result == PaymentComplete.UNKNOWN
+                || result == PaymentComplete.SUCCESS) {
             // Dismiss the dialog immediately.
             callback.run();
         } else {