use user chosen country code to format and validate phone number for addresses.

chrome/android/java/src/org/chromium/chrome/browser/payments/AddressEditor.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 android.app.ProgressDialog;
 import android.os.Handler;
 import android.util.Pair;
 
@@ skipping 105 matching lines @@
              * If the selected country on the country dropdown list is changed,
              * the first element of eventData is the recently selected dropdown key,
              * the second element is the callback to invoke for when the dropdown
              * change has been processed.
              */
             @Override
             public void onResult(Pair<String, Runnable> eventData) {
                 mEditor.removeAllFields();
                 showProgressDialog();
                 mRecentlySelectedCountry = eventData.first;
+                mEditorDialog.updateCountryOfPhoneFormatter(mRecentlySelectedCountry);
                 mCountryChangeCallback = eventData.second;
                 loadAdminAreasForCountry(mRecentlySelectedCountry);
             }
         });
 
         // Country dropdown is cached, so the selected item needs to be updated for the new profile
         // that's being edited. This will not fire the dropdown callback.
         mCountryField.setValue(AutofillAddress.getCountryCode(mProfile));
 
         // There's a finite number of fields for address editing. Changing the country will re-order
@@ skipping 252 matching lines @@
         }
         // Phone number must be the last field.
         mEditor.addField(mPhoneField);
     }
 
     private EditorFieldValidator getPhoneValidator() {
         if (mPhoneValidator == null) {
             mPhoneValidator = new EditorFieldValidator() {
                 @Override
                 public boolean isValid(@Nullable CharSequence value) {
-                    return value != null && PhoneNumberUtil.isValidNumber(value.toString());
+                    return value != null
+                            && PhoneNumberUtil.isValidNumber(
+                                       value.toString(), mRecentlySelectedCountry);

[gogerald1, 2017/06/08 16:20:43]

mRecentlySelectedCountry is null if the user doesn't change the default selected
country,

https://cs.chromium.org/chromium/src/chrome/android/java/src/org/chromium/chr...

[wuandy1, 2017/06/08 20:34:58]

Which is handled by isValidNumber..do you mean a @Nullable here?

[gogerald1, 2017/06/09 13:33:46]

NO, the selected country is set according to profile when creating it, which is
not the same as mRecentlySelectedCountry.
https://cs.chromium.org/chromium/src/chrome/android/java/src/org/chromium/chr...

[wuandy1, 2017/06/12 14:15:25]

ok i see. what is your suggestion then?

[gogerald1, 2017/06/12 17:02:11]

What about using 'mCountryField.getValue().toString()' instead of
'mRecentlySelectedCountry'?

Please also update the exact behavior after this CL in bug and cc'ed zkoch@ and
rouslan@ to confirm.

[wuandy1, 2017/06/13 19:47:25]

Done.

                 }
 
                 @Override
                 public boolean isLengthMaximum(@Nullable CharSequence value) {
                     return false;
                 }
             };
         }
         return mPhoneValidator;
     }
 }