[Autofill] Style cleanup in phone number-related code.

chrome/browser/autofill/phone_number_i18n.h

 // Copyright (c) 2011 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.
 
 #ifndef CHROME_BROWSER_AUTOFILL_PHONE_NUMBER_I18N_H_
 #define CHROME_BROWSER_AUTOFILL_PHONE_NUMBER_I18N_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 11 matching lines @@
 namespace autofill_i18n {
 
 // Most of the following functions require |region| to operate. The |region| is
 // a ISO 3166 standard code ("US" for USA, "CZ" for Czech Republic, etc.).
 
 // Normalizes phone number, by changing digits in the extended fonts
 // (such as \xFF1x) into '0'-'9'. Also strips out non-digit characters.
 string16 NormalizePhoneNumber(const string16& value,
                               const std::string& region);
 
-// Parses |value| to extract the components of a phone number.  |number|
-// returns the local number, |city_code| returns the city code, and
-// |country_code| returns country code. For some regions the |city_code| is
-// empty.
-// The parsing is based on current region - |region|.
-// Separator characters are stripped before parsing the digits.
-// Returns true if parsing was successful, false otherwise.
+// Parses the number stored in |value| as a phone number interpreted in the
+// given |region|, and stores the results into the remaining arguments.  The
+// |region| should be a 2-letter country code.  This is an internal function,
+// exposed in the header file so that it can be tested.
 bool ParsePhoneNumber(const string16& value,
                       const std::string& region,
                       string16* country_code,
                       string16* city_code,
                       string16* number) WARN_UNUSED_RESULT;
 
 enum FullPhoneFormat {
   E164,  // Example: +16502345678
   INTERNATIONAL,  // Example: +1 650-234-5678
   NATIONAL,  // Example: (650) 234-5678
   RFC3966  // Example: +1-650-234-5678
 };
 
 // Constructs whole phone number from parts.
 // |city_code| - area code, could be empty.
-// |country_code| - country code, could be empty
+// |country_code| - country code, could be empty.
 // |number| - local number, should not be empty.
 // |region| - current region, the parsing is based on.
 // |phone_format| - whole number constructed in that format,
 // |whole_number| - constructed whole number.
 // Separator characters are stripped before parsing the digits.
 // Returns true if parsing was successful, false otherwise.
 bool ConstructPhoneNumber(const string16& country_code,
                           const string16& city_code,
                           const string16& number,
                           const std::string& region,
                           FullPhoneFormat phone_format,
                           string16* whole_number) WARN_UNUSED_RESULT;
 
+// Returns true if |number_a| and |number_b| parse to the same phone number in
+// the given |region|.
 bool PhoneNumbersMatch(const string16& number_a,
                        const string16& number_b,
                        const std::string& region);
 
 // The cached phone number, does parsing only once, improves performance.
 class PhoneObject {
  public:
   PhoneObject(const string16& number, const std::string& region);
   PhoneObject(const PhoneObject&);
   PhoneObject();
@@ skipping 15 matching lines @@
   string16 country_code_;
   string16 number_;
   mutable string16 whole_number_;  // Set on first request.
   std::string region_;
   scoped_ptr<i18n::phonenumbers::PhoneNumber> i18n_number_;
 };
 
 }  // namespace autofill_i18n
 
 #endif  // CHROME_BROWSER_AUTOFILL_PHONE_NUMBER_I18N_H_