Chromium Code Reviews| Index: chrome/common/variations/variations_util_unittest.cc |
| diff --git a/chrome/common/variations/variations_util_unittest.cc b/chrome/common/variations/variations_util_unittest.cc |
| index 87d58ad5636acab17cf253f42efc3eaaad40e534..ea7ffc66fefdf5888d8efe897769eecc281417e5 100644 |
| --- a/chrome/common/variations/variations_util_unittest.cc |
| +++ b/chrome/common/variations/variations_util_unittest.cc |
| @@ -49,6 +49,13 @@ TEST_F(VariationsUtilTest, AssociateParamsFromString) { |
| EXPECT_EQ("/", params["a"]); |
| } |
| +TEST_F(VariationsUtilTest, AssociateParamsFromStringWithSameStudy) { |
| + const std::string kTrialName = "AssociateVariationParams"; |
| + const std::string kVariationsString = |
| + "AssociateVariationParams.A:a/10/b/test,AssociateVariationParams.A:a/%2F"; |
|
Mark P
2016/02/10 04:53:41
As the escape string here isn't needed / important
danduong
2016/02/10 18:19:53
Done.
|
| + ASSERT_FALSE(AssociateParamsFromString(kVariationsString)); |
| +} |
| + |
| TEST_F(VariationsUtilTest, AssociateParamsFromFieldTrialConfig) { |
| const FieldTrialGroupParams array_kFieldTrialConfig_params[] = {{"x", "1"}, |
| {"y", "2"}}; |