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

Unified Diff: chrome/browser/ui/search/search_unittest.cc

Issue 11743035: Disable the embedded search field trials for users with themes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 12 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: chrome/browser/ui/search/search_unittest.cc
diff --git a/chrome/browser/ui/search/search_unittest.cc b/chrome/browser/ui/search/search_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..fc0d65629778facc1b30b7e4dcea27c87e947381
--- /dev/null
+++ b/chrome/browser/ui/search/search_unittest.cc
@@ -0,0 +1,66 @@
+// Copyright (c) 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/ui/search/search.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace chrome {
+namespace search {
+
+TEST(EmbeddedSearchFieldTrialTest, GetFieldTrialInfo) {
+ chrome::search::FieldTrialFlags flags;
dhollowa 2013/01/04 17:11:49 nit: Omit the chrome::search:: prefix here and thr
David Black 2013/01/04 19:55:32 Done.
+ uint64 group_number = 0;
+ const uint64 ZERO = 0;
+
+ chrome::search::GetFieldTrialInfo("", &flags, &group_number);
+ EXPECT_EQ(ZERO, group_number);
+ EXPECT_EQ(ZERO, flags.size());
+
+ chrome::search::GetFieldTrialInfo("Group77", &flags, &group_number);
+ EXPECT_EQ(uint64(77), group_number);
+ EXPECT_EQ(ZERO, flags.size());
+
+ group_number = 0;
+ chrome::search::GetFieldTrialInfo("Group77.2", &flags, &group_number);
+ EXPECT_EQ(ZERO, group_number);
+ EXPECT_EQ(ZERO, flags.size());
+
+ chrome::search::GetFieldTrialInfo("Invalid77", &flags, &group_number);
+ EXPECT_EQ(ZERO, group_number);
+ EXPECT_EQ(ZERO, flags.size());
+
+ chrome::search::GetFieldTrialInfo("Group77 ", &flags, &group_number);
+ EXPECT_EQ(uint64(77), group_number);
+ EXPECT_EQ(ZERO, flags.size());
+
+ group_number = 0;
+ flags.clear();
+ EXPECT_EQ(uint64(9999), GetUInt64ValueForFlagWithDefault("foo", 9999, flags));
+ chrome::search::GetFieldTrialInfo("Group77 foo:6", &flags, &group_number);
+ EXPECT_EQ(uint64(77), group_number);
+ EXPECT_EQ(uint64(1), flags.size());
+ EXPECT_EQ(uint64(6), GetUInt64ValueForFlagWithDefault("foo", 9999, flags));
+
+ group_number = 0;
+ flags.clear();
+ chrome::search::GetFieldTrialInfo(
+ "Group77 bar:1 baz:7 cat:dogs", &flags, &group_number);
+ EXPECT_EQ(uint64(77), group_number);
+ EXPECT_EQ(uint64(3), flags.size());
+ EXPECT_EQ(true, GetBoolValueForFlagWithDefault("bar", false, flags));
+ EXPECT_EQ(uint64(7), GetUInt64ValueForFlagWithDefault("baz", 0, flags));
+ EXPECT_EQ("dogs", GetStringValueForFlagWithDefault("cat", "", flags));
+ EXPECT_EQ("default", GetStringValueForFlagWithDefault(
+ "moose", "default", flags));
+
+ group_number = 0;
+ flags.clear();
+ chrome::search::GetFieldTrialInfo(
+ "Group77 bar:1 baz:7 cat:dogs DISABLED", &flags, &group_number);
+ EXPECT_EQ(ZERO, group_number);
+ EXPECT_EQ(ZERO, flags.size());
+}
+
+} // namespace search
+} // namespace chrome

Powered by Google App Engine
This is Rietveld 408576698