Show all experiments, even those that are unavailable on the current platform

chrome/browser/about_flags_unittest.cc

Index: chrome/browser/about_flags_unittest.cc
diff --git a/chrome/browser/about_flags_unittest.cc b/chrome/browser/about_flags_unittest.cc
index a58cb466d95ec3bbedbe764f1ae7f72b5701355b..2cde57e045f187e4490612ba9f640d4a4582fcbb 100644
--- a/chrome/browser/about_flags_unittest.cc
+++ b/chrome/browser/about_flags_unittest.cc
@@ -223,7 +223,7 @@ TEST_F(AboutFlagsTest, PersistAndPrune) {
   // Experiment 3 should show still be persisted in preferences though.
   scoped_ptr<ListValue> switch_prefs(GetFlagsExperimentsData(&prefs_));
   ASSERT_TRUE(switch_prefs.get());
-  EXPECT_EQ(arraysize(kExperiments) - 1, switch_prefs->GetSize());
+  EXPECT_EQ(arraysize(kExperiments), switch_prefs->GetSize());

[Tyler Breisacher (Chromium), 2011/12/19 23:23:26]

The kExperiments array used for testing has a few experiments that are available
on the current platform, and 1 that is not. So GetFlagsExperimentsData would
always return a list whose length is one less than kExperiments. Now,
GetFlagsExperimentsData always returns *all* experiments, so the sizes should
match exactly. The same thing applies at line 276.

 }
 
 // Tests that switches which should have values get them in the command
@@ -273,7 +273,7 @@ TEST_F(AboutFlagsTest, CheckValues) {
   // And it should persist
   scoped_ptr<ListValue> switch_prefs(GetFlagsExperimentsData(&prefs_));
   ASSERT_TRUE(switch_prefs.get());
-  EXPECT_EQ(arraysize(kExperiments) - 1, switch_prefs->GetSize());
+  EXPECT_EQ(arraysize(kExperiments), switch_prefs->GetSize());
 }
 
 // Tests multi-value type experiments.