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

Unified Diff: chrome/browser/about_flags.cc

Issue 2707013002: [chrome://flags] Let features override params in the same trial (Closed)
Patch Set: Comments #2 Created 3 years, 10 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
« no previous file with comments | « no previous file | components/flags_ui/feature_entry.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/about_flags.cc
diff --git a/chrome/browser/about_flags.cc b/chrome/browser/about_flags.cc
index 18dd9c483d9b622b0f44c4e47e13fd813a8e0b9c..14ab517f0401aa12721d00237a1b921b91b6b12a 100644
--- a/chrome/browser/about_flags.cc
+++ b/chrome/browser/about_flags.cc
@@ -1872,16 +1872,13 @@ const FeatureEntry kFeatureEntries[] = {
SINGLE_VALUE_TYPE(extensions::switches::kDisableTabForDesktopShare)},
#endif // ENABLE_EXTENSIONS
#if defined(OS_ANDROID)
- // TODO(vitaliii): Use ntp_snippets::kStudyName as the feature trial.
- // Currently, only a single FEATURE_WITH_VARIATIONS_VALUE_TYPE can use a
- // given study name. See crbug.com/690450.
{"content-suggestions-category-order",
IDS_FLAGS_CONTENT_SUGGESTIONS_CATEGORY_ORDER_NAME,
IDS_FLAGS_CONTENT_SUGGESTIONS_CATEGORY_ORDER_DESCRIPTION, kOsAndroid,
FEATURE_WITH_VARIATIONS_VALUE_TYPE(
ntp_snippets::kCategoryOrder,
kContentSuggestionsCategoryOrderFeatureVariations,
- "content-suggestions-category-order-feature-trial")},
+ ntp_snippets::kStudyName)},
{"content-suggestions-category-ranker",
IDS_FLAGS_CONTENT_SUGGESTIONS_CATEGORY_RANKER_NAME,
IDS_FLAGS_CONTENT_SUGGESTIONS_CATEGORY_RANKER_DESCRIPTION, kOsAndroid,
« no previous file with comments | « no previous file | components/flags_ui/feature_entry.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698