Adds wallet::Address::CreateAddress() for when IDs are optional (for billing addresses).

chrome/browser/autofill/wallet/full_wallet_unittest.cc

 // Copyright (c) 2012 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 "base/json/json_reader.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/utf_string_conversions.h"
 #include "base/values.h"
 #include "chrome/browser/autofill/wallet/full_wallet.h"
 #include "chrome/browser/autofill/wallet/required_action.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace {
 
 const char kFullWalletValidResponse[] =
     "{"
     "  \"expiration_month\":12,"
     "  \"expiration_year\":2012,"
     "  \"iin\":\"iin\","
     "  \"rest\":\"rest\","
     "  \"billing_address\":"
     "  {"
-    "    \"id\":\"id\","
     "    \"phone_number\":\"phone_number\","
     "    \"postal_address\":"
     "    {"
     "      \"recipient_name\":\"recipient_name\","
     "      \"address_line\":"
     "      ["
     "        \"address_line_1\","
     "        \"address_line_2\""
     "      ],"
     "      \"locality_name\":\"locality_name\","
@@ skipping 262 matching lines @@
     "}";
 
 const char kFullWalletMalformedBillingAddress[] =
     "{"
     "  \"expiration_month\":12,"
     "  \"expiration_year\":2012,"
     "  \"iin\":\"iin\","
     "  \"rest\":\"rest\","
     "  \"billing_address\":"
     "  {"
-    "    \"id\":\"id\","
     "    \"phone_number\":\"phone_number\","
     "    \"postal_address\":"
     "    {"
     "      \"recipient_name\":\"recipient_name\","
     "      \"address_line\":"
     "      ["
     "        \"address_line_1\","
     "        \"address_line_2\""
     "      ],"
     "      \"locality_name\":\"locality_name\","
@@ skipping 102 matching lines @@
   EXPECT_NE(full_wallet, different_required_actions);
 }
 
 TEST_F(FullWalletTest, CreateFullWalletWithInvalidRequiredActions) {
   SetUpDictionary(kFullWalletWithInvalidRequiredActions);
   EXPECT_EQ(NULL, FullWallet::CreateFullWallet(*dict).get());
 }
 
 TEST_F(FullWalletTest, CreateFullWallet) {
   SetUpDictionary(kFullWalletValidResponse);
+  // NOTE: FullWallet billing address doesn't require an ID.
   scoped_ptr<Address> billing_address(new Address(
       "country_name_code",
       ASCIIToUTF16("recipient_name"),
       ASCIIToUTF16("address_line_1"),
       ASCIIToUTF16("address_line_2"),
       ASCIIToUTF16("locality_name"),
       ASCIIToUTF16("administrative_area_name"),
       ASCIIToUTF16("postal_code_number"),
       ASCIIToUTF16("phone_number"),
-      "id"));
+      ""));
   scoped_ptr<Address> shipping_address(new Address(
       "ship_country_name_code",
       ASCIIToUTF16("ship_recipient_name"),
       ASCIIToUTF16("ship_address_line_1"),
       ASCIIToUTF16("ship_address_line_2"),
       ASCIIToUTF16("ship_locality_name"),
       ASCIIToUTF16("ship_admin_area_name"),
       ASCIIToUTF16("ship_postal_code_number"),
       ASCIIToUTF16("ship_phone_number"),
       "ship_id"));
   std::vector<RequiredAction> required_actions;
   FullWallet full_wallet(12,
                          2012,
                          "iin",
                          "rest",
                          billing_address.Pass(),
                          shipping_address.Pass(),
                          required_actions);
   EXPECT_EQ(full_wallet, *FullWallet::CreateFullWallet(*dict));
 }
 
 // TODO(ahutter): Add tests for GetPan and GetCvn.
 
 }  // namespace wallet
 }  // namespace autofill