| Index: components/variations/variations_http_header_provider_unittest.cc
|
| diff --git a/components/variations/variations_http_header_provider_unittest.cc b/components/variations/variations_http_header_provider_unittest.cc
|
| index 90552c844765bc00a86e024df8b61a7213f242ac..585ff56805d6097d8703b746b84d16596869cb3c 100644
|
| --- a/components/variations/variations_http_header_provider_unittest.cc
|
| +++ b/components/variations/variations_http_header_provider_unittest.cc
|
| @@ -65,7 +65,7 @@ TEST_F(VariationsHttpHeaderProviderTest, SetDefaultVariationIds_Valid) {
|
| VariationsHttpHeaderProvider provider;
|
|
|
| // Valid experiment ids.
|
| - EXPECT_TRUE(provider.SetDefaultVariationIds("12,456,t789"));
|
| + EXPECT_TRUE(provider.SetDefaultVariationIds({"12", "456", "t789"}));
|
| provider.InitVariationIDsCacheIfNeeded();
|
| std::string variations = provider.GetClientDataHeader();
|
| EXPECT_FALSE(variations.empty());
|
| @@ -83,12 +83,12 @@ TEST_F(VariationsHttpHeaderProviderTest, SetDefaultVariationIds_Invalid) {
|
| VariationsHttpHeaderProvider provider;
|
|
|
| // Invalid experiment ids.
|
| - EXPECT_FALSE(provider.SetDefaultVariationIds("abcd12,456"));
|
| + EXPECT_FALSE(provider.SetDefaultVariationIds({"abcd12", "456"}));
|
| provider.InitVariationIDsCacheIfNeeded();
|
| EXPECT_TRUE(provider.GetClientDataHeader().empty());
|
|
|
| // Invalid trigger experiment id
|
| - EXPECT_FALSE(provider.SetDefaultVariationIds("12,tabc456"));
|
| + EXPECT_FALSE(provider.SetDefaultVariationIds({"12", "tabc456"}));
|
| provider.InitVariationIDsCacheIfNeeded();
|
| EXPECT_TRUE(provider.GetClientDataHeader().empty());
|
| }
|
| @@ -131,7 +131,7 @@ TEST_F(VariationsHttpHeaderProviderTest, GetVariationsString) {
|
| CreateTrialAndAssociateId("t2", "g2", GOOGLE_WEB_PROPERTIES, 124);
|
|
|
| VariationsHttpHeaderProvider provider;
|
| - provider.SetDefaultVariationIds("100,200");
|
| + provider.SetDefaultVariationIds({"100", "200"});
|
| EXPECT_EQ(" 100 123 124 200 ", provider.GetVariationsString());
|
| }
|
|
|
|
|