Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(462)

Unified Diff: components/autofill/browser/autofill_xml_parser.cc

Issue 16611003: Ignore ajax on specified pages. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: remove some debug statements I missed before Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..0db5fed057add235e41d75995877dfdf50f31ac6 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);
ahutter 2013/06/07 17:37:47 these might be extraneous parens according to the
Dane Wallinga 2013/06/07 21:06:47 Done.
++attrs;
}
} else if (element.compare("page_advance_button") == 0) {

Powered by Google App Engine
This is Rietveld 408576698