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

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

Issue 2955963002: Update Chrome Upstream flow to reflect new UI mocks (Closed)
Patch Set: Put changes behind AutofillUpstreamShowNewUi flag Created 3 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: components/autofill/core/browser/autofill_experiments.cc
diff --git a/components/autofill/core/browser/autofill_experiments.cc b/components/autofill/core/browser/autofill_experiments.cc
index 6466e071eeeda61995077f0eb774d0360522d808..d28a21eb049c2dcf68269ccac3e8aea1ff37e949 100644
--- a/components/autofill/core/browser/autofill_experiments.cc
+++ b/components/autofill/core/browser/autofill_experiments.cc
@@ -42,6 +42,8 @@ const base::Feature kAutofillSuppressDisusedAddresses{
"AutofillSuppressDisusedAddresses", base::FEATURE_DISABLED_BY_DEFAULT};
const base::Feature kAutofillUpstreamRequestCvcIfMissing{
"AutofillUpstreamRequestCvcIfMissing", base::FEATURE_DISABLED_BY_DEFAULT};
+const base::Feature kAutofillUpstreamShowNewUi{
+ "AutofillUpstreamShowNewUi", base::FEATURE_DISABLED_BY_DEFAULT};
const base::Feature kAutofillUpstreamUseAutofillProfileComparator{
"AutofillUpstreamUseAutofillProfileComparator",
base::FEATURE_ENABLED_BY_DEFAULT};
@@ -263,6 +265,10 @@ bool IsAutofillUpstreamRequestCvcIfMissingExperimentEnabled() {
#endif
}
+bool IsAutofillUpstreamShowNewUiExperimentEnabled() {
jiahuiguo 2017/08/09 00:33:37 Also add macro?
Jared Saul 2017/08/09 00:45:24 Done; discussed offline.
+ return base::FeatureList::IsEnabled(kAutofillUpstreamShowNewUi);
+}
+
base::TimeDelta GetMaxTimeSinceAutofillProfileUseForCardUpload() {
int value;
const std::string param_value = variations::GetVariationParamValueByFeature(

Powered by Google App Engine
This is Rietveld 408576698