Add support for autofilling radio buttons and checkboxes.

chrome/browser/autofill/autofill_xml_parser.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/autofill/autofill_xml_parser.h"
 
 #include <stdlib.h>
 #include <string.h>
 
 #include "base/logging.h"
+#include "chrome/browser/autofill/autofill_server_field_info.h"
 #include "third_party/libjingle/source/talk/xmllite/qname.h"
 
 AutofillXmlParser::AutofillXmlParser()
     : succeeded_(true) {
 }
 
 void AutofillXmlParser::CharacterData(
     buzz::XmlParseContext* context, const char* text, int len) {
 }
 
 void AutofillXmlParser::EndElement(buzz::XmlParseContext* context,
                                    const char* name) {
 }
 
 void AutofillXmlParser::Error(buzz::XmlParseContext* context,
                               XML_Error error_code) {
   succeeded_ = false;
 }
 
 AutofillQueryXmlParser::AutofillQueryXmlParser(
-    std::vector<AutofillFieldType>* field_types,
+    std::vector<AutofillServerFieldInfo>* field_infos,
     UploadRequired* upload_required,
     std::string* experiment_id)
-    : field_types_(field_types),
+    : field_infos_(field_infos),
       upload_required_(upload_required),
       experiment_id_(experiment_id) {
   DCHECK(upload_required_);
   DCHECK(experiment_id_);
 }
 
 void AutofillQueryXmlParser::StartElement(buzz::XmlParseContext* context,
                                           const char* name,
                                           const char** attrs) {
   buzz::QName qname = context->ResolveQName(name, false);
   const std::string& element = qname.LocalPart();
   if (element.compare("autofillqueryresponse") == 0) {
     // We check for the upload required attribute below, but if it's not
     // present, we use the default upload rates. Likewise, by default we assume
     // an empty experiment id.
     *upload_required_ = USE_UPLOAD_RATES;
     *experiment_id_ = std::string();
 
     // |attrs| is a NULL-terminated list of (attribute, value) pairs.

[Ilya Sherman, 2012/12/07 01:47:22]

nit: Please add a similar comment above line 81 as well.

[Raman Kakilate, 2012/12/10 18:36:45]

Done.

     while (*attrs) {
-      buzz::QName attribute_qname = context->ResolveQName(attrs[0], true);
+      buzz::QName attribute_qname = context->ResolveQName(*attrs, true);
+      attrs++;

[Ilya Sherman, 2012/12/07 01:47:22]

nit: ++attrs

[Raman Kakilate, 2012/12/10 18:36:45]

Done.

       const std::string& attribute_name = attribute_qname.LocalPart();
       if (attribute_name.compare("uploadrequired") == 0) {
-        if (strcmp(attrs[1], "true") == 0)
+        if (strcmp(*attrs, "true") == 0)
           *upload_required_ = UPLOAD_REQUIRED;
-        else if (strcmp(attrs[1], "false") == 0)
+        else if (strcmp(*attrs, "false") == 0)
           *upload_required_ = UPLOAD_NOT_REQUIRED;
       } else if (attribute_name.compare("experimentid") == 0) {
-        *experiment_id_ = attrs[1];
+        *experiment_id_ = *attrs;
       }
-
-      // Advance to the next (attribute, value) pair.
-      attrs += 2;
+      attrs ++;

[Ilya Sherman, 2012/12/07 01:47:22]

nit: ++attrs

[Raman Kakilate, 2012/12/10 18:36:45]

Done.

     }
   } else if (element.compare("field") == 0) {
     if (!attrs[0]) {

[Ilya Sherman, 2012/12/07 01:47:22]

nit: *attrs

[Raman Kakilate, 2012/12/10 18:36:45]

Done.

       // Missing the "autofilltype" attribute, abort.
       context->RaiseError(XML_ERROR_ABORTED);
       return;
     }
 
     // Determine the field type from the attribute value.  There should be one
     // attribute (autofilltype) with an integer value.
-    AutofillFieldType field_type = UNKNOWN_TYPE;
-    buzz::QName attribute_qname = context->ResolveQName(attrs[0], true);
-    const std::string& attribute_name = attribute_qname.LocalPart();
+    AutofillServerFieldInfo field_info;
+    field_info.field_type = UNKNOWN_TYPE;
 
-    if (attribute_name.compare("autofilltype") == 0) {
-      int value = GetIntValue(context, attrs[1]);
-      field_type = static_cast<AutofillFieldType>(value);
-      if (field_type < 0 || field_type > MAX_VALID_FIELD_TYPE) {
-        field_type = NO_SERVER_DATA;
+    while (*attrs) {
+      buzz::QName attribute_qname = context->ResolveQName(*attrs, true);
+      attrs++;

[Ilya Sherman, 2012/12/07 01:47:22]

nit: ++attrs

[Raman Kakilate, 2012/12/10 18:36:45]

Done.

+      const std::string& attribute_name = attribute_qname.LocalPart();
+      if (attribute_name.compare("autofilltype") == 0) {

[Ilya Sherman, 2012/12/07 01:47:22]

nit: Can this be written as """if (attribute_name == "autofilltype")"""?

[Raman Kakilate, 2012/12/10 18:36:45]

There are more than few compare calls in this file. I can do it in a future CL.
For now, I will be consistent with rest of the file.

+        int value = GetIntValue(context, *attrs);
+        field_info.field_type = static_cast<AutofillFieldType>(value);
+        if (field_info.field_type < 0 ||
+            field_info.field_type > MAX_VALID_FIELD_TYPE)
+          field_info.field_type = NO_SERVER_DATA;
+      } else if (field_info.field_type == FIELD_WITH_DEFAULT_VALUE &&
+                 attribute_name.compare("defaultvalue") == 0) {
+        field_info.default_value.assign(*attrs);

[Ilya Sherman, 2012/12/07 01:47:22]

nit: Can this be written as "field_info.default_value = *attrs"?

[Raman Kakilate, 2012/12/10 18:36:45]

Done.

       }
+      attrs ++;

[Ilya Sherman, 2012/12/07 01:47:22]

nit: ++attrs

[Raman Kakilate, 2012/12/10 18:36:45]

Done.

     }
 
-    // Record this field type.
-    field_types_->push_back(field_type);
+    // Record this field type, default value pair.
+    field_infos_->push_back(field_info);
   }
 }
 
 int AutofillQueryXmlParser::GetIntValue(buzz::XmlParseContext* context,
                                         const char* attribute) {
   char* attr_end = NULL;
   int value = strtol(attribute, &attr_end, 10);
   if (attr_end != NULL && attr_end == attribute) {
     context->RaiseError(XML_ERROR_SYNTAX);
     return 0;
@@ skipping 33 matching lines @@
 double AutofillUploadXmlParser::GetDoubleValue(buzz::XmlParseContext* context,
                                                const char* attribute) {
   char* attr_end = NULL;
   double value = strtod(attribute, &attr_end);
   if (attr_end != NULL && attr_end == attribute) {
     context->RaiseError(XML_ERROR_SYNTAX);
     return 0.0;
   }
   return value;
 }