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

Unified Diff: components/ntp_snippets/features.h

Issue 2804633003: Add base::FeatureParam<> struct (Closed)
Patch Set: rebase 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/ntp_snippets/features.h
diff --git a/components/ntp_snippets/features.h b/components/ntp_snippets/features.h
index e092a09a2bb73f65129432216c15dae596d69122..f0fb3fbd46344f5705b8d25e7600053442829ab7 100644
--- a/components/ntp_snippets/features.h
+++ b/components/ntp_snippets/features.h
@@ -9,6 +9,7 @@
#include <string>
#include "base/feature_list.h"
+#include "base/metrics/field_trial_params.h"
#include "components/ntp_snippets/category_rankers/category_ranker.h"
#include "components/prefs/pref_service.h"
@@ -53,17 +54,13 @@ extern const base::Feature kPublisherFaviconsFromNewServerFeature;
extern const base::Feature kRemoteSuggestionsEmulateM58FetchingSchedule;
// Parameter and its values for the kCategoryRanker feature flag.
-extern const char kCategoryRankerParameter[];
-extern const char kCategoryRankerConstantRanker[];
-extern const char kCategoryRankerClickBasedRanker[];
-
enum class CategoryRankerChoice {
CONSTANT,
CLICK_BASED,
};
-
-// Returns which CategoryRanker to use according to kCategoryRanker feature.
-CategoryRankerChoice GetSelectedCategoryRanker();
+extern const char kCategoryRankerConstantRanker[];
+extern const char kCategoryRankerClickBasedRanker[];
+extern const base::FeatureParam<CategoryRankerChoice> kCategoryRankerParameter;
// Builds a CategoryRanker according to kCategoryRanker feature.
std::unique_ptr<CategoryRanker> BuildSelectedCategoryRanker(
@@ -74,14 +71,13 @@ std::unique_ptr<CategoryRanker> BuildSelectedCategoryRanker(
extern const base::Feature kCategoryOrder;
// Parameter and its values for the kCategoryOrder feature flag.
-extern const char kCategoryOrderParameter[];
-extern const char kCategoryOrderGeneral[];
-extern const char kCategoryOrderEmergingMarketsOriented[];
-
enum class CategoryOrderChoice {
GENERAL,
EMERGING_MARKETS_ORIENTED,
};
+extern const char kCategoryOrderGeneral[];
+extern const char kCategoryOrderEmergingMarketsOriented[];
+extern const base::FeatureParam<CategoryOrderChoice> kCategoryOrderParameter;
// Returns which category order to use according to kCategoryOrder feature.
CategoryOrderChoice GetSelectedCategoryOrder();

Powered by Google App Engine
This is Rietveld 408576698