[Payments] Format and validate phone number by using libphonenumber

chrome/android/java/src/org/chromium/chrome/browser/autofill/PhoneNumberUtil.java

Index: chrome/android/java/src/org/chromium/chrome/browser/autofill/PhoneNumberUtil.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/autofill/PhoneNumberUtil.java b/chrome/android/java/src/org/chromium/chrome/browser/autofill/PhoneNumberUtil.java
new file mode 100644
index 0000000000000000000000000000000000000000..521c58afe722638de433686ebbf0bad8a4b6ac6b
--- /dev/null
+++ b/chrome/android/java/src/org/chromium/chrome/browser/autofill/PhoneNumberUtil.java
@@ -0,0 +1,70 @@
+// Copyright 2017 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.autofill;
+
+import android.text.Editable;
+import android.text.TextWatcher;
+
+import org.chromium.base.annotations.JNINamespace;
+
+/**
+ * Android wrapper of i18n::phonenumbers::PhoneNumberUtil which provides convenient methods to
+ * format and validate phone number.
+ */
+@JNINamespace("autofill")
+public class PhoneNumberUtil {
+    // Avoid instantiation by accident.
+    private PhoneNumberUtil() {}
+
+    /** TextWatcher to watch phone number changes so as to format it dynamically */
+    public static class FormatTextWatcher implements TextWatcher {
+        /** Indicates the change was caused by ourselves. */
+        private boolean mSelfChange = false;

[please use gerrit instead, 2017/03/14 18:17:15]

'= false' is redundant, because 'false' is the default value for booleans in
Java. Also, The resulting binary will be ever so slightly bigger if you specify
the redundant initialization. I don't know why. It's silly. But we have to live
with it.

[gogerald1, 2017/03/14 19:32:33]

Done.

+
+        @Override
+        public void afterTextChanged(Editable s) {
+            if (mSelfChange) return;
+
+            String formattedNumber = format(s.toString());
+            mSelfChange = true;
+            s.replace(0, s.length(), formattedNumber, 0, formattedNumber.length());
+            mSelfChange = false;
+        }
+
+        @Override
+        public void beforeTextChanged(CharSequence s, int start, int count, int after) {}
+
+        @Override
+        public void onTextChanged(CharSequence s, int start, int before, int count) {}
+    }
+
+    /**
+     * Formats the given phone number in INTERNATIONAL format
+     * [i18n::phonenumbers::PhoneNumberUtil::PhoneNumberFormat::INTERNATIONAL], returning the
+     * original number if no formatting can be made. For example, the number of the Google Zürich
+     * office will be formatted as "+41 44 668 1800" in INTERNATIONAL format.
+     *
+     * @param phoneNumber The given phone number.
+     * @return formatted phone number.
+     */
+    public static String format(String phoneNumber) {
+        return nativeFormat(phoneNumber);
+    }
+
+    /**
+     * Checks whether the given phone number matches a valid pattern according to region code. The
+     * region code is from the given phone number if it starts with '+', otherwise application
+     * locale is used to figure out the region code.
+     *
+     * @param phoneNumber The given phone number.
+     * @return True if the given number is valid, otherwise return false.
+     */
+    public static boolean isValidNumber(String phoneNumber) {
+        return nativeIsValidNumber(phoneNumber);
+    }
+
+    private static native String nativeFormat(String phoneNumber);
+    private static native boolean nativeIsValidNumber(String phoneNumber);
+}