[email protected] | e2ddbc9 | 2010-10-15 20:02:07 | [diff] [blame] | 1 | // Copyright (c) 2010 The Chromium Authors. All rights reserved. |
| 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 5 | #include "base/string_number_conversions.h" |
[email protected] | a8274453 | 2011-02-11 16:15:53 | [diff] [blame^] | 6 | #include "base/utf_string_conversions.h" |
[email protected] | a314ee5a | 2010-10-26 21:23:28 | [diff] [blame] | 7 | #include "base/values.h" |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 8 | #include "chrome/browser/about_flags.h" |
[email protected] | e2ddbc9 | 2010-10-15 20:02:07 | [diff] [blame] | 9 | #include "chrome/common/chrome_switches.h" |
| 10 | #include "chrome/common/pref_names.h" |
| 11 | #include "chrome/test/testing_pref_service.h" |
[email protected] | a314ee5a | 2010-10-26 21:23:28 | [diff] [blame] | 12 | #include "grit/chromium_strings.h" |
[email protected] | e2ddbc9 | 2010-10-15 20:02:07 | [diff] [blame] | 13 | #include "testing/gtest/include/gtest/gtest.h" |
| 14 | |
[email protected] | a314ee5a | 2010-10-26 21:23:28 | [diff] [blame] | 15 | const char kFlags1[] = "flag1"; |
| 16 | const char kFlags2[] = "flag2"; |
| 17 | const char kFlags3[] = "flag3"; |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 18 | const char kFlags4[] = "flag4"; |
[email protected] | e2ddbc9 | 2010-10-15 20:02:07 | [diff] [blame] | 19 | |
[email protected] | a314ee5a | 2010-10-26 21:23:28 | [diff] [blame] | 20 | const char kSwitch1[] = "switch"; |
| 21 | const char kSwitch2[] = "switch2"; |
| 22 | const char kSwitch3[] = "switch3"; |
[email protected] | a8274453 | 2011-02-11 16:15:53 | [diff] [blame^] | 23 | const char kValueForSwitch2[] = "value_for_switch2"; |
[email protected] | e2ddbc9 | 2010-10-15 20:02:07 | [diff] [blame] | 24 | |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 25 | const char kMultiSwitch1[] = "multi_switch1"; |
| 26 | const char kMultiSwitch2[] = "multi_switch2"; |
[email protected] | a8274453 | 2011-02-11 16:15:53 | [diff] [blame^] | 27 | const char kValueForMultiSwitch2[] = "value_for_multi_switch2"; |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 28 | |
[email protected] | e2ddbc9 | 2010-10-15 20:02:07 | [diff] [blame] | 29 | namespace about_flags { |
| 30 | |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 31 | const Experiment::Choice kMultiChoices[] = { |
[email protected] | a8274453 | 2011-02-11 16:15:53 | [diff] [blame^] | 32 | { IDS_PRODUCT_NAME, "", "" }, |
| 33 | { IDS_PRODUCT_NAME, kMultiSwitch1, "" }, |
| 34 | { IDS_PRODUCT_NAME, kMultiSwitch2, kValueForMultiSwitch2 }, |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 35 | }; |
| 36 | |
| 37 | // The experiments that are set for these tests. The 3rd experiment is not |
| 38 | // supported on the current platform, all others are. |
[email protected] | a314ee5a | 2010-10-26 21:23:28 | [diff] [blame] | 39 | static Experiment kExperiments[] = { |
| 40 | { |
| 41 | kFlags1, |
| 42 | IDS_PRODUCT_NAME, |
| 43 | IDS_PRODUCT_NAME, |
| 44 | 0, // Ends up being mapped to the current platform. |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 45 | Experiment::SINGLE_VALUE, |
| 46 | kSwitch1, |
[email protected] | a8274453 | 2011-02-11 16:15:53 | [diff] [blame^] | 47 | "", |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 48 | NULL, |
| 49 | 0 |
[email protected] | a314ee5a | 2010-10-26 21:23:28 | [diff] [blame] | 50 | }, |
| 51 | { |
| 52 | kFlags2, |
| 53 | IDS_PRODUCT_NAME, |
| 54 | IDS_PRODUCT_NAME, |
| 55 | 0, // Ends up being mapped to the current platform. |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 56 | Experiment::SINGLE_VALUE, |
| 57 | kSwitch2, |
[email protected] | a8274453 | 2011-02-11 16:15:53 | [diff] [blame^] | 58 | kValueForSwitch2, |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 59 | NULL, |
| 60 | 0 |
[email protected] | a314ee5a | 2010-10-26 21:23:28 | [diff] [blame] | 61 | }, |
| 62 | { |
| 63 | kFlags3, |
| 64 | IDS_PRODUCT_NAME, |
| 65 | IDS_PRODUCT_NAME, |
| 66 | 0, // This ends up enabling for an OS other than the current. |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 67 | Experiment::SINGLE_VALUE, |
| 68 | kSwitch3, |
[email protected] | a8274453 | 2011-02-11 16:15:53 | [diff] [blame^] | 69 | "", |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 70 | NULL, |
| 71 | 0 |
| 72 | }, |
| 73 | { |
| 74 | kFlags4, |
| 75 | IDS_PRODUCT_NAME, |
| 76 | IDS_PRODUCT_NAME, |
| 77 | 0, // Ends up being mapped to the current platform. |
| 78 | Experiment::MULTI_VALUE, |
| 79 | "", |
[email protected] | a8274453 | 2011-02-11 16:15:53 | [diff] [blame^] | 80 | "", |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 81 | kMultiChoices, |
| 82 | arraysize(kMultiChoices) |
[email protected] | a314ee5a | 2010-10-26 21:23:28 | [diff] [blame] | 83 | }, |
| 84 | }; |
| 85 | |
[email protected] | e2ddbc9 | 2010-10-15 20:02:07 | [diff] [blame] | 86 | class AboutFlagsTest : public ::testing::Test { |
| 87 | protected: |
| 88 | AboutFlagsTest() { |
| 89 | prefs_.RegisterListPref(prefs::kEnabledLabsExperiments); |
[email protected] | ba816424 | 2010-11-16 21:31:00 | [diff] [blame] | 90 | #if defined(OS_CHROMEOS) |
| 91 | prefs_.RegisterBooleanPref(prefs::kLabsMediaplayerEnabled, false); |
| 92 | prefs_.RegisterBooleanPref(prefs::kLabsAdvancedFilesystemEnabled, false); |
| 93 | prefs_.RegisterBooleanPref(prefs::kUseVerticalTabs, false); |
| 94 | #endif |
[email protected] | e2ddbc9 | 2010-10-15 20:02:07 | [diff] [blame] | 95 | testing::ClearState(); |
| 96 | } |
| 97 | |
[email protected] | a314ee5a | 2010-10-26 21:23:28 | [diff] [blame] | 98 | virtual void SetUp() { |
| 99 | for (size_t i = 0; i < arraysize(kExperiments); ++i) |
| 100 | kExperiments[i].supported_platforms = GetCurrentPlatform(); |
| 101 | |
| 102 | int os_other_than_current = 1; |
| 103 | while (os_other_than_current == GetCurrentPlatform()) |
| 104 | os_other_than_current <<= 1; |
| 105 | kExperiments[2].supported_platforms = os_other_than_current; |
| 106 | |
| 107 | testing::SetExperiments(kExperiments, arraysize(kExperiments)); |
| 108 | } |
| 109 | |
| 110 | virtual void TearDown() { |
| 111 | testing::SetExperiments(NULL, 0); |
| 112 | } |
| 113 | |
[email protected] | e2ddbc9 | 2010-10-15 20:02:07 | [diff] [blame] | 114 | TestingPrefService prefs_; |
| 115 | }; |
| 116 | |
| 117 | TEST_F(AboutFlagsTest, ChangeNeedsRestart) { |
[email protected] | e2ddbc9 | 2010-10-15 20:02:07 | [diff] [blame] | 118 | EXPECT_FALSE(IsRestartNeededToCommitChanges()); |
| 119 | SetExperimentEnabled(&prefs_, kFlags1, true); |
| 120 | EXPECT_TRUE(IsRestartNeededToCommitChanges()); |
| 121 | } |
| 122 | |
| 123 | TEST_F(AboutFlagsTest, AddTwoFlagsRemoveOne) { |
[email protected] | e2ddbc9 | 2010-10-15 20:02:07 | [diff] [blame] | 124 | // Add two experiments, check they're there. |
| 125 | SetExperimentEnabled(&prefs_, kFlags1, true); |
| 126 | SetExperimentEnabled(&prefs_, kFlags2, true); |
| 127 | |
| 128 | ListValue* experiments_list = prefs_.GetMutableList( |
| 129 | prefs::kEnabledLabsExperiments); |
| 130 | ASSERT_TRUE(experiments_list != NULL); |
| 131 | |
| 132 | ASSERT_EQ(2u, experiments_list->GetSize()); |
| 133 | |
| 134 | std::string s0; |
| 135 | ASSERT_TRUE(experiments_list->GetString(0, &s0)); |
| 136 | std::string s1; |
| 137 | ASSERT_TRUE(experiments_list->GetString(1, &s1)); |
| 138 | |
| 139 | EXPECT_TRUE(s0 == kFlags1 || s1 == kFlags1); |
| 140 | EXPECT_TRUE(s0 == kFlags2 || s1 == kFlags2); |
| 141 | |
| 142 | // Remove one experiment, check the other's still around. |
| 143 | SetExperimentEnabled(&prefs_, kFlags2, false); |
| 144 | |
| 145 | experiments_list = prefs_.GetMutableList(prefs::kEnabledLabsExperiments); |
| 146 | ASSERT_TRUE(experiments_list != NULL); |
| 147 | ASSERT_EQ(1u, experiments_list->GetSize()); |
| 148 | ASSERT_TRUE(experiments_list->GetString(0, &s0)); |
| 149 | EXPECT_TRUE(s0 == kFlags1); |
| 150 | } |
| 151 | |
| 152 | TEST_F(AboutFlagsTest, AddTwoFlagsRemoveBoth) { |
[email protected] | e2ddbc9 | 2010-10-15 20:02:07 | [diff] [blame] | 153 | // Add two experiments, check the pref exists. |
| 154 | SetExperimentEnabled(&prefs_, kFlags1, true); |
| 155 | SetExperimentEnabled(&prefs_, kFlags2, true); |
| 156 | ListValue* experiments_list = prefs_.GetMutableList( |
| 157 | prefs::kEnabledLabsExperiments); |
| 158 | ASSERT_TRUE(experiments_list != NULL); |
| 159 | |
| 160 | // Remove both, the pref should have been removed completely. |
| 161 | SetExperimentEnabled(&prefs_, kFlags1, false); |
| 162 | SetExperimentEnabled(&prefs_, kFlags2, false); |
| 163 | experiments_list = prefs_.GetMutableList(prefs::kEnabledLabsExperiments); |
| 164 | EXPECT_TRUE(experiments_list == NULL || experiments_list->GetSize() == 0); |
| 165 | } |
| 166 | |
| 167 | TEST_F(AboutFlagsTest, ConvertFlagsToSwitches) { |
[email protected] | e2ddbc9 | 2010-10-15 20:02:07 | [diff] [blame] | 168 | SetExperimentEnabled(&prefs_, kFlags1, true); |
| 169 | |
[email protected] | 947446b | 2010-10-21 03:36:31 | [diff] [blame] | 170 | CommandLine command_line(CommandLine::NO_PROGRAM); |
[email protected] | e2ddbc9 | 2010-10-15 20:02:07 | [diff] [blame] | 171 | command_line.AppendSwitch("foo"); |
| 172 | |
| 173 | EXPECT_TRUE(command_line.HasSwitch("foo")); |
[email protected] | a314ee5a | 2010-10-26 21:23:28 | [diff] [blame] | 174 | EXPECT_FALSE(command_line.HasSwitch(kSwitch1)); |
[email protected] | e2ddbc9 | 2010-10-15 20:02:07 | [diff] [blame] | 175 | |
| 176 | ConvertFlagsToSwitches(&prefs_, &command_line); |
| 177 | |
| 178 | EXPECT_TRUE(command_line.HasSwitch("foo")); |
[email protected] | a314ee5a | 2010-10-26 21:23:28 | [diff] [blame] | 179 | EXPECT_TRUE(command_line.HasSwitch(kSwitch1)); |
[email protected] | e2ddbc9 | 2010-10-15 20:02:07 | [diff] [blame] | 180 | } |
| 181 | |
| 182 | TEST_F(AboutFlagsTest, RemoveFlagSwitches) { |
[email protected] | e2ddbc9 | 2010-10-15 20:02:07 | [diff] [blame] | 183 | std::map<std::string, CommandLine::StringType> switch_list; |
[email protected] | a314ee5a | 2010-10-26 21:23:28 | [diff] [blame] | 184 | switch_list[kSwitch1] = CommandLine::StringType(); |
[email protected] | e2ddbc9 | 2010-10-15 20:02:07 | [diff] [blame] | 185 | switch_list[switches::kFlagSwitchesBegin] = CommandLine::StringType(); |
| 186 | switch_list[switches::kFlagSwitchesEnd] = CommandLine::StringType(); |
| 187 | switch_list["foo"] = CommandLine::StringType(); |
| 188 | |
| 189 | SetExperimentEnabled(&prefs_, kFlags1, true); |
| 190 | |
| 191 | // This shouldn't do anything before ConvertFlagsToSwitches() wasn't called. |
| 192 | RemoveFlagsSwitches(&switch_list); |
| 193 | ASSERT_EQ(4u, switch_list.size()); |
[email protected] | a314ee5a | 2010-10-26 21:23:28 | [diff] [blame] | 194 | EXPECT_TRUE(switch_list.find(kSwitch1) != switch_list.end()); |
[email protected] | e2ddbc9 | 2010-10-15 20:02:07 | [diff] [blame] | 195 | EXPECT_TRUE(switch_list.find(switches::kFlagSwitchesBegin) != |
| 196 | switch_list.end()); |
| 197 | EXPECT_TRUE(switch_list.find(switches::kFlagSwitchesEnd) != |
| 198 | switch_list.end()); |
| 199 | EXPECT_TRUE(switch_list.find("foo") != switch_list.end()); |
| 200 | |
| 201 | // Call ConvertFlagsToSwitches(), then RemoveFlagsSwitches() again. |
[email protected] | 947446b | 2010-10-21 03:36:31 | [diff] [blame] | 202 | CommandLine command_line(CommandLine::NO_PROGRAM); |
[email protected] | e2ddbc9 | 2010-10-15 20:02:07 | [diff] [blame] | 203 | command_line.AppendSwitch("foo"); |
| 204 | ConvertFlagsToSwitches(&prefs_, &command_line); |
| 205 | RemoveFlagsSwitches(&switch_list); |
| 206 | |
| 207 | // Now the about:flags-related switch should have been removed. |
| 208 | ASSERT_EQ(1u, switch_list.size()); |
| 209 | EXPECT_TRUE(switch_list.find("foo") != switch_list.end()); |
| 210 | } |
| 211 | |
[email protected] | a314ee5a | 2010-10-26 21:23:28 | [diff] [blame] | 212 | // Tests enabling experiments that aren't supported on the current platform. |
| 213 | TEST_F(AboutFlagsTest, PersistAndPrune) { |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 214 | // Enable experiments 1 and 3. |
[email protected] | a314ee5a | 2010-10-26 21:23:28 | [diff] [blame] | 215 | SetExperimentEnabled(&prefs_, kFlags1, true); |
| 216 | SetExperimentEnabled(&prefs_, kFlags3, true); |
| 217 | CommandLine command_line(CommandLine::NO_PROGRAM); |
| 218 | EXPECT_FALSE(command_line.HasSwitch(kSwitch1)); |
| 219 | EXPECT_FALSE(command_line.HasSwitch(kSwitch3)); |
| 220 | |
| 221 | // Convert the flags to switches. Experiment 3 shouldn't be among the switches |
| 222 | // as it is not applicable to the current platform. |
| 223 | ConvertFlagsToSwitches(&prefs_, &command_line); |
| 224 | EXPECT_TRUE(command_line.HasSwitch(kSwitch1)); |
| 225 | EXPECT_FALSE(command_line.HasSwitch(kSwitch3)); |
| 226 | |
| 227 | // Experiment 3 should show still be persisted in preferences though. |
| 228 | scoped_ptr<ListValue> switch_prefs(GetFlagsExperimentsData(&prefs_)); |
| 229 | ASSERT_TRUE(switch_prefs.get()); |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 230 | EXPECT_EQ(arraysize(kExperiments) - 1, switch_prefs->GetSize()); |
| 231 | } |
| 232 | |
[email protected] | a8274453 | 2011-02-11 16:15:53 | [diff] [blame^] | 233 | // Tests that switches which should have values get them in the command |
| 234 | // line. |
| 235 | TEST_F(AboutFlagsTest, CheckValues) { |
| 236 | // Enable experiments 1 and 2. |
| 237 | SetExperimentEnabled(&prefs_, kFlags1, true); |
| 238 | SetExperimentEnabled(&prefs_, kFlags2, true); |
| 239 | CommandLine command_line(CommandLine::NO_PROGRAM); |
| 240 | EXPECT_FALSE(command_line.HasSwitch(kSwitch1)); |
| 241 | EXPECT_FALSE(command_line.HasSwitch(kSwitch2)); |
| 242 | |
| 243 | // Convert the flags to switches. |
| 244 | ConvertFlagsToSwitches(&prefs_, &command_line); |
| 245 | EXPECT_TRUE(command_line.HasSwitch(kSwitch1)); |
| 246 | EXPECT_EQ(std::string(""), |
| 247 | command_line.GetSwitchValueASCII(kSwitch1)); |
| 248 | EXPECT_TRUE(command_line.HasSwitch(kSwitch2)); |
| 249 | EXPECT_EQ(std::string(kValueForSwitch2), |
| 250 | command_line.GetSwitchValueASCII(kSwitch2)); |
| 251 | |
| 252 | // Confirm that there is no '=' in the command line for simple switches. |
| 253 | std::string switch1_with_equals = std::string("--") + |
| 254 | std::string(kSwitch1) + |
| 255 | std::string("="); |
| 256 | #if defined(OS_WIN) |
| 257 | EXPECT_EQ(std::wstring::npos, |
| 258 | command_line.command_line_string().find( |
| 259 | ASCIIToWide(switch1_with_equals))); |
| 260 | #else |
| 261 | EXPECT_EQ(std::string::npos, |
| 262 | command_line.command_line_string().find(switch1_with_equals)); |
| 263 | #endif |
| 264 | |
| 265 | // And confirm there is a '=' for switches with values. |
| 266 | std::string switch2_with_equals = std::string("--") + |
| 267 | std::string(kSwitch2) + |
| 268 | std::string("="); |
| 269 | #if defined(OS_WIN) |
| 270 | EXPECT_NE(std::wstring::npos, |
| 271 | command_line.command_line_string().find( |
| 272 | ASCIIToWide(switch2_with_equals))); |
| 273 | #else |
| 274 | EXPECT_NE(std::string::npos, |
| 275 | command_line.command_line_string().find(switch2_with_equals)); |
| 276 | #endif |
| 277 | |
| 278 | // And it should persist |
| 279 | scoped_ptr<ListValue> switch_prefs(GetFlagsExperimentsData(&prefs_)); |
| 280 | ASSERT_TRUE(switch_prefs.get()); |
| 281 | EXPECT_EQ(arraysize(kExperiments) - 1, switch_prefs->GetSize()); |
| 282 | } |
| 283 | |
[email protected] | 28e35af | 2011-02-09 12:56:22 | [diff] [blame] | 284 | // Tests multi-value type experiments. |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 285 | TEST_F(AboutFlagsTest, MultiValues) { |
[email protected] | 28e35af | 2011-02-09 12:56:22 | [diff] [blame] | 286 | // Initially, the first "deactivated" option of the multi experiment should |
| 287 | // be set. |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 288 | { |
| 289 | CommandLine command_line(CommandLine::NO_PROGRAM); |
| 290 | ConvertFlagsToSwitches(&prefs_, &command_line); |
[email protected] | 28e35af | 2011-02-09 12:56:22 | [diff] [blame] | 291 | EXPECT_FALSE(command_line.HasSwitch(kMultiSwitch1)); |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 292 | EXPECT_FALSE(command_line.HasSwitch(kMultiSwitch2)); |
| 293 | } |
| 294 | |
[email protected] | 28e35af | 2011-02-09 12:56:22 | [diff] [blame] | 295 | // Enable the 2nd choice of the multi-value. |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 296 | SetExperimentEnabled(&prefs_, std::string(kFlags4) + |
| 297 | std::string(testing::kMultiSeparator) + |
[email protected] | 28e35af | 2011-02-09 12:56:22 | [diff] [blame] | 298 | base::IntToString(2), true); |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 299 | { |
| 300 | CommandLine command_line(CommandLine::NO_PROGRAM); |
| 301 | ConvertFlagsToSwitches(&prefs_, &command_line); |
| 302 | EXPECT_FALSE(command_line.HasSwitch(kMultiSwitch1)); |
| 303 | EXPECT_TRUE(command_line.HasSwitch(kMultiSwitch2)); |
[email protected] | a8274453 | 2011-02-11 16:15:53 | [diff] [blame^] | 304 | EXPECT_EQ(std::string(kValueForMultiSwitch2), |
| 305 | command_line.GetSwitchValueASCII(kMultiSwitch2)); |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 306 | } |
| 307 | |
| 308 | // Disable the multi-value experiment. |
[email protected] | 28e35af | 2011-02-09 12:56:22 | [diff] [blame] | 309 | SetExperimentEnabled(&prefs_, std::string(kFlags4) + |
| 310 | std::string(testing::kMultiSeparator) + |
| 311 | base::IntToString(0), true); |
[email protected] | 8a6ff28d | 2010-12-02 16:35:19 | [diff] [blame] | 312 | { |
| 313 | CommandLine command_line(CommandLine::NO_PROGRAM); |
| 314 | ConvertFlagsToSwitches(&prefs_, &command_line); |
| 315 | EXPECT_FALSE(command_line.HasSwitch(kMultiSwitch1)); |
| 316 | EXPECT_FALSE(command_line.HasSwitch(kMultiSwitch2)); |
| 317 | } |
| 318 | } |
| 319 | |
| 320 | // Makes sure there are no separators in any of the experiment names. |
| 321 | TEST_F(AboutFlagsTest, NoSeparators) { |
| 322 | testing::SetExperiments(NULL, 0); |
| 323 | size_t count; |
| 324 | const Experiment* experiments = testing::GetExperiments(&count); |
| 325 | for (size_t i = 0; i < count; ++i) { |
| 326 | std::string name = experiments->internal_name; |
| 327 | EXPECT_EQ(std::string::npos, name.find(testing::kMultiSeparator)) << i; |
| 328 | } |
[email protected] | a314ee5a | 2010-10-26 21:23:28 | [diff] [blame] | 329 | } |
| 330 | |
[email protected] | e2ddbc9 | 2010-10-15 20:02:07 | [diff] [blame] | 331 | } // namespace about_flags |