[Autofill] Offer credit card suggestions for some form action.

components/autofill/core/browser/autofill_assistant_unittest.cc

 // Copyright 2016 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 "components/autofill/core/browser/autofill_assistant.h"
 
 #include <memory>
 
 #include "base/callback.h"
 #include "base/feature_list.h"
@@ skipping 127 matching lines @@
 
   std::vector<std::unique_ptr<FormStructure>> form_structures;
   EXPECT_FALSE(autofill_assistant_.CanShowCreditCardAssist(form_structures));
 
   // With valid input, the function extracts the credit card form properly.
   form_structures.push_back(std::move(form_structure));
   EXPECT_TRUE(autofill_assistant_.CanShowCreditCardAssist(form_structures));
 }
 
 // Tests that with the feature enabled and proper input,
-// CanShowCreditCardAssist() behaves as expected for secure vs insecure
-// contexts.
+// CanShowCreditCardAssist() behaves as expected for secure contexts.
+TEST_F(AutofillAssistantTest, CanShowCreditCardAssist_FeatureOn_Secure) {
+  EnableAutofillCreditCardAssist();
+
+  // Can be shown if the context is secure.
+  FormData form = CreateValidCreditCardFormData();
+  std::unique_ptr<FormStructure> form_structure(new FormStructure(form));
+  form_structure->DetermineHeuristicTypes();
+
+  std::vector<std::unique_ptr<FormStructure>> form_structures;
+  form_structures.push_back(std::move(form_structure));
+  EXPECT_TRUE(autofill_assistant_.CanShowCreditCardAssist(form_structures));
+}
+
+// Tests that with the feature enabled and proper input,
+// CanShowCreditCardAssist() behaves as expected for insecure contexts.
 TEST_F(AutofillAssistantTest, CanShowCreditCardAssist_FeatureOn_NotSecure) {
   EnableAutofillCreditCardAssist();
 
-  {
-    // Cannot be shown if the context is not secure.
-    FormData form = CreateValidCreditCardFormData();
-    form.action = GURL("http://myform.com");
-    form.action = GURL("http://myform.com/submit");
-    std::unique_ptr<FormStructure> form_structure(new FormStructure(form));
-    form_structure->DetermineHeuristicTypes();
+  // Cannot be shown if the context is not secure.
+  FormData form = CreateValidCreditCardFormData();
+  form.origin = GURL("http://myform.com");
+  form.action = GURL("http://myform.com/submit");
+  std::unique_ptr<FormStructure> form_structure(new FormStructure(form));
+  form_structure->DetermineHeuristicTypes();
 
-    std::vector<std::unique_ptr<FormStructure>> form_structures;
-    form_structures.push_back(std::move(form_structure));
-    EXPECT_FALSE(autofill_assistant_.CanShowCreditCardAssist(form_structures));
-  }
+  std::vector<std::unique_ptr<FormStructure>> form_structures;
+  form_structures.push_back(std::move(form_structure));
+  EXPECT_FALSE(autofill_assistant_.CanShowCreditCardAssist(form_structures));
+}
 
-  {
-    // Can be shown if the context is secure.
-    FormData form = CreateValidCreditCardFormData();
-    std::unique_ptr<FormStructure> form_structure(new FormStructure(form));
-    form_structure->DetermineHeuristicTypes();
+TEST_F(AutofillAssistantTest, CanShowCreditCardAssist_FeatureOn_Javascript) {
+  EnableAutofillCreditCardAssist();
 
-    std::vector<std::unique_ptr<FormStructure>> form_structures;
-    form_structures.push_back(std::move(form_structure));
-    EXPECT_TRUE(autofill_assistant_.CanShowCreditCardAssist(form_structures));
-  }
+  // Can be shown if the context is secure and the form action is a javascript
+  // function (which is a valid url).
+  FormData form = CreateValidCreditCardFormData();
+  form.action = GURL("javascript:alert('hello');");
+  std::unique_ptr<FormStructure> form_structure(new FormStructure(form));
+  form_structure->DetermineHeuristicTypes();
+
+  std::vector<std::unique_ptr<FormStructure>> form_structures;
+  form_structures.push_back(std::move(form_structure));
+  EXPECT_TRUE(autofill_assistant_.CanShowCreditCardAssist(form_structures));
+}
+
+TEST_F(AutofillAssistantTest, CanShowCreditCardAssist_FeatureOn_EmptyAction) {
+  EnableAutofillCreditCardAssist();
+
+  // Can be shown if the context is secure and the form action is empty.
+  FormData form = CreateValidCreditCardFormData();
+  form.action = GURL();
+  std::unique_ptr<FormStructure> form_structure(new FormStructure(form));
+  form_structure->DetermineHeuristicTypes();
+
+  std::vector<std::unique_ptr<FormStructure>> form_structures;
+  form_structures.push_back(std::move(form_structure));
+  EXPECT_TRUE(autofill_assistant_.CanShowCreditCardAssist(form_structures));
 }

[sebsg, 2016/10/21 19:10:55]

Can you add one more test where the js function is invalid but the context is
secure?

[Mathieu, 2016/10/21 20:53:59]

Done.

 
 TEST_F(AutofillAssistantTest, ShowAssistForCreditCard_ValidCard_CancelCvc) {
   EnableAutofillCreditCardAssist();
   std::unique_ptr<FormStructure> form_structure = CreateValidCreditCardForm();
 
   // Will extract the credit card form data.
   std::vector<std::unique_ptr<FormStructure>> form_structures;
   form_structures.push_back(std::move(form_structure));
   EXPECT_TRUE(autofill_assistant_.CanShowCreditCardAssist(form_structures));
 
@@ skipping 33 matching lines @@
   autofill_assistant_.ShowAssistForCreditCard(card);
 
   CardUnmaskDelegate::UnmaskResponse unmask_response;
   unmask_response.cvc = base::ASCIIToUTF16("123");
   static_cast<CardUnmaskDelegate*>(
       autofill_manager_.GetOrCreateFullCardRequest())
       ->OnUnmaskResponse(unmask_response);
 }
 
 }  // namespace autofill