[Autofill] Autofill fails to fill credit card number when split across fields.

components/autofill/core/browser/personal_data_manager_unittest.cc

Index: components/autofill/core/browser/personal_data_manager_unittest.cc
diff --git a/components/autofill/core/browser/personal_data_manager_unittest.cc b/components/autofill/core/browser/personal_data_manager_unittest.cc
index 9568cb52453d271e94ca689015db6d00124469f5..ed2430da8880c0b688d485ca99cb40cd03ab2774 100644
--- a/components/autofill/core/browser/personal_data_manager_unittest.cc
+++ b/components/autofill/core/browser/personal_data_manager_unittest.cc
@@ -2269,7 +2269,7 @@ TEST_F(PersonalDataManagerTest, GetNonEmptyTypes) {
   base::MessageLoop::current()->Run();
 
   personal_data_->GetNonEmptyTypes(&non_empty_types);
-  EXPECT_EQ(27U, non_empty_types.size());
+  EXPECT_EQ(30U, non_empty_types.size());
   EXPECT_TRUE(non_empty_types.count(NAME_FIRST));
   EXPECT_TRUE(non_empty_types.count(NAME_MIDDLE));
   EXPECT_TRUE(non_empty_types.count(NAME_MIDDLE_INITIAL));
@@ -2291,6 +2291,10 @@ TEST_F(PersonalDataManagerTest, GetNonEmptyTypes) {
   EXPECT_TRUE(non_empty_types.count(PHONE_HOME_WHOLE_NUMBER));
   EXPECT_TRUE(non_empty_types.count(CREDIT_CARD_NAME));
   EXPECT_TRUE(non_empty_types.count(CREDIT_CARD_NUMBER));
+  EXPECT_TRUE(non_empty_types.count(CREDIT_CARD_NUMBER_PART1));
+  EXPECT_TRUE(non_empty_types.count(CREDIT_CARD_NUMBER_PART2));
+  EXPECT_TRUE(non_empty_types.count(CREDIT_CARD_NUMBER_PART3));
+  EXPECT_FALSE(non_empty_types.count(CREDIT_CARD_NUMBER_PART4));

[Ilya Sherman, 2014/07/25 03:37:49]

Hrm.  It might be better to update the card used in this test to have 16 digits.

[Pritam Nikam, 2014/07/26 11:29:51]

Done.

   EXPECT_TRUE(non_empty_types.count(CREDIT_CARD_TYPE));
   EXPECT_TRUE(non_empty_types.count(CREDIT_CARD_EXP_MONTH));
   EXPECT_TRUE(non_empty_types.count(CREDIT_CARD_EXP_2_DIGIT_YEAR));