From eba6ceed44d6a0c6b327adf05c8aa7b3dfdd577a Mon Sep 17 00:00:00 2001 From: treib Date: Wed, 9 Sep 2015 06:25:59 -0700 Subject: [PATCH] Popular sites on the NTP: check that experiment group StartsWith (rather than IS) "Enabled" This will give us the option of creating additional "enabled" groups in the future BUG=516618 Review URL: https://codereview.chromium.org/1314503008 Cr-Commit-Position: refs/heads/master@{#347920} --- chrome/browser/android/most_visited_sites.cc | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/chrome/browser/android/most_visited_sites.cc b/chrome/browser/android/most_visited_sites.cc index 19216b2f7764..b9ecdde7a2eb 100644 --- a/chrome/browser/android/most_visited_sites.cc +++ b/chrome/browser/android/most_visited_sites.cc @@ -13,6 +13,7 @@ #include "base/metrics/histogram.h" #include "base/metrics/sparse_histogram.h" #include "base/strings/string_number_conversions.h" +#include "base/strings/string_util.h" #include "base/strings/stringprintf.h" #include "base/strings/utf_string_conversions.h" #include "base/time/time.h" @@ -121,7 +122,8 @@ bool ShouldShowPopularSites() { return false; if (cmd_line->HasSwitch(switches::kEnableNTPPopularSites)) return true; - return group_name == "Enabled"; + return base::StartsWith(group_name, "Enabled", + base::CompareCase::INSENSITIVE_ASCII); } std::string GetPopularSitesFilename() { -- 2.11.4.GIT