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

Unified Diff: chrome/browser/google/google_update_settings_unittest.cc

Issue 2814083002: Make the GGRV brand code non-organic. (Closed)
Patch Set: Address review comments Created 3 years, 8 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
« no previous file with comments | « chrome/browser/google/google_brand.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/google/google_update_settings_unittest.cc
diff --git a/chrome/browser/google/google_update_settings_unittest.cc b/chrome/browser/google/google_update_settings_unittest.cc
index 8b3572aa9a62544928f7c5674ff92d929ff3cc74..c78235d16c0dae886d3238587a6f46021175598d 100644
--- a/chrome/browser/google/google_update_settings_unittest.cc
+++ b/chrome/browser/google/google_update_settings_unittest.cc
@@ -50,6 +50,20 @@ TEST_F(GoogleUpdateTest, LastRunTime) {
#endif // defined(OS_WIN)
+TEST_F(GoogleUpdateTest, IsOrganic) {
+ // Test some brand codes to ensure that future changes to this method won't
+ // go unnoticed.
+
+ // GGRV is non-organic.
+ EXPECT_FALSE(google_brand::IsOrganic("GGRV"));
+
+ // Other GGR* are organic.
+ EXPECT_TRUE(google_brand::IsOrganic("GGRA"));
+
+ // GGLS must always be organic.
+ EXPECT_TRUE(google_brand::IsOrganic("GGLS"));
+}
+
TEST_F(GoogleUpdateTest, IsOrganicFirstRunBrandCodes) {
// Test some brand codes to ensure that future changes to this method won't
// go unnoticed.
« no previous file with comments | « chrome/browser/google/google_brand.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698