Adjust autofill phone number format.

components/autofill/core/browser/phone_number_i18n.cc

 // Copyright 2013 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.
 
 #include "components/autofill/core/browser/phone_number_i18n.h"
 
 #include "base/basictypes.h"
 #include "base/logging.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
@@ skipping 45 matching lines @@
                            const base::string16& country_code,
                            base::string16* formatted_number,
                            base::string16* normalized_number) {
   PhoneNumberUtil::PhoneNumberFormat format =
       country_code.empty() ?
       PhoneNumberUtil::NATIONAL :
       PhoneNumberUtil::INTERNATIONAL;
 
   PhoneNumberUtil* phone_util = PhoneNumberUtil::GetInstance();
   std::string processed_number;
-  phone_util->Format(number, format, &processed_number);
+
+  std::string region_code;
+  phone_util->GetRegionCodeForNumber(number, &region_code);
+
+  // Formats a phone number in the specified format using default rules.
+  // This applies to the following cases:
+  //   The phone number contains alphabetic letter(s).
+  //   The phone number with a leading IDD.

[Ilya Sherman, 2014/07/02 22:23:54]

nit: Please spell out the acronym "IDD", and clarify what it means to have "a
leading international direct dialing", perhaps by appending "code" to the end of
the sentence.

+  //   The phone number without a leading '+' sign.
+  if (phone_util->IsAlphaNumber(number.raw_input()) ||
+      (number.has_country_code_source() &&
+       (number.country_code_source() == PhoneNumber::FROM_NUMBER_WITH_IDD ||
+        number.country_code_source() ==
+            PhoneNumber::FROM_NUMBER_WITHOUT_PLUS_SIGN))) {
+    phone_util->Format(number, format, &processed_number);
+  } else {
+    phone_util->FormatInOriginalFormat (number, region_code, &processed_number);

[Ilya Sherman, 2014/07/02 22:23:54]

nit: Please omit the space before the paren.

+  }
 
   if (formatted_number)
     *formatted_number = base::UTF8ToUTF16(processed_number);

[Ilya Sherman, 2014/07/02 22:23:54]

Where is the |formatted_number| used?  Should the '+' be dropped for US numbers
here as well?

 
   if (normalized_number) {
-    phone_util->NormalizeDigitsOnly(&processed_number);
+    // Keep the leading '+' while normalising numbers, excluding US numbers.
+    if (processed_number[0] == '+' && region_code.compare("US")) {

[Ilya Sherman, 2014/07/02 22:23:54]

nit: Please write the second condition as |region_code != "US"|.

+      processed_number = processed_number.substr(1, processed_number.length());
+      phone_util->NormalizeDigitsOnly(&processed_number);
+      processed_number.insert(processed_number.begin(), '+');
+    } else {
+      phone_util->NormalizeDigitsOnly(&processed_number);
+    }
     *normalized_number = base::UTF8ToUTF16(processed_number);
   }
 }
 
 }  // namespace
 
 namespace i18n {
 
 // Parses the number stored in |value| as it should be interpreted in the given
 // |default_region|, and stores the results into the remaining arguments.
@@ skipping 10 matching lines @@
   number->clear();
   *i18n_number = PhoneNumber();
 
   std::string number_text(base::UTF16ToUTF8(value));
 
   // Parse phone number based on the region.
   PhoneNumberUtil* phone_util = PhoneNumberUtil::GetInstance();
 
   // The |default_region| should already be sanitized.
   DCHECK_EQ(2U, default_region.size());
-  if (phone_util->Parse(number_text, default_region, i18n_number) !=
+  if (phone_util->ParseAndKeepRawInput(number_text,
+                                       default_region,
+                                       i18n_number) !=
           PhoneNumberUtil::NO_PARSING_ERROR) {
     return false;
   }
 
   if (!IsValidPhoneNumber(*i18n_number))
     return false;
 
   std::string national_significant_number;
   phone_util->GetNationalSignificantNumber(*i18n_number,
                                            &national_significant_number);
@@ skipping 186 matching lines @@
   number_ = other.number_;
 
   formatted_number_ = other.formatted_number_;
   whole_number_ = other.whole_number_;
 
   return *this;
 }
 
 }  // namespace i18n
 }  // namespace autofill