Autofill DOMUI Prefs should work with i18n phone numbers

chrome/browser/ui/webui/options/autofill_options_handler.cc

 // 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.
 
 #include "chrome/browser/ui/webui/options/autofill_options_handler.h"
 
 #include <vector>
 
 #include "base/logging.h"
 #include "base/string16.h"
 #include "base/string_number_conversions.h"
 #include "base/utf_string_conversions.h"
 #include "base/values.h"
 #include "chrome/browser/autofill/autofill_country.h"
 #include "chrome/browser/autofill/autofill_profile.h"
 #include "chrome/browser/autofill/credit_card.h"
+#include "chrome/browser/autofill/phone_number_i18n.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/webui/web_ui_util.h"
 #include "chrome/common/guid.h"
 #include "grit/generated_resources.h"
 #include "grit/webkit_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 
 namespace {
 
 // Converts a credit card type to the appropriate resource ID of the CC icon.
@@ skipping 81 matching lines @@
                   AutofillProfile* profile) {
   std::vector<string16> values(list->GetSize());
   for (size_t i = 0; i < list->GetSize(); ++i) {
     string16 value;
     if (list->GetString(i, &value))
       values[i] = value;
   }
   profile->SetMultiInfo(type, values);
 }
 
+// Pulls the phone number |index|, |list_value|, and |country_code| from the
+// |args| input.
+void ExtractPhoneNumberInformation(const ListValue* args,
+                                   size_t* index,
+                                   ListValue** list_value,

[James Hawkins, 2011/05/06 20:34:11]

What are the contents of |list_value|? I propose that the answer to that
question should be used to name this parameter better.

[dhollowa, 2011/05/06 21:00:19]

Done.

+                                   std::string* country_code) {
+  double number = 0.0;

[James Hawkins, 2011/05/06 20:34:11]

Why particularly using a double?

[dhollowa, 2011/05/06 21:00:19]

All numbers come across from JavaScript as doubles, even integer values.  I've
added comment to clarify.

On 2011/05/06 20:34:11, James Hawkins wrote:

+  if (!args->GetDouble(0, &number)) {
+    NOTREACHED();
+    return;
+  }
+  *index = number;
+
+  if (!args->GetList(1, list_value)) {
+    NOTREACHED();
+    return;
+  }
+
+  if (!args->GetString(2, country_code)) {
+    NOTREACHED();
+    return;
+  }
+}
+
+// Searches the |list| for the value at |index|.  If this value is present
+// in any of the rest of the list, then the item (at |index|) is removed.
+// The comparison of phone number values is done on normalized versions of the
+// phone number values.
+void RemoveDuplicatePhoneNumberAtIndex(size_t index,
+                                       const std::string& country_code,
+                                       ListValue* list) {
+  string16 new_value;
+  list->GetString(index, &new_value);
+
+  bool is_duplicate = false;
+  for (size_t i = 0; i < list->GetSize() && !is_duplicate; ++i) {
+    if (i == index)
+      continue;
+
+    string16 existing_value;
+    list->GetString(i, &existing_value);
+    is_duplicate = autofill_i18n::PhoneNumbersMatch(new_value,
+                                                    existing_value,
+                                                    country_code);
+  }
+
+  if (is_duplicate)
+    list->Remove(index, NULL);
+}
+
+void ValidatePhoneArguments(const ListValue* args, ListValue** list) {
+  size_t index = 0;
+  std::string country_code;
+  ExtractPhoneNumberInformation(args, &index, list, &country_code);
+
+  RemoveDuplicatePhoneNumberAtIndex(index, country_code, *list);
+}
+
 }  // namespace
 
 AutofillOptionsHandler::AutofillOptionsHandler()
     : personal_data_(NULL) {
 }
 
 AutofillOptionsHandler::~AutofillOptionsHandler() {
   if (personal_data_)
     personal_data_->RemoveObserver(this);
 }
@@ skipping 46 matching lines @@
       NewCallback(this, &AutofillOptionsHandler::LoadAddressEditor));
   web_ui_->RegisterMessageCallback(
       "loadCreditCardEditor",
       NewCallback(this, &AutofillOptionsHandler::LoadCreditCardEditor));
   web_ui_->RegisterMessageCallback(
       "setAddress",
       NewCallback(this, &AutofillOptionsHandler::SetAddress));
   web_ui_->RegisterMessageCallback(
       "setCreditCard",
       NewCallback(this, &AutofillOptionsHandler::SetCreditCard));
+  web_ui_->RegisterMessageCallback(
+      "validatePhoneNumbers",
+      NewCallback(this, &AutofillOptionsHandler::ValidatePhoneNumbers));
+  web_ui_->RegisterMessageCallback(
+      "validateFaxNumbers",
+      NewCallback(this, &AutofillOptionsHandler::ValidateFaxNumbers));
 }
 
 /////////////////////////////////////////////////////////////////////////////
 // PersonalDataManager::Observer implementation:
 void  AutofillOptionsHandler::OnPersonalDataLoaded() {
   LoadAutofillData();
 }
 
 void AutofillOptionsHandler::OnPersonalDataChanged() {
   LoadAutofillData();
@@ skipping 251 matching lines @@
   if (args->GetString(4, &value))
     credit_card.SetInfo(CREDIT_CARD_EXP_4_DIGIT_YEAR, value);
 
   if (!guid::IsValidGUID(credit_card.guid())) {
     credit_card.set_guid(guid::GenerateGUID());
     personal_data_->AddCreditCard(credit_card);
   } else {
     personal_data_->UpdateCreditCard(credit_card);
   }
 }
+
+void AutofillOptionsHandler::ValidatePhoneNumbers(const ListValue* args) {
+  if (!personal_data_->IsDataLoaded())
+    return;
+
+  ListValue* list_value = NULL;
+  ValidatePhoneArguments(args, &list_value);
+
+  web_ui_->CallJavascriptFunction(
+    "AutofillEditAddressOverlay.setValidatedPhoneNumbers", *list_value);
+}
+
+void AutofillOptionsHandler::ValidateFaxNumbers(const ListValue* args) {
+  if (!personal_data_->IsDataLoaded())
+    return;
+
+  ListValue* list_value = NULL;
+  ValidatePhoneArguments(args, &list_value);
+
+  web_ui_->CallJavascriptFunction(
+      "AutofillEditAddressOverlay.setValidatedFaxNumbers", *list_value);
+}