Expanded Autofill Credit Card Popup Layout Experiment in Android.

components/autofill/core/browser/autofill_experiments.cc

 // Copyright 2015 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_experiments.h"
 
 #include "base/command_line.h"
 #include "base/feature_list.h"
 #include "base/metrics/field_trial.h"
+#include "base/strings/string16.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
+#include "base/strings/utf_string_conversions.h"
 #include "build/build_config.h"
+#include "components/autofill/core/browser/suggestion.h"
 #include "components/autofill/core/common/autofill_pref_names.h"
 #include "components/autofill/core/common/autofill_switches.h"
 #include "components/prefs/pref_service.h"
 #include "components/security_state/core/switches.h"
 #include "components/sync/driver/sync_service.h"
 #include "components/variations/variations_associated_data.h"
 #include "google_apis/gaia/gaia_auth_util.h"
+#include "grit/components_strings.h"
+#include "ui/base/l10n/l10n_util.h"
 
 namespace autofill {
 
 const base::Feature kAutofillCreditCardAssist{
     "AutofillCreditCardAssist", base::FEATURE_DISABLED_BY_DEFAULT};
 const base::Feature kAutofillCreditCardSigninPromo{
     "AutofillCreditCardSigninPromo", base::FEATURE_DISABLED_BY_DEFAULT};
 const base::Feature kAutofillProfileCleanup{"AutofillProfileCleanup",
                                             base::FEATURE_DISABLED_BY_DEFAULT};
 const base::Feature kAutofillScanCardholderName{
     "AutofillScanCardholderName", base::FEATURE_DISABLED_BY_DEFAULT};
+const base::Feature kAutofillCreditCardPopupLayout{
+    "AutofillCreditCardPopupLayout", base::FEATURE_DISABLED_BY_DEFAULT};
 const char kCreditCardSigninPromoImpressionLimitParamKey[] = "impression_limit";
+const char kAutofillCreditCardPopupBackgroundColorKey[] = "background_color";
+const char kAutofillCreditCardPopupDividerColorKey[] = "dropdown_divider_color";
+const char kAutofillCreditCardPopupValueBoldKey[] = "is_value_bold";
+const char kAutofillCreditCardPopupDropdownItemHeightKey[] =
+    "dropdown_item_height";
+const char kAutofillCreditCardPopupIsIconAtLeftKey[] =
+    "is_credit_card_icon_at_left";
 
 bool IsAutofillEnabled(const PrefService* pref_service) {
   return pref_service->GetBoolean(prefs::kAutofillEnabled);
 }
 
 bool IsInAutofillSuggestionsDisabledExperiment() {
   std::string group_name =
       base::FieldTrialList::FindFullName("AutofillEnabled");
   return group_name == "Disabled";
 }
@@ skipping 18 matching lines @@
   int impression_limit;
   std::string param_value = variations::GetVariationParamValueByFeature(
       kAutofillCreditCardSigninPromo,
       kCreditCardSigninPromoImpressionLimitParamKey);
   if (!param_value.empty() && base::StringToInt(param_value, &impression_limit))
     return impression_limit;
 
   return 0;
 }
 
+bool IsAutofillCreditCardPopupLayoutExperimentEnabled() {
+  return base::FeatureList::IsEnabled(kAutofillCreditCardPopupLayout);
+}
+
+int GetCreditCardPopupBackgroundColor() {
+  int background_color;

[Mathieu, 2016/11/30 21:27:35]

lines 89-94 could be an anonymous function to be reused in three of your new
functions here.

GetCreditCardPopupParameterIntValue(kAutofillCreditCardPopupBackgroundColorKey)

[csashi, 2016/12/01 01:13:43]

Done.

+  const std::string param_value = variations::GetVariationParamValueByFeature(
+      kAutofillCreditCardPopupLayout,
+      kAutofillCreditCardPopupBackgroundColorKey);
+  if (!param_value.empty() &&
+      base::StringToUint(param_value, &background_color))
+    return background_color;
+
+  return 0;
+}
+
+int GetCreditCardPopupDividerColor() {
+  int background_color;
+  const std::string param_value = variations::GetVariationParamValueByFeature(
+      kAutofillCreditCardPopupLayout, kAutofillCreditCardPopupDividerColorKey);
+  if (!param_value.empty() &&
+      base::StringToUint(param_value, &background_color))
+    return background_color;
+
+  return 0;
+}
+
+bool IsCreditCardPopupValueBold() {
+  const std::string param_value = variations::GetVariationParamValueByFeature(
+      kAutofillCreditCardPopupLayout, kAutofillCreditCardPopupValueBoldKey);
+  return param_value.compare("true") == 0;

[Mathieu, 2016/11/30 21:27:35]

what about 

return param_value == "true"?

[csashi, 2016/12/01 01:13:43]

Done.

+}
+
+int GetCreditCardPopupDropdownItemHeight() {
+  int dropdown_item_height;
+  const std::string param_value = variations::GetVariationParamValueByFeature(
+      kAutofillCreditCardPopupLayout,
+      kAutofillCreditCardPopupDropdownItemHeightKey);
+  if (!param_value.empty() &&
+      base::StringToInt(param_value, &dropdown_item_height))
+    return dropdown_item_height;
+  return -1;
+}
+
+bool IsCreditCardIconInPopupAtLeft() {
+  const std::string param_value = variations::GetVariationParamValueByFeature(
+      kAutofillCreditCardPopupLayout, kAutofillCreditCardPopupIsIconAtLeftKey);
+  return param_value.compare("true") == 0;
+}
+
+void ModifyAutofillCreditCardSuggestion(Suggestion* suggestion) {

[Mathieu, 2016/11/30 21:27:34]

Add a comment detailing the final result (it's a bit obscure to read now :))

[csashi, 2016/12/01 01:13:43]

Done.

+  if (IsAutofillCreditCardPopupLayoutExperimentEnabled()) {

[Mathieu, 2016/11/30 21:27:34]

DCHECK(IsAutofillCreditCardPopupLayoutExperimentEnabled())?

It should be a precondition that this is true, so let's not check again in
release builds

[csashi, 2016/12/01 01:13:43]

Done.

+    suggestion->value.append(base::ASCIIToUTF16(", "));
+    suggestion->value.append(l10n_util::GetStringUTF16(
+        IDS_AUTOFILL_CREDIT_CARD_EXPIRATION_DATE_LABEL));
+    suggestion->value.append(base::ASCIIToUTF16(" "));
+    suggestion->value.append(suggestion->label);
+    suggestion->label.clear();
+  }
+}
+
 bool OfferStoreUnmaskedCards() {
 #if defined(OS_LINUX) && !defined(OS_CHROMEOS)
   // The checkbox can be forced on with a flag, but by default we don't store
   // on Linux due to lack of system keychain integration. See crbug.com/162735
   return base::CommandLine::ForCurrentProcess()->HasSwitch(
       switches::kEnableOfferStoreUnmaskedWalletCards);
 #else
   // Query the field trial before checking command line flags to ensure UMA
   // reports the correct group.
   std::string group_name =
@@ skipping 64 matching lines @@
 
 bool IsCreditCardAutofillHttpWarningEnabled() {
   std::string choice =
       base::CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
           security_state::switches::kMarkHttpAs);
   return choice == security_state::switches::
                        kMarkHttpWithPasswordsOrCcWithChipAndFormWarning;
 }
 
 }  // namespace autofill