[Payments] Field trial and flag to abort payment request if no card.

chrome/browser/android/chrome_feature_list.cc

Index: chrome/browser/android/chrome_feature_list.cc
diff --git a/chrome/browser/android/chrome_feature_list.cc b/chrome/browser/android/chrome_feature_list.cc
index 0f9705fff70df981df9e574da2e46e6981fdf8a7..479de5ebe56a4471fba1c13312a967ddd6e5dcb8 100644
--- a/chrome/browser/android/chrome_feature_list.cc
+++ b/chrome/browser/android/chrome_feature_list.cc
@@ -31,18 +31,12 @@ namespace {
 // this array may either refer to features defined in the header of this file or
 // in other locations in the code base (e.g. chrome/, components/, etc).
 const base::Feature* kFeaturesExposedToJava[] = {
-    &autofill::kAutofillScanCardholderName,
-    &features::kAutoplayMutedVideos,
-    &features::kCredentialManagementAPI,
-    &features::kSimplifiedFullscreenUI,
-    &features::kWebPayments,
     &kAndroidPayIntegrationV1,
+    &kCCTExternalLinkHandling,
     &kChromeHomeFeature,
     &kDownloadsUiFeature,
     &kImportantSitesInCBD,
-    &ntp_snippets::kContentSuggestionsFeature,
-    &ntp_snippets::kSaveToOfflineFeature,
-    &ntp_snippets::kSectionDismissalFeature,
+    &kNoCreditCardAbort,
     &kNTPFakeOmniboxTextFeature,
     &kNTPOfflinePagesFeature,
     &kPhysicalWebFeature,
@@ -51,9 +45,16 @@ const base::Feature* kFeaturesExposedToJava[] = {
     &kSpecialLocaleFeature,
     &kSpecialLocaleWrapper,
     &kTabReparenting,
-    &kCCTExternalLinkHandling,
-    &offline_pages::kOfflinePagesBackgroundLoadingFeature,
+    &autofill::kAutofillScanCardholderName,

[please use gerrit instead, 2016/10/21 19:00:20]

What kind of sorting puts "a" below "k"? :-P

[sebsg, 2016/10/25 15:13:09]

I saw it like the include files where you would include
   chrome/browser/text.h 
before
   chrome/browser/test/test.h

Happy to change it

+    &features::kAutoplayMutedVideos,
+    &features::kCredentialManagementAPI,
+    &features::kSimplifiedFullscreenUI,
+    &features::kWebPayments,
+    &ntp_snippets::kContentSuggestionsFeature,
+    &ntp_snippets::kSaveToOfflineFeature,
+    &ntp_snippets::kSectionDismissalFeature,
     &offline_pages::kBackgroundLoaderForDownloadsFeature,
+    &offline_pages::kOfflinePagesBackgroundLoadingFeature,
     &offline_pages::kOfflinePagesCTFeature,  // See crbug.com/620421.
     &offline_pages::kOfflinePagesSharingFeature,
     &password_manager::features::kViewPasswords,
@@ -64,6 +65,9 @@ const base::Feature* kFeaturesExposedToJava[] = {
 const base::Feature kAndroidPayIntegrationV1{"AndroidPayIntegrationV1",
                                              base::FEATURE_ENABLED_BY_DEFAULT};
 
+const base::Feature kCCTExternalLinkHandling{"CCTExternalLinkHandling",
+                                             base::FEATURE_ENABLED_BY_DEFAULT};
+
 const base::Feature kChromeHomeFeature{"ChromeHome",
                                        base::FEATURE_DISABLED_BY_DEFAULT};
 
@@ -73,6 +77,9 @@ const base::Feature kDownloadsUiFeature{"DownloadsUi",
 const base::Feature kImportantSitesInCBD{"ImportantSitesInCBD",
                                          base::FEATURE_DISABLED_BY_DEFAULT};
 
+const base::Feature kNoCreditCardAbort{"NoCreditCardAbort",
+                                       base::FEATURE_DISABLED_BY_DEFAULT};
+
 const base::Feature kNTPOfflinePagesFeature{"NTPOfflinePages",
                                             base::FEATURE_DISABLED_BY_DEFAULT};
 
@@ -97,9 +104,6 @@ const base::Feature kSpecialLocaleWrapper{"SpecialLocaleWrapper",
 const base::Feature kTabReparenting{"TabReparenting",
                                     base::FEATURE_ENABLED_BY_DEFAULT};
 
-const base::Feature kCCTExternalLinkHandling{"CCTExternalLinkHandling",
-                                    base::FEATURE_ENABLED_BY_DEFAULT};
-
 const base::Feature kUserMediaScreenCapturing{
     "UserMediaScreenCapturing", base::FEATURE_DISABLED_BY_DEFAULT};