Autofill:Autocheckout: Enable looking at all elements for Autocheckout flow (ignoring 3 element lim…

components/autofill/browser/autofill_manager_unittest.cc

 // Copyright (c) 2012 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 <algorithm>
 #include <vector>
 
 #include "base/command_line.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/scoped_vector.h"
@@ skipping 186 matching lines @@
 
     credit_card = new CreditCard;
     autofill_test::SetCreditCardInfo(credit_card, "", "", "", "");
     credit_card->set_guid("00000000-0000-0000-0000-000000000006");
     credit_cards->push_back(credit_card);
   }
 
   DISALLOW_COPY_AND_ASSIGN(TestPersonalDataManager);
 };
 
+// Populates |form| with 3 fields and a field with autocomplete attribute.
+void CreateTestFormWithAutocompleteAttribute(FormData* form) {
+  form->name = ASCIIToUTF16("UserSpecified");
+  form->method = ASCIIToUTF16("POST");
+  form->origin = GURL("http://myform.com/userspecified.html");
+  form->action = GURL("http://myform.com/submit.html");
+  form->user_submitted = true;
+
+  FormFieldData field;
+  autofill_test::CreateTestFormField(
+      "First Name", "firstname", "", "text", &field);
+  form->fields.push_back(field);
+  autofill_test::CreateTestFormField(
+      "Middle Name", "middlename", "", "text", &field);
+  form->fields.push_back(field);
+  autofill_test::CreateTestFormField(
+      "Last Name", "lastname", "", "text", &field);
+  form->fields.push_back(field);
+  field.autocomplete_attribute="cc-type";
+  autofill_test::CreateTestFormField(
+    "cc-type", "cc-type", "", "text", &field);
+  form->fields.push_back(field);
+}
+
+// Populates |form| with data corresponding to a simple shipping options form.
+void CreateTestShippingOptionsFormData(FormData* form) {
+  form->name = ASCIIToUTF16("Shipping Options");
+  form->method = ASCIIToUTF16("POST");
+  form->origin = GURL("http://myform.com/shipping.html");
+  form->action = GURL("http://myform.com/submit.html");
+  form->user_submitted = true;
+
+  FormFieldData field;
+  autofill_test::CreateTestFormField(
+    "Shipping1", "option", "option1", "radio", &field);
+  form->fields.push_back(field);
+  autofill_test::CreateTestFormField(
+    "Shipping2", "option", "option2", "radio", &field);
+  form->fields.push_back(field);
+}
+
 // Populates |form| with data corresponding to a simple address form.
 // Note that this actually appends fields to the form data, which can be useful
 // for building up more complex test forms.
 void CreateTestAddressFormData(FormData* form) {
   form->name = ASCIIToUTF16("MyForm");
   form->method = ASCIIToUTF16("POST");
   form->origin = GURL("http://myform.com/form.html");
   form->action = GURL("http://myform.com/submit.html");
   form->user_submitted = true;
 
@@ skipping 250 matching lines @@
         message_loop_is_running_(false) {
   }
   virtual ~TestAutofillManager() {}
 
   virtual bool IsAutofillEnabled() const OVERRIDE { return autofill_enabled_; }
 
   void set_autofill_enabled(bool autofill_enabled) {
     autofill_enabled_ = autofill_enabled;
   }
 
+  void set_autocheckout_url_prefix(std::string autocheckout_url_prefix) {

[Ilya Sherman, 2013/03/15 23:37:18]

nit: Pass by const-reference.

[Raman Kakilate, 2013/03/18 16:14:32]

Done.

+    autocheckout_url_prefix_ = autocheckout_url_prefix;
+  }
+
+  virtual std::string GetAutocheckoutURLPrefix() const OVERRIDE {
+    return autocheckout_url_prefix_;
+  }
+
   const std::vector<std::pair<WebFormElement::AutocompleteResult, FormData> >&
       request_autocomplete_results() const {
     return request_autocomplete_results_;
   }
 
 
   void set_expected_submitted_field_types(
       const std::vector<FieldTypeSet>& expected_types) {
     expected_submitted_field_types_ = expected_types;
   }
@@ skipping 90 matching lines @@
     std::string credit_card_guid =
         base::StringPrintf("00000000-0000-0000-0000-%012d", credit_card_id);
 
     return PackGUIDs(GUIDPair(credit_card_guid, 0), GUIDPair(std::string(), 0));
   }
 
   void AddSeenForm(FormStructure* form) {
     form_structures()->push_back(form);
   }
 
+  void ClearFormStructures() {
+    form_structures()->clear();
+  }
+
   virtual void ReturnAutocompleteResult(
       WebFormElement::AutocompleteResult result,
       const FormData& form_data) OVERRIDE {
     request_autocomplete_results_.push_back(std::make_pair(result, form_data));
   }
 
  private:
   // Weak reference.
   TestPersonalDataManager* personal_data_;
 
   bool autofill_enabled_;
   std::vector<std::pair<WebFormElement::AutocompleteResult, FormData> >
       request_autocomplete_results_;
 
   bool did_finish_async_form_submit_;
   bool message_loop_is_running_;
 
+  std::string autocheckout_url_prefix_;
   std::string submitted_form_signature_;
   std::vector<FieldTypeSet> expected_submitted_field_types_;
   std::vector<bool> sent_states_;
 
   DISALLOW_COPY_AND_ASSIGN(TestAutofillManager);
 };
 
 }  // namespace
 
 class AutofillManagerTest : public ChromeRenderViewHostTestHarness {
@@ skipping 221 matching lines @@
     ASCIIToUTF16("3734 Elvis Presley Blvd."),
     ASCIIToUTF16("123 Apple St.")
   };
   string16 expected_icons[] = {string16(), string16()};
   int expected_unique_ids[] = {1, 2};
   ExpectSuggestions(page_id, values, labels, icons, unique_ids,
                     kDefaultPageID, arraysize(expected_values), expected_values,
                     expected_labels, expected_icons, expected_unique_ids);
 }
 
+// Test that in the case of Autocheckout, forms seen are in order supplied.
+TEST_F(AutofillManagerTest, AutocheckoutFormsSeen) {
+  FormData shipping_options;
+  CreateTestShippingOptionsFormData(&shipping_options);
+  FormData user_supplied;
+  CreateTestFormWithAutocompleteAttribute(&user_supplied);
+  FormData address;
+  CreateTestAddressFormData(&address);
+
+  // Push user_supplied before address and observe order changing when
+  // Autocheckout is not enabled..
+  std::vector<FormData> forms;
+  forms.push_back(shipping_options);
+  forms.push_back(user_supplied);
+  forms.push_back(address);
+
+  // Test without enabling Autocheckout. FormStructure should only contain
+  // form1. Shipping Options form will not qualify as parsable form.
+  FormsSeen(forms);
+  std::vector<FormStructure*> form_structures;
+  form_structures = autofill_manager_->GetFormStructures();
+  ASSERT_EQ(2U, form_structures.size());
+  ASSERT_EQ("/form.html",

[Ilya Sherman, 2013/03/15 23:37:18]

nit: EXPECT_EQ

[Raman Kakilate, 2013/03/18 16:14:32]

Done.

+            form_structures[0]->source_url().path());

[Ilya Sherman, 2013/03/15 23:37:18]

nit: Looks like this fits on the previous line.  (I think this applies for
several others below as well.)

[Raman Kakilate, 2013/03/18 16:14:32]

Done.

+  ASSERT_EQ("/userspecified.html",

[Ilya Sherman, 2013/03/15 23:37:18]

nit: EXPECT_EQ

[Raman Kakilate, 2013/03/18 16:14:32]

Done.

+            form_structures[1]->source_url().path());
+  autofill_manager_->ClearFormStructures();
+
+  // Test after enabling Autocheckout. Order should be shipping_options,
+  // userspecified and then address form.
+  autofill_manager_->set_autocheckout_url_prefix("yes-autocheckout");
+  FormsSeen(forms);
+  form_structures = autofill_manager_->GetFormStructures();
+  ASSERT_EQ(3U, form_structures.size());
+  ASSERT_EQ("/shipping.html",
+            form_structures[0]->source_url().path());
+  ASSERT_EQ("/userspecified.html",
+            form_structures[1]->source_url().path());
+  ASSERT_EQ("/form.html",
+            form_structures[2]->source_url().path());
+}
+
 // Test that we return only matching address profile suggestions when the
 // selected form field has been partially filled out.
 TEST_F(AutofillManagerTest, GetProfileSuggestionsMatchCharacter) {
   // Set up our form data.
   FormData form;
   CreateTestAddressFormData(&form);
   std::vector<FormData> forms(1, form);
   FormsSeen(forms);
 
   FormFieldData field;
@@ skipping 2364 matching lines @@
 
   FormData form;
   CreateTestAddressFormData(&form);
   std::vector<FormData> forms(1, form);
   FormsSeen(forms);
   const FormFieldData& field = form.fields[0];
   GetAutofillSuggestions(form, field);  // should call the delegate's OnQuery()
 
   autofill_manager_->SetExternalDelegate(NULL);
 }