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

Unified Diff: components/autofill/core/browser/autofill_manager.cc

Issue 2146523004: Make full-form autofill the only implementation on iOS. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 5 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/core/browser/autofill_manager.cc
diff --git a/components/autofill/core/browser/autofill_manager.cc b/components/autofill/core/browser/autofill_manager.cc
index e43e85b26d8268c3f1c56f4bc09cb35c5e81517b..41962b58f23f9685ac2014b139d20effa9cacabc 100644
--- a/components/autofill/core/browser/autofill_manager.cc
+++ b/components/autofill/core/browser/autofill_manager.cc
@@ -66,7 +66,6 @@
#include "url/gurl.h"
#if defined(OS_IOS)
-#include "components/autofill/core/browser/autofill_field_trial_ios.h"
#include "components/autofill/core/browser/keyboard_accessory_metrics_logger.h"
#endif
@@ -591,13 +590,6 @@ bool AutofillManager::WillFillCreditCardNumber(const FormData& form,
if (autofill_field->Type().GetStorableType() == CREDIT_CARD_NUMBER)
return true;
-#if defined(OS_IOS)
- // On iOS, we only fill out one field at a time (assuming the new full-form
- // feature isn't enabled). So we only need to check the current field.
- if (!AutofillFieldTrialIOS::IsFullFormAutofillEnabled())
- return false;
-#endif
-
// If the relevant section is already autofilled, the new fill operation will
// only fill |autofill_field|.
if (SectionIsAutofilled(*form_structure, form, autofill_field->section()))

Powered by Google App Engine
This is Rietveld 408576698