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

chrome/browser/autofill/credit_card_field.cc

 // Copyright (c) 2010 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/autofill/credit_card_field.h"
 
 #include "base/scoped_ptr.h"
 #include "base/string16.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/autofill/autofill_field.h"
@@ skipping 97 matching lines @@
 
     if (is_ecml)
       pattern = GetEcmlPattern(kEcmlCardNumber);
     else
       pattern = ASCIIToUTF16("number|card #|card no.|card_number|card number");
 
     if (credit_card_field->number_ == NULL && ParseText(&q, pattern,
         &credit_card_field->number_))
       continue;
 
-    // "Expiration date" is the most common label here, but some pages have
-    // "Expires", "exp. date" or "exp. month" and "exp. year".  We also look for
-    // the field names ccmonth and ccyear, which appear on at least 4 of our
-    // test pages.
-    //
-    // -> On at least one page (The China Shop2.html) we find only the labels
-    // "month" and "year".  So for now we match these words directly; we'll
-    // see if this turns out to be too general.
-    //
-    // Toolbar Bug 51451: indeed, simply matching "month" is too general for
-    //   https://rps.fidelity.com/ftgw/rps/RtlCust/CreatePIN/Init.
-    // Instead, we match only words beginning with "month".
-    if (is_ecml)
-      pattern = GetEcmlPattern(kEcmlCardExpireMonth);
-    else
-      pattern = ASCIIToUTF16("expir|exp.*month|exp date|ccmonth");
+    if ((*q) && LowerCaseEqualsASCII((*q)->form_control_type(), "month")) {

[dhollowa, 2011/01/07 03:47:15]

This is looking good.

However, I tried it against a form with input element:

  <label for="random_id">HTML5 M-o-n-t-h:</label>
  <input type="month" id="random_id"><br> 

And it wasn't matching.  I would have thought it should...  See:

  http://www/~dhollowa/autofill/html5_65654b.html

For the full example.

[honten.org, 2011/01/07 04:05:59]

I can't see your full example. Did you put your example in Google office?
Sorry, I'm not Googler...

Could you attach it?

On 2011/01/07 03:47:15, dhollowa wrote:

+      credit_card_field->expiration_month_ = *q++;
+    } else {
+      // "Expiration date" is the most common label here, but some pages have
+      // "Expires", "exp. date" or "exp. month" and "exp. year".  We also look
+      // for the field names ccmonth and ccyear, which appear on at least 4 of
+      // our test pages.
+      //
+      // -> On at least one page (The China Shop2.html) we find only the labels
+      // "month" and "year".  So for now we match these words directly; we'll
+      // see if this turns out to be too general.
+      //
+      // Toolbar Bug 51451: indeed, simply matching "month" is too general for
+      //   https://rps.fidelity.com/ftgw/rps/RtlCust/CreatePIN/Init.
+      // Instead, we match only words beginning with "month".
+      if (is_ecml)
+        pattern = GetEcmlPattern(kEcmlCardExpireMonth);
+      else
+        pattern = ASCIIToUTF16("expir|exp.*month|exp date|ccmonth");
 
-    if ((!credit_card_field->expiration_month_ ||
-        credit_card_field->expiration_month_->IsEmpty()) &&
+      if ((!credit_card_field->expiration_month_ ||
+          credit_card_field->expiration_month_->IsEmpty()) &&
         ParseText(&q, pattern, &credit_card_field->expiration_month_)) {
-      if (is_ecml)
-        pattern = GetEcmlPattern(kEcmlCardExpireYear);
-      else
-        pattern = ASCIIToUTF16("|exp|^/|ccyear|year");
 
-      if (!ParseText(&q, pattern, &credit_card_field->expiration_year_))
-        return NULL;
+        // If type="month", year is included into the expiration_month_.

[dhollowa, 2011/01/07 03:47:15]

This comment can be removed now.

+        // So we don't need another year input field.
+        if (is_ecml)
+          pattern = GetEcmlPattern(kEcmlCardExpireYear);
+        else
+          pattern = ASCIIToUTF16("|exp|^/|ccyear|year");
 
-      continue;
+        if (!ParseText(&q, pattern, &credit_card_field->expiration_year_)) {
+          return NULL;
+        }

[Ilya Sherman, 2011/01/07 07:28:28]

nit: This is a one-line if-stmt, so no need for braces.

+        continue;
+      }
     }
 
     if (ParseText(&q, GetEcmlPattern(kEcmlCardExpireDay)))
       continue;
 
     // Some pages (e.g. ExpediaBilling.html) have a "card description"
     // field; we parse this field but ignore it.
     // We also ignore any other fields within a credit card block that
     // start with "card", under the assumption that they are related to
     // the credit card section being processed but are uninteresting to us.
@@ skipping 13 matching lines @@
 
   // On some pages, the user selects a card type using radio buttons
   // (e.g. test page Apple Store Billing.html).  We can't handle that yet,
   // so we treat the card type as optional for now.
   // The existence of a number or cvc in combination with expiration date is
   // a strong enough signal that this is a credit card.  It is possible that
   // the number and name were parsed in a separate part of the form.  So if
   // the cvc and date were found independently they are returned.
   if ((credit_card_field->number_ || credit_card_field->verification_) &&
       credit_card_field->expiration_month_ &&
-      credit_card_field->expiration_year_) {
+      (credit_card_field->expiration_year_ ||
+      (LowerCaseEqualsASCII(
+          credit_card_field->expiration_month_->form_control_type(),
+          "month")))) {
       *iter = q;
       return credit_card_field.release();
   }
 
   return NULL;
 }
 
 CreditCardField::CreditCardField()
     : cardholder_(NULL),
       cardholder_last_(NULL),
       type_(NULL),
       number_(NULL),
       verification_(NULL),
       expiration_month_(NULL),
       expiration_year_(NULL) {
 }