Ignore ajax on specified pages.

components/autofill/browser/autofill_xml_parser.cc

Index: components/autofill/browser/autofill_xml_parser.cc
diff --git a/components/autofill/browser/autofill_xml_parser.cc b/components/autofill/browser/autofill_xml_parser.cc
index 01d28105c9ee456d40c88e03ebc37b28b3c40d42..eff277040ea8bb6f65f254a082e4381783b92fb8 100644
--- a/components/autofill/browser/autofill_xml_parser.cc
+++ b/components/autofill/browser/autofill_xml_parser.cc
@@ -119,6 +119,8 @@ void AutofillQueryXmlParser::StartElement(buzz::XmlParseContext* context,
         page_meta_data_->current_page_number = GetIntValue(context, *attrs);
       else if (attribute_name.compare("total_pages") == 0)
         page_meta_data_->total_pages = GetIntValue(context, *attrs);
+      else if (attribute_name.compare("ignore_ajax") == 0)
+        page_meta_data_->ignore_ajax = strcmp(*attrs, "true") == 0;

[Ilya Sherman, 2013/06/11 01:11:20]

Hmm, using strcmp here makes me a little nervous.  Please double-check with the
security team whether this is ok.

[Dane Wallinga, 2013/06/11 19:03:32]

Even though this file is already replete with strcmps?

[Ilya Sherman, 2013/06/11 23:14:32]

By replete with, I guess you mean "has two strcmps"?  I'd still prefer to have
the security team double-check this, but I'm less worried about it now that I
see the two pre-existing uses.

       ++attrs;
     }
   } else if (element.compare("page_advance_button") == 0) {