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

Side by Side 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, 9 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
« no previous file with comments | « no previous file | components/flags_ui/feature_entry.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/about_flags.h" 5 #include "chrome/browser/about_flags.h"
6 6
7 #include <iterator> 7 #include <iterator>
8 #include <map> 8 #include <map>
9 #include <set> 9 #include <set>
10 #include <utility> 10 #include <utility>
(...skipping 1854 matching lines...) Expand 10 before | Expand all | Expand 10 after
1865 {"disable-audio-support-for-desktop-share", 1865 {"disable-audio-support-for-desktop-share",
1866 IDS_FLAG_DISABLE_AUDIO_FOR_DESKTOP_SHARE, 1866 IDS_FLAG_DISABLE_AUDIO_FOR_DESKTOP_SHARE,
1867 IDS_FLAG_DISABLE_AUDIO_FOR_DESKTOP_SHARE_DESCRIPTION, kOsAll, 1867 IDS_FLAG_DISABLE_AUDIO_FOR_DESKTOP_SHARE_DESCRIPTION, kOsAll,
1868 SINGLE_VALUE_TYPE(switches::kDisableAudioSupportForDesktopShare)}, 1868 SINGLE_VALUE_TYPE(switches::kDisableAudioSupportForDesktopShare)},
1869 #if BUILDFLAG(ENABLE_EXTENSIONS) 1869 #if BUILDFLAG(ENABLE_EXTENSIONS)
1870 {"tab-for-desktop-share", IDS_FLAG_DISABLE_TAB_FOR_DESKTOP_SHARE, 1870 {"tab-for-desktop-share", IDS_FLAG_DISABLE_TAB_FOR_DESKTOP_SHARE,
1871 IDS_FLAG_DISABLE_TAB_FOR_DESKTOP_SHARE_DESCRIPTION, kOsAll, 1871 IDS_FLAG_DISABLE_TAB_FOR_DESKTOP_SHARE_DESCRIPTION, kOsAll,
1872 SINGLE_VALUE_TYPE(extensions::switches::kDisableTabForDesktopShare)}, 1872 SINGLE_VALUE_TYPE(extensions::switches::kDisableTabForDesktopShare)},
1873 #endif // ENABLE_EXTENSIONS 1873 #endif // ENABLE_EXTENSIONS
1874 #if defined(OS_ANDROID) 1874 #if defined(OS_ANDROID)
1875 // TODO(vitaliii): Use ntp_snippets::kStudyName as the feature trial.
1876 // Currently, only a single FEATURE_WITH_VARIATIONS_VALUE_TYPE can use a
1877 // given study name. See crbug.com/690450.
1878 {"content-suggestions-category-order", 1875 {"content-suggestions-category-order",
1879 IDS_FLAGS_CONTENT_SUGGESTIONS_CATEGORY_ORDER_NAME, 1876 IDS_FLAGS_CONTENT_SUGGESTIONS_CATEGORY_ORDER_NAME,
1880 IDS_FLAGS_CONTENT_SUGGESTIONS_CATEGORY_ORDER_DESCRIPTION, kOsAndroid, 1877 IDS_FLAGS_CONTENT_SUGGESTIONS_CATEGORY_ORDER_DESCRIPTION, kOsAndroid,
1881 FEATURE_WITH_VARIATIONS_VALUE_TYPE( 1878 FEATURE_WITH_VARIATIONS_VALUE_TYPE(
1882 ntp_snippets::kCategoryOrder, 1879 ntp_snippets::kCategoryOrder,
1883 kContentSuggestionsCategoryOrderFeatureVariations, 1880 kContentSuggestionsCategoryOrderFeatureVariations,
1884 "content-suggestions-category-order-feature-trial")}, 1881 ntp_snippets::kStudyName)},
1885 {"content-suggestions-category-ranker", 1882 {"content-suggestions-category-ranker",
1886 IDS_FLAGS_CONTENT_SUGGESTIONS_CATEGORY_RANKER_NAME, 1883 IDS_FLAGS_CONTENT_SUGGESTIONS_CATEGORY_RANKER_NAME,
1887 IDS_FLAGS_CONTENT_SUGGESTIONS_CATEGORY_RANKER_DESCRIPTION, kOsAndroid, 1884 IDS_FLAGS_CONTENT_SUGGESTIONS_CATEGORY_RANKER_DESCRIPTION, kOsAndroid,
1888 FEATURE_WITH_VARIATIONS_VALUE_TYPE( 1885 FEATURE_WITH_VARIATIONS_VALUE_TYPE(
1889 ntp_snippets::kCategoryRanker, 1886 ntp_snippets::kCategoryRanker,
1890 kContentSuggestionsCategoryRankerFeatureVariations, 1887 kContentSuggestionsCategoryRankerFeatureVariations,
1891 ntp_snippets::kStudyName)}, 1888 ntp_snippets::kStudyName)},
1892 {"override-ntp-suggestions-source", IDS_FLAGS_OVERRIDE_SNIPPETS_SOURCE_NAME, 1889 {"override-ntp-suggestions-source", IDS_FLAGS_OVERRIDE_SNIPPETS_SOURCE_NAME,
1893 IDS_FLAGS_OVERRIDE_SNIPPETS_SOURCE_DESCRIPTION, kOsAndroid, 1890 IDS_FLAGS_OVERRIDE_SNIPPETS_SOURCE_DESCRIPTION, kOsAndroid,
1894 FEATURE_WITH_VARIATIONS_VALUE_TYPE(ntp_snippets::kContentSuggestionsSource, 1891 FEATURE_WITH_VARIATIONS_VALUE_TYPE(ntp_snippets::kContentSuggestionsSource,
(...skipping 635 matching lines...) Expand 10 before | Expand all | Expand 10 after
2530 const base::HistogramBase::Sample kBadSwitchFormatHistogramId = 0; 2527 const base::HistogramBase::Sample kBadSwitchFormatHistogramId = 0;
2531 2528
2532 const FeatureEntry* GetFeatureEntries(size_t* count) { 2529 const FeatureEntry* GetFeatureEntries(size_t* count) {
2533 *count = arraysize(kFeatureEntries); 2530 *count = arraysize(kFeatureEntries);
2534 return kFeatureEntries; 2531 return kFeatureEntries;
2535 } 2532 }
2536 2533
2537 } // namespace testing 2534 } // namespace testing
2538 2535
2539 } // namespace about_flags 2536 } // namespace about_flags
OLDNEW
« 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