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

Unified Diff: chrome/browser/autofill/form_structure_unittest.cc

Issue 11953100: Add url prefix to AutofillQuery requests when autocheckout enabled, and set accepts="a" (autochecko… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: sync to head Created 7 years, 10 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
« no previous file with comments | « chrome/browser/autofill/form_structure.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/autofill/form_structure_unittest.cc
diff --git a/chrome/browser/autofill/form_structure_unittest.cc b/chrome/browser/autofill/form_structure_unittest.cc
index 44ffdc949e5f260ab7496b1c5027a11eaf76a12f..d312f8e00c767703f8634f00c0cc6038ebde6c6b 100644
--- a/chrome/browser/autofill/form_structure_unittest.cc
+++ b/chrome/browser/autofill/form_structure_unittest.cc
@@ -1532,11 +1532,10 @@ TEST(FormStructureTest, EncodeQueryRequest) {
EXPECT_EQ(0U, encoded_signatures.size());
EXPECT_EQ("", encoded_xml);
- // Check the behaviour with kEnableExperimentalFormFilling switch on.
- // Add the previous form but with flag set.
+ // Check the behaviour with autocheckout enabled.
ScopedVector<FormStructure> checkable_forms;
checkable_forms.push_back(
- new FormStructure(form, "https://www.abc.com/checkout"));
+ new FormStructure(form, "https://www.sample1.com/query/path"));
ASSERT_TRUE(FormStructure::EncodeQueryRequest(checkable_forms.get(),
&encoded_signatures,
@@ -1544,8 +1543,10 @@ TEST(FormStructureTest, EncodeQueryRequest) {
const char * const kSignature3 = "7747357776717901584";
const char * const kResponse3 =
"<?xml version=\"1.0\" encoding=\"UTF-8\"?><autofillquery "
- "clientversion=\"6.1.1715.1442/en (GGLL)\" accepts=\"e\">"
- "<form signature=\"7747357776717901584\"><field signature=\"412125936\"/>"
+ "clientversion=\"6.1.1715.1442/en (GGLL)\" accepts=\"a,e\" "
+ "urlprefixsignature=\"7648393911063090788\">"
+ "<form signature=\"7747357776717901584\">"
+ "<field signature=\"412125936\"/>"
"<field signature=\"1917667676\"/><field signature=\"2226358947\"/><field"
" signature=\"747221617\"/><field signature=\"4108155786\"/><field "
"signature=\"3410250678\"/><field signature=\"509334676\"/><field "
« no previous file with comments | « chrome/browser/autofill/form_structure.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698