Add a getter for search provider logos.

chrome/browser/search_engines/template_url_prepopulate_data_unittest.cc

Index: chrome/browser/search_engines/template_url_prepopulate_data_unittest.cc
diff --git a/chrome/browser/search_engines/template_url_prepopulate_data_unittest.cc b/chrome/browser/search_engines/template_url_prepopulate_data_unittest.cc
index f0e33c0c15b153c6d053000d4000beccd2aa5798..c53230704d8e4d61cd6e3bd0ebc61a8b169d81fe 100644
--- a/chrome/browser/search_engines/template_url_prepopulate_data_unittest.cc
+++ b/chrome/browser/search_engines/template_url_prepopulate_data_unittest.cc
@@ -176,3 +176,23 @@ TEST_F(TemplateURLPrepopulateDataTest, GetEngineTypeAdvanced) {
   EXPECT_EQ(SEARCH_ENGINE_OTHER,
             TemplateURLPrepopulateData::GetEngineType("invalid:search:url"));
 }
+
+TEST(TemplateURLPrepopulateDataTest, GetLogoURLGoogle) {
+  EXPECT_EQ("www.google.com",

[msw, 2012/08/27 21:57:26]

nit: get each logo URL once and test the host/scheme on locals.

[samarth, 2012/08/28 16:18:15]

Done.

+            TemplateURLPrepopulateData::GetLogoURL(
+                "http://www.google.com/").host());
+  EXPECT_EQ("www.google.com",
+            TemplateURLPrepopulateData::GetLogo200PercentURL(
+                "http://www.google.com/").host());
+  EXPECT_TRUE(TemplateURLPrepopulateData::GetLogoURL(
+      "http://www.google.com/").SchemeIsSecure());
+  EXPECT_TRUE(TemplateURLPrepopulateData::GetLogo200PercentURL(
+      "http://www.google.com/").SchemeIsSecure());
+}
+
+TEST(TemplateURLPrepopulateDataTest, GetLogoURLUnknown) {
+  EXPECT_TRUE(TemplateURLPrepopulateData::GetLogoURL(
+      "http://www.yahoo.com/").is_empty());

[msw, 2012/08/27 21:57:26]

nit: this will fail if/when yahoo is added, use a fake/obscure domain.

[samarth, 2012/08/28 16:18:15]

Done.

+  EXPECT_TRUE(TemplateURLPrepopulateData::GetLogoURL(
+      "invalid:search:url").is_empty());
+}