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

chrome/android/java/src/org/chromium/chrome/browser/payments/ui/PaymentResultUIManager.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.content.Context;
 import android.graphics.Bitmap;
 import android.view.LayoutInflater;
 import android.view.View;
 import android.view.View.OnClickListener;
 import android.view.ViewGroup;
 import android.view.ViewGroup.MarginLayoutParams;
 import android.widget.ImageView;
 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.
  */
 public class PaymentResultUIManager {
     private final ViewGroup mResultLayout;
 
     /**
      * Constructs a UI that indicates the status of the payment request.
      *
@@ skipping 27 matching lines @@
                 (MarginLayoutParams) pageInfoGroup.getLayoutParams(), titleEndMargin);
 
         // Indicate that we're processing the data.
         TextView messageView = (TextView) mResultLayout.findViewById(R.id.message);
         messageView.setText(R.string.payments_processing_message);
     }
 
     /**
      * 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 {
             // Describe the error.
             Context context = mResultLayout.getContext();
             TextView resultMessage = (TextView) mResultLayout.findViewById(R.id.message);
             resultMessage.setText(context.getString(R.string.payments_error_message));
             resultMessage.setTextColor(ApiCompatibilityUtils.getColor(
                     context.getResources(), R.color.error_text_color));
             ApiCompatibilityUtils.setTextAlignment(resultMessage, View.TEXT_ALIGNMENT_VIEW_START);
@@ skipping 42 matching lines @@
      * TODO(dfalcantara): The dialog should listen for configuration changes and resize accordingly.
      */
     public static int computeMaxWidth(Context context, int availableWidth, int availableHeight) {
         int baseUnit = context.getResources().getDimensionPixelSize(R.dimen.dialog_width_unit);
         int maxSize = Math.min(availableWidth, availableHeight);
         int multiplier = maxSize / baseUnit;
         int floatingDialogWidth = multiplier * baseUnit;
         return floatingDialogWidth;
     }
 }