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

Unified Diff: base/metrics/field_trial_unittest.cc

Issue 9705074: Supporting command line argument to force field trials (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 8 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 side-by-side diff with in-line comments
Download patch
Index: base/metrics/field_trial_unittest.cc
diff --git a/base/metrics/field_trial_unittest.cc b/base/metrics/field_trial_unittest.cc
index 7dc70ec39482a5b0817099505de09fc880580080..abf3e89556ea04abaafc3e8f07fc87d049d64c62 100644
--- a/base/metrics/field_trial_unittest.cc
+++ b/base/metrics/field_trial_unittest.cc
@@ -47,8 +47,8 @@ TEST_F(FieldTrialTest, Registration) {
EXPECT_FALSE(FieldTrialList::Find(name1));
EXPECT_FALSE(FieldTrialList::Find(name2));
- FieldTrial* trial1 =
- new FieldTrial(name1, 10, "default name 1 test", next_year_, 12, 31);
+ FieldTrial* trial1 = FieldTrial::CreateInstance(
+ name1, 10, "default name 1 test", next_year_, 12, 31);
EXPECT_EQ(FieldTrial::kNotFinalized, trial1->group_);
EXPECT_EQ(name1, trial1->name());
EXPECT_EQ("", trial1->group_name_internal());
@@ -58,8 +58,8 @@ TEST_F(FieldTrialTest, Registration) {
EXPECT_EQ(trial1, FieldTrialList::Find(name1));
EXPECT_FALSE(FieldTrialList::Find(name2));
- FieldTrial* trial2 =
- new FieldTrial(name2, 10, "default name 2 test", next_year_, 12, 31);
+ FieldTrial* trial2 = FieldTrial::CreateInstance(
+ name2, 10, "default name 2 test", next_year_, 12, 31);
EXPECT_EQ(FieldTrial::kNotFinalized, trial2->group_);
EXPECT_EQ(name2, trial2->name());
EXPECT_EQ("", trial2->group_name_internal());
@@ -83,18 +83,16 @@ TEST_F(FieldTrialTest, AbsoluteProbabilities) {
always_false[0] = i;
default_always_false[0] = i;
- FieldTrial* trial_true =
- new FieldTrial(
- always_true, 10, default_always_true, next_year_, 12, 31);
+ FieldTrial* trial_true = FieldTrial::CreateInstance(
+ always_true, 10, default_always_true, next_year_, 12, 31);
const std::string winner = "TheWinner";
int winner_group = trial_true->AppendGroup(winner, 10);
EXPECT_EQ(winner_group, trial_true->group());
EXPECT_EQ(winner, trial_true->group_name());
- FieldTrial* trial_false =
- new FieldTrial(
- always_false, 10, default_always_false, next_year_, 12, 31);
+ FieldTrial* trial_false = FieldTrial::CreateInstance(
+ always_false, 10, default_always_false, next_year_, 12, 31);
int loser_group = trial_false->AppendGroup("ALoser", 0);
EXPECT_NE(loser_group, trial_false->group());
@@ -109,7 +107,7 @@ TEST_F(FieldTrialTest, RemainingProbability) {
int counter = 0;
do {
std::string name = StringPrintf("trial%d", ++counter);
- trial = new FieldTrial(name, 10, winner, next_year_, 12, 31);
+ trial = FieldTrial::CreateInstance(name, 10, winner, next_year_, 12, 31);
trial->AppendGroup(loser, 5); // 50% chance of not being chosen.
// If a group is not assigned, group_ will be kNotFinalized.
} while (trial->group_ != FieldTrial::kNotFinalized);
@@ -133,8 +131,8 @@ TEST_F(FieldTrialTest, FiftyFiftyProbability) {
std::string name = base::StringPrintf("FiftyFifty%d", ++counter);
std::string default_group_name = base::StringPrintf("Default FiftyFifty%d",
++counter);
- scoped_refptr<FieldTrial> trial(
- new FieldTrial(name, 2, default_group_name, next_year_, 12, 31));
+ scoped_refptr<FieldTrial> trial(FieldTrial::CreateInstance(
+ name, 2, default_group_name, next_year_, 12, 31));
trial->AppendGroup("first", 1); // 50% chance of being chosen.
// If group_ is kNotFinalized, then a group assignement hasn't been done.
if (trial->group_ != FieldTrial::kNotFinalized) {
@@ -157,8 +155,8 @@ TEST_F(FieldTrialTest, MiddleProbabilities) {
for (int i = 1; i < 250; ++i) {
name[0] = i;
default_group_name[0] = i;
- FieldTrial* trial =
- new FieldTrial(name, 10, default_group_name, next_year_, 12, 31);
+ FieldTrial* trial = FieldTrial::CreateInstance(
+ name, 10, default_group_name, next_year_, 12, 31);
int might_win = trial->AppendGroup("MightWin", 5);
if (trial->group() == might_win) {
@@ -180,9 +178,8 @@ TEST_F(FieldTrialTest, OneWinner) {
char default_group_name[] = "Default some name";
int group_count(10);
- FieldTrial* trial =
- new FieldTrial(
- name, group_count, default_group_name, next_year_, 12, 31);
+ FieldTrial* trial = FieldTrial::CreateInstance(
+ name, group_count, default_group_name, next_year_, 12, 31);
int winner_index(-2);
std::string winner_name;
@@ -210,7 +207,7 @@ TEST_F(FieldTrialTest, DisableProbability) {
// Create a field trail that has expired.
scoped_refptr<FieldTrial> trial;
- trial = new FieldTrial(
+ trial = FieldTrial::CreateInstance(
name, 1000000000, default_group_name, last_year_, 1, 1);
trial->AppendGroup(loser, 999999999); // 99.9999999% chance of being chosen.
@@ -245,7 +242,7 @@ TEST_F(FieldTrialTest, HashName) {
TEST_F(FieldTrialTest, NameGroupIds) {
std::string no_group("No Group");
uint32 no_group_id = FieldTrial::HashName(no_group);
- scoped_refptr<FieldTrial> trial(new FieldTrial(
+ scoped_refptr<FieldTrial> trial(FieldTrial::CreateInstance(
no_group, 10, "Default", next_year_, 12, 31));
// There is no winner yet, so no NameGroupId should be returned.
@@ -255,7 +252,8 @@ TEST_F(FieldTrialTest, NameGroupIds) {
// Create a single winning group.
std::string one_winner("One Winner");
uint32 one_winner_id = FieldTrial::HashName(one_winner);
- trial = new FieldTrial(one_winner, 10, "Default", next_year_, 12, 31);
+ trial = FieldTrial::CreateInstance(
+ one_winner, 10, "Default", next_year_, 12, 31);
std::string winner("Winner");
uint32 winner_group_id = FieldTrial::HashName(winner);
trial->AppendGroup(winner, 10);
@@ -265,8 +263,8 @@ TEST_F(FieldTrialTest, NameGroupIds) {
std::string multi_group("MultiGroup");
uint32 multi_group_id = FieldTrial::HashName(multi_group);
- scoped_refptr<FieldTrial> multi_group_trial =
- new FieldTrial(multi_group, 9, "Default", next_year_, 12, 31);
+ scoped_refptr<FieldTrial> multi_group_trial = FieldTrial::CreateInstance(
+ multi_group, 9, "Default", next_year_, 12, 31);
multi_group_trial->AppendGroup("Me", 3);
multi_group_trial->AppendGroup("You", 3);
@@ -294,9 +292,8 @@ TEST_F(FieldTrialTest, NameGroupIds) {
TEST_F(FieldTrialTest, Save) {
std::string save_string;
- FieldTrial* trial =
- new FieldTrial(
- "Some name", 10, "Default some name", next_year_, 12, 31);
+ FieldTrial* trial = FieldTrial::CreateInstance(
+ "Some name", 10, "Default some name", next_year_, 12, 31);
// There is no winner yet, so no textual group name is associated with trial.
// In this case, the trial should not be included.
EXPECT_EQ("", trial->group_name_internal());
@@ -311,8 +308,8 @@ TEST_F(FieldTrialTest, Save) {
save_string.clear();
// Create a second trial and winning group.
- FieldTrial* trial2 =
- new FieldTrial("xxx", 10, "Default xxx", next_year_, 12, 31);
+ FieldTrial* trial2 = FieldTrial::CreateInstance(
+ "xxx", 10, "Default xxx", next_year_, 12, 31);
trial2->AppendGroup("yyyy", 10);
FieldTrialList::StatesToString(&save_string);
@@ -324,7 +321,7 @@ TEST_F(FieldTrialTest, Restore) {
EXPECT_TRUE(FieldTrialList::Find("Some_name") == NULL);
EXPECT_TRUE(FieldTrialList::Find("xxx") == NULL);
- FieldTrialList::CreateTrialsInChildProcess("Some_name/Winner/xxx/yyyy/");
+ FieldTrialList::CreateTrialsFromString("Some_name/Winner/xxx/yyyy/");
FieldTrial* trial = FieldTrialList::Find("Some_name");
ASSERT_NE(static_cast<FieldTrial*>(NULL), trial);
@@ -338,28 +335,27 @@ TEST_F(FieldTrialTest, Restore) {
}
TEST_F(FieldTrialTest, BogusRestore) {
- EXPECT_FALSE(FieldTrialList::CreateTrialsInChildProcess("MissingSlash"));
- EXPECT_FALSE(FieldTrialList::CreateTrialsInChildProcess("MissingGroupName/"));
- EXPECT_FALSE(FieldTrialList::CreateTrialsInChildProcess(
+ EXPECT_FALSE(FieldTrialList::CreateTrialsFromString("MissingSlash"));
+ EXPECT_FALSE(FieldTrialList::CreateTrialsFromString("MissingGroupName/"));
+ EXPECT_FALSE(FieldTrialList::CreateTrialsFromString(
"MissingFinalSlash/gname"));
- EXPECT_FALSE(FieldTrialList::CreateTrialsInChildProcess(
+ EXPECT_FALSE(FieldTrialList::CreateTrialsFromString(
"noname, only group/"));
}
TEST_F(FieldTrialTest, DuplicateRestore) {
- FieldTrial* trial =
- new FieldTrial(
- "Some name", 10, "Default some name", next_year_, 12, 31);
+ FieldTrial* trial = FieldTrial::CreateInstance(
+ "Some name", 10, "Default some name", next_year_, 12, 31);
trial->AppendGroup("Winner", 10);
std::string save_string;
FieldTrialList::StatesToString(&save_string);
EXPECT_EQ("Some name/Winner/", save_string);
// It is OK if we redundantly specify a winner.
- EXPECT_TRUE(FieldTrialList::CreateTrialsInChildProcess(save_string));
+ EXPECT_TRUE(FieldTrialList::CreateTrialsFromString(save_string));
// But it is an error to try to change to a different winner.
- EXPECT_FALSE(FieldTrialList::CreateTrialsInChildProcess(
+ EXPECT_FALSE(FieldTrialList::CreateTrialsFromString(
"Some name/Loser/"));
}
@@ -375,9 +371,8 @@ TEST_F(FieldTrialTest, CreateFieldTrial) {
}
TEST_F(FieldTrialTest, DuplicateFieldTrial) {
- FieldTrial* trial =
- new FieldTrial(
- "Some_name", 10, "Default some name", next_year_, 12, 31);
+ FieldTrial* trial = FieldTrial::CreateInstance(
+ "Some_name", 10, "Default some name", next_year_, 12, 31);
trial->AppendGroup("Winner", 10);
// It is OK if we redundantly specify a winner.
@@ -390,8 +385,8 @@ TEST_F(FieldTrialTest, DuplicateFieldTrial) {
}
TEST_F(FieldTrialTest, MakeName) {
- FieldTrial* trial =
- new FieldTrial("Field Trial", 10, "Winner", next_year_, 12, 31);
+ FieldTrial* trial = FieldTrial::CreateInstance(
+ "Field Trial", 10, "Winner", next_year_, 12, 31);
trial->group();
EXPECT_EQ("Histogram_Winner",
FieldTrial::MakeName("Histogram", "Field Trial"));
@@ -467,8 +462,8 @@ TEST_F(FieldTrialTest, UseOneTimeRandomization) {
// _might_ actually give the same result, but we know that given
// the particular client_id we use for unit tests they won't.
scoped_refptr<FieldTrial> trials[] = {
- new FieldTrial("one", 100, "default", next_year_, 1, 1),
- new FieldTrial("two", 100, "default", next_year_, 1, 1),
+ FieldTrial::CreateInstance("one", 100, "default", next_year_, 1, 1),
+ FieldTrial::CreateInstance("two", 100, "default", next_year_, 1, 1),
};
for (size_t i = 0; i < arraysize(trials); ++i) {
@@ -487,7 +482,7 @@ TEST_F(FieldTrialTest, UseOneTimeRandomization) {
TEST_F(FieldTrialTest, DisableImmediately) {
FieldTrial* trial =
- new FieldTrial("trial", 100, "default", next_year_, 12, 31);
+ FieldTrial::CreateInstance("trial", 100, "default", next_year_, 12, 31);
trial->Disable();
ASSERT_EQ("default", trial->group_name());
ASSERT_EQ(FieldTrial::kDefaultGroupNumber, trial->group());
@@ -495,7 +490,7 @@ TEST_F(FieldTrialTest, DisableImmediately) {
TEST_F(FieldTrialTest, DisableAfterInitialization) {
FieldTrial* trial =
- new FieldTrial("trial", 100, "default", next_year_, 12, 31);
+ FieldTrial::CreateInstance("trial", 100, "default", next_year_, 12, 31);
trial->AppendGroup("non_default", 100);
ASSERT_EQ("non_default", trial->group_name());
trial->Disable();

Powered by Google App Engine
This is Rietveld 408576698