Support autocompletion for HTMl5 tags:"email", "month" and "tel".

chrome/browser/autofill/autofill_manager.cc

Index: chrome/browser/autofill/autofill_manager.cc
diff --git a/chrome/browser/autofill/autofill_manager.cc b/chrome/browser/autofill/autofill_manager.cc
index 162375268776d8418d802d495793a96847dc7b5f..874ca538629fc00fdaf000edd91f1e149958928e 100644
--- a/chrome/browser/autofill/autofill_manager.cc
+++ b/chrome/browser/autofill/autofill_manager.cc
@@ -688,6 +688,12 @@ void AutoFillManager::FillCreditCardFormField(const CreditCard* credit_card,
 
   if (field->form_control_type() == ASCIIToUTF16("select-one"))
     autofill::FillSelectControl(credit_card, type, field);
+  else if (field->form_control_type() == ASCIIToUTF16("month"))
+    // THML5 input="month" consisnts of year-month.

[James Hawkins, 2011/01/03 19:50:00]

Spelling errors in this comment.

+    field->set_value(
+        credit_card->GetFieldText(AutoFillType(CREDIT_CARD_EXP_4_DIGIT_YEAR)) +

[James Hawkins, 2011/01/03 19:50:00]

Please extract a variable from these lines and send that to set_value.  Note
that the style guide says each continuation of one line must be indented four
spaces from the previous line.

Personally, I'd do:

string16 month = credit_card->GetFieldText(...);
month += ASCIIToUTF16("-");
month += credit_card->GetFieldText(...);

+        string16(ASCIIToUTF16("-")) +

[James Hawkins, 2011/01/03 19:50:00]

No need to wrap with string16().

+        credit_card->GetFieldText(AutoFillType(CREDIT_CARD_EXP_MONTH)));
   else
     field->set_value(credit_card->GetFieldText(type));
 }