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

Side by Side Diff: chrome/browser/android/chrome_feature_list.cc

Issue 2394273002: [Android] Permanently disable "All bookmarks" and remove its experiment (Closed)
Patch Set: isherman's comment Created 4 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/android/chrome_feature_list.h" 5 #include "chrome/browser/android/chrome_feature_list.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <string> 9 #include <string>
10 10
(...skipping 18 matching lines...) Expand all
29 29
30 // Array of features exposed through the Java ChromeFeatureList API. Entries in 30 // Array of features exposed through the Java ChromeFeatureList API. Entries in
31 // this array may either refer to features defined in the header of this file or 31 // this array may either refer to features defined in the header of this file or
32 // in other locations in the code base (e.g. chrome/, components/, etc). 32 // in other locations in the code base (e.g. chrome/, components/, etc).
33 const base::Feature* kFeaturesExposedToJava[] = { 33 const base::Feature* kFeaturesExposedToJava[] = {
34 &autofill::kAutofillScanCardholderName, 34 &autofill::kAutofillScanCardholderName,
35 &features::kAutoplayMutedVideos, 35 &features::kAutoplayMutedVideos,
36 &features::kCredentialManagementAPI, 36 &features::kCredentialManagementAPI,
37 &features::kSimplifiedFullscreenUI, 37 &features::kSimplifiedFullscreenUI,
38 &features::kWebPayments, 38 &features::kWebPayments,
39 &kAllBookmarksFeature,
40 &kAndroidPayIntegrationV1, 39 &kAndroidPayIntegrationV1,
41 &kDownloadsUiFeature, 40 &kDownloadsUiFeature,
42 &kImportantSitesInCBD, 41 &kImportantSitesInCBD,
43 &ntp_snippets::kContentSuggestionsFeature, 42 &ntp_snippets::kContentSuggestionsFeature,
44 &ntp_snippets::kSaveToOfflineFeature, 43 &ntp_snippets::kSaveToOfflineFeature,
45 &kNTPFakeOmniboxTextFeature, 44 &kNTPFakeOmniboxTextFeature,
46 &kNTPMaterialDesign, 45 &kNTPMaterialDesign,
47 &kNTPOfflinePagesFeature, 46 &kNTPOfflinePagesFeature,
48 &kPhysicalWebFeature, 47 &kPhysicalWebFeature,
49 &kPhysicalWebIgnoreOtherClientsFeature, 48 &kPhysicalWebIgnoreOtherClientsFeature,
50 &kReadItLaterInMenu, 49 &kReadItLaterInMenu,
51 &kScanCardsInWebPayments, 50 &kScanCardsInWebPayments,
52 &kSpecialLocaleFeature, 51 &kSpecialLocaleFeature,
53 &kSpecialLocaleWrapper, 52 &kSpecialLocaleWrapper,
54 &kTabReparenting, 53 &kTabReparenting,
55 &offline_pages::kOfflinePagesBackgroundLoadingFeature, 54 &offline_pages::kOfflinePagesBackgroundLoadingFeature,
56 &offline_pages::kBackgroundLoaderForDownloadsFeature, 55 &offline_pages::kBackgroundLoaderForDownloadsFeature,
57 &offline_pages::kOfflinePagesCTFeature, // See crbug.com/620421. 56 &offline_pages::kOfflinePagesCTFeature, // See crbug.com/620421.
58 &offline_pages::kOfflinePagesSharingFeature, 57 &offline_pages::kOfflinePagesSharingFeature,
59 &password_manager::features::kViewPasswords, 58 &password_manager::features::kViewPasswords,
60 }; 59 };
61 60
62 } // namespace 61 } // namespace
63 62
64 const base::Feature kAllBookmarksFeature{"AllBookmarks",
65 base::FEATURE_ENABLED_BY_DEFAULT};
66
67 const base::Feature kAndroidPayIntegrationV1{"AndroidPayIntegrationV1", 63 const base::Feature kAndroidPayIntegrationV1{"AndroidPayIntegrationV1",
68 base::FEATURE_ENABLED_BY_DEFAULT}; 64 base::FEATURE_ENABLED_BY_DEFAULT};
69 65
70 const base::Feature kDownloadsUiFeature{"DownloadsUi", 66 const base::Feature kDownloadsUiFeature{"DownloadsUi",
71 base::FEATURE_DISABLED_BY_DEFAULT}; 67 base::FEATURE_DISABLED_BY_DEFAULT};
72 68
73 const base::Feature kImportantSitesInCBD{"ImportantSitesInCBD", 69 const base::Feature kImportantSitesInCBD{"ImportantSitesInCBD",
74 base::FEATURE_DISABLED_BY_DEFAULT}; 70 base::FEATURE_DISABLED_BY_DEFAULT};
75 71
76 const base::Feature kNTPMaterialDesign{"NTPMaterialDesign", 72 const base::Feature kNTPMaterialDesign{"NTPMaterialDesign",
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
118 NOTREACHED(); 114 NOTREACHED();
119 return false; 115 return false;
120 } 116 }
121 117
122 bool RegisterChromeFeatureListJni(JNIEnv* env) { 118 bool RegisterChromeFeatureListJni(JNIEnv* env) {
123 return RegisterNativesImpl(env); 119 return RegisterNativesImpl(env);
124 } 120 }
125 121
126 } // namespace android 122 } // namespace android
127 } // namespace chrome 123 } // namespace chrome
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698