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

Side by Side Diff: chrome/browser/about_flags.cc

Issue 2665743002: [Remote suggestions] Clean up variation params in the fetcher. (Closed)
Patch Set: Further changes & unit-test fixes 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 unified diff | Download patch
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 522 matching lines...) Expand 10 before | Expand all | Expand 10 after
533 #endif // OS_ANDROID 533 #endif // OS_ANDROID
534 534
535 #if defined(OS_ANDROID) 535 #if defined(OS_ANDROID)
536 const FeatureEntry::FeatureParam kNTPSnippetsFeatureVariationChromeReader[] = { 536 const FeatureEntry::FeatureParam kNTPSnippetsFeatureVariationChromeReader[] = {
537 {"content_suggestions_backend", ntp_snippets::kChromeReaderServer}}; 537 {"content_suggestions_backend", ntp_snippets::kChromeReaderServer}};
538 538
539 const FeatureEntry::FeatureParam kNTPSnippetsFeatureVariationServer[] = { 539 const FeatureEntry::FeatureParam kNTPSnippetsFeatureVariationServer[] = {
540 {"content_suggestions_backend", 540 {"content_suggestions_backend",
541 ntp_snippets::kContentSuggestionsServer}}; 541 ntp_snippets::kContentSuggestionsServer}};
542 542
543 const FeatureEntry::FeatureParam
jkrcal 2017/02/03 10:17:49 I've forgotten to remove this in the previous patc
544 kNTPSnippetsFeatureVariationServerNonPersonalized[] = {
545 {"content_suggestions_backend",
546 ntp_snippets::kContentSuggestionsServer},
547 {"fetching_personalization", "non_personal"}};
548
549 const FeatureEntry::FeatureVariation kNTPSnippetsFeatureVariations[] = { 543 const FeatureEntry::FeatureVariation kNTPSnippetsFeatureVariations[] = {
550 {"via ChromeReader", kNTPSnippetsFeatureVariationChromeReader, 544 {"via ChromeReader", kNTPSnippetsFeatureVariationChromeReader,
551 arraysize(kNTPSnippetsFeatureVariationChromeReader), nullptr}, 545 arraysize(kNTPSnippetsFeatureVariationChromeReader), nullptr},
552 {"via content suggestion server (backed by ChromeReader)", 546 {"via content suggestion server (backed by ChromeReader)",
553 kNTPSnippetsFeatureVariationServer, 547 kNTPSnippetsFeatureVariationServer,
554 arraysize(kNTPSnippetsFeatureVariationServer), nullptr}, 548 arraysize(kNTPSnippetsFeatureVariationServer), nullptr},
555 {"via content suggestion server (backed by ChromeReader, non-personalized)",
556 kNTPSnippetsFeatureVariationServerNonPersonalized,
557 arraysize(kNTPSnippetsFeatureVariationServerNonPersonalized), nullptr},
558 {"via content suggestion server (backed by Google Now)", 549 {"via content suggestion server (backed by Google Now)",
559 kNTPSnippetsFeatureVariationServer, 550 kNTPSnippetsFeatureVariationServer,
560 arraysize(kNTPSnippetsFeatureVariationServer), "3313279"}, 551 arraysize(kNTPSnippetsFeatureVariationServer), "3313279"}};
561 {"via content suggestion server (backed by Google Now, non-personalized)",
562 kNTPSnippetsFeatureVariationServerNonPersonalized,
563 arraysize(kNTPSnippetsFeatureVariationServerNonPersonalized), "3313279"}};
564 #endif // OS_ANDROID 552 #endif // OS_ANDROID
565 553
566 #if defined(OS_ANDROID) 554 #if defined(OS_ANDROID)
567 const FeatureEntry::Choice kUpdateMenuItemSummaryChoices[] = { 555 const FeatureEntry::Choice kUpdateMenuItemSummaryChoices[] = {
568 {IDS_FLAGS_UPDATE_MENU_ITEM_NO_SUMMARY, "", ""}, 556 {IDS_FLAGS_UPDATE_MENU_ITEM_NO_SUMMARY, "", ""},
569 {IDS_FLAGS_UPDATE_MENU_ITEM_DEFAULT_SUMMARY, 557 {IDS_FLAGS_UPDATE_MENU_ITEM_DEFAULT_SUMMARY,
570 switches::kForceShowUpdateMenuItemSummary, ""}, 558 switches::kForceShowUpdateMenuItemSummary, ""},
571 {IDS_FLAGS_UPDATE_MENU_ITEM_NEW_FEATURES_SUMMARY, 559 {IDS_FLAGS_UPDATE_MENU_ITEM_NEW_FEATURES_SUMMARY,
572 switches::kForceShowUpdateMenuItemNewFeaturesSummary, ""}, 560 switches::kForceShowUpdateMenuItemNewFeaturesSummary, ""},
573 {IDS_FLAGS_UPDATE_MENU_ITEM_CUSTOM_SUMMARY, 561 {IDS_FLAGS_UPDATE_MENU_ITEM_CUSTOM_SUMMARY,
(...skipping 1837 matching lines...) Expand 10 before | Expand all | Expand 10 after
2411 const base::HistogramBase::Sample kBadSwitchFormatHistogramId = 0; 2399 const base::HistogramBase::Sample kBadSwitchFormatHistogramId = 0;
2412 2400
2413 const FeatureEntry* GetFeatureEntries(size_t* count) { 2401 const FeatureEntry* GetFeatureEntries(size_t* count) {
2414 *count = arraysize(kFeatureEntries); 2402 *count = arraysize(kFeatureEntries);
2415 return kFeatureEntries; 2403 return kFeatureEntries;
2416 } 2404 }
2417 2405
2418 } // namespace testing 2406 } // namespace testing
2419 2407
2420 } // namespace about_flags 2408 } // namespace about_flags
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698