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

Unified Diff: chrome/renderer/autofill/form_autofill_browsertest.cc

Issue 23033016: Remove autocheckout code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Even more deletes, and Ilya review. Created 7 years, 4 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: chrome/renderer/autofill/form_autofill_browsertest.cc
diff --git a/chrome/renderer/autofill/form_autofill_browsertest.cc b/chrome/renderer/autofill/form_autofill_browsertest.cc
index 68fa4c7c3bbecb351b2d386fc8a8bfe1e2823909..7ae91ecec5fc10c9230762ab574f16dc2fd518a4 100644
--- a/chrome/renderer/autofill/form_autofill_browsertest.cc
+++ b/chrome/renderer/autofill/form_autofill_browsertest.cc
@@ -1020,32 +1020,6 @@ TEST_F(FormAutofillTest, FillFormIncludingNonFocusableElements) {
&WebInputElement::value);
}
-TEST_F(FormAutofillTest, FillFormForAllElements) {
- static const AutofillFieldCase field_cases[] = {
- // fields: name, initial_value, autocomplete_attribute,
- // should_be_autofilled, autofill_value, expected_value
-
- // All fields except hidden fields (type="hidden") should be Autofilled.
- {"firstname", "", "", true, "filled firstname", "filled firstname"},
- {"lastname", "", "", true, "filled lastname", "filled lastname"},
- // hidden fields should not be extracted to form_data.
- {"notempty", "Hi", "", true, "filled notempty", "filled notempty"},
- {"noautocomplete", "", "off", true, "filled noautocomplete",
- "filled noautocomplete"},
- {"notenabled", "", "", true, "filled notenabled", "filled notenabled"},
- {"readonly", "", "", true, "filled readonly", "filled readonly"},
- {"invisible", "", "", true, "filled invisible", "filled invisible"},
- {"displaynone", "", "", true, "filled displaynone", "filled displaynone"},
- };
- // Enable Autocheckout because |FillFormForAllElements| is only used by
- // Autocheckout.
- base::FieldTrialList field_trial_list(
- new metrics::SHA1EntropyProvider("foo"));
- base::FieldTrialList::CreateFieldTrial("Autocheckout", "Yes");
- TestFormFillFunctions(kFormHtml, field_cases, arraysize(field_cases),
- &FillFormForAllFieldsWrapper, &WebInputElement::value);
-}
-
TEST_F(FormAutofillTest, PreviewForm) {
static const char* html =
"<FORM name=\"TestForm\" action=\"http://buh.com\" method=\"post\">"

Powered by Google App Engine
This is Rietveld 408576698