Accept any string for currency code in PaymentRequest.

chrome/android/junit/src/org/chromium/chrome/browser/payments/CurrencyStringFormatterTest.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;
 
 import org.junit.Assert;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.junit.runners.Parameterized;
@@ skipping 51 matching lines @@
 
             {"123456789012345678901234567890.123456789012345678901234567890", "USD", "fr-FR",
                     "$123" + SPACE + "456" + SPACE + "789" + SPACE + "012" + SPACE + "345"
                             + SPACE + "678" + SPACE + "901" + SPACE + "234" + SPACE + "567"
                             + SPACE + "890,123456789012345678901234567890",
                     ExpectedValidity.VALID_AMOUNT},
             {"123456789012345678901234567890.123456789012345678901234567890", "USD", "en-US",
                     "$123,456,789,012,345,678,901,234,567,890.123456789012345678901234567890",
                     ExpectedValidity.VALID_AMOUNT},
 
-            // Invalid amount currency codes.
-            {"55.00", "", "en-US", null, ExpectedValidity.INVALID_AMOUNT_CURRENCY_CODE},
-            {"55.00", "usd", "en-US", null, ExpectedValidity.INVALID_AMOUNT_CURRENCY_CODE},
-            {"55.00", "US8", "en-US", null, ExpectedValidity.INVALID_AMOUNT_CURRENCY_CODE},
-            {"55.00", "US", "en-US", null, ExpectedValidity.INVALID_AMOUNT_CURRENCY_CODE},
-            {"55.00", "USDR", "en-US", null, ExpectedValidity.INVALID_AMOUNT_CURRENCY_CODE},
-            {"55.00", "USDr", "en-US", null, ExpectedValidity.INVALID_AMOUNT_CURRENCY_CODE},

[please use gerrit instead, 2016/08/25 16:47:38]

Once you move currency truncation into CurrenycStringFormatter, please add a few
test cases that check the truncation logic.

[pals, 2016/08/26 07:20:26]

If it is ok, I'll do the application related changes as a follow up patch?

[please use gerrit instead, 2016/08/26 17:01:37]

OK

-
             // Invalid amount values.
             {"", "USD", "en-US", null, ExpectedValidity.INVALID_AMOUNT_VALUE},
             {"-", "USD", "en-US", null, ExpectedValidity.INVALID_AMOUNT_VALUE},
             {"notdigits", "USD", "en-US", null, ExpectedValidity.INVALID_AMOUNT_VALUE},
             {"ALSONOTDIGITS", "USD", "en-US", null, ExpectedValidity.INVALID_AMOUNT_VALUE},
             {"10.", "USD", "en-US", null, ExpectedValidity.INVALID_AMOUNT_VALUE},
             {".99", "USD", "en-US", null, ExpectedValidity.INVALID_AMOUNT_VALUE},
             {"10-", "USD", "en-US", null, ExpectedValidity.INVALID_AMOUNT_VALUE},
             {"1-0", "USD", "en-US", null, ExpectedValidity.INVALID_AMOUNT_VALUE},
             {"1.0.0", "USD", "en-US", null, ExpectedValidity.INVALID_AMOUNT_VALUE},
@@ skipping 37 matching lines @@
                     formatter.isValidAmountValue(mAmount));
         }
 
         if (mExpectedValidity == ExpectedValidity.VALID_AMOUNT) {
             Assert.assertEquals("\"" + mCurrency + "\" \"" + mAmount + "\" (\"" + mLanguageTag
                             + "\" locale) should be formatted into \"" + mExpectedFormatting + "\"",
                     mExpectedFormatting, formatter.format(mAmount));
         }
     }
 }