Add a getter for search provider logos.

chrome/browser/search_engines/template_url_prepopulate_data_unittest.cc

 // Copyright (c) 2012 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 "base/file_util.h"
 #include "base/memory/scoped_vector.h"
 #include "base/scoped_temp_dir.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/search_engines/search_terms_data.h"
 #include "chrome/browser/search_engines/template_url.h"
@@ skipping 158 matching lines @@
   std::string kNigmaURL = "http://www.nigma.ru/?s={searchTerms}&arg1=value1";
   EXPECT_EQ(SEARCH_ENGINE_NIGMA,
             TemplateURLPrepopulateData::GetEngineType(kNigmaURL));
   // Search URL for which no prepopulated search provider exists.
   std::string kExampleSearchURL = "http://example.net/search?q={searchTerms}";
   EXPECT_EQ(SEARCH_ENGINE_OTHER,
             TemplateURLPrepopulateData::GetEngineType(kExampleSearchURL));
   EXPECT_EQ(SEARCH_ENGINE_OTHER,
             TemplateURLPrepopulateData::GetEngineType("invalid:search:url"));
 }
+
+TEST(TemplateURLPrepopulateDataTest, GetLogoURLGoogle100Percent) {
+  TemplateURLData data;
+  data.SetURL("http://www.google.com/");
+  TemplateURL turl(NULL, data);
+  GURL logo_url = TemplateURLPrepopulateData::GetLogoURL(
+      turl, TemplateURLPrepopulateData::LOGO_100_PERCENT);
+
+  EXPECT_EQ("www.google.com", logo_url.host());
+  EXPECT_TRUE(logo_url.SchemeIsSecure());

[Peter Kasting, 2012/08/28 23:20:20]

Why is testing for a secure scheme something the unittest should ensure?  Seems
like this should be omitted.

[samarth, 2012/08/28 23:29:20]

Done.

+}
+
+TEST(TemplateURLPrepopulateDataTest, GetLogoURLGoogle200Percent) {

[Peter Kasting, 2012/08/28 23:20:20]

Nit: Combine these first two tests, a la:

  ...
  GURL logo_100 = ...;
  EXPECT_EQ("www.google.com", logo_100.host());

  GURL logo_200 = ...;
  EXPECT_EQ("www.google.com", logo_200.host());

  EXPECT_NE(logo_100, logo_200);

[samarth, 2012/08/28 23:29:20]

Done.

+  TemplateURLData data;
+  data.SetURL("http://www.google.com/");
+  TemplateURL turl(NULL, data);
+  GURL logo_url = TemplateURLPrepopulateData::GetLogoURL(
+      turl, TemplateURLPrepopulateData::LOGO_200_PERCENT);
+
+  EXPECT_EQ("www.google.com", logo_url.host());
+  EXPECT_TRUE(logo_url.SchemeIsSecure());
+}
+
+TEST(TemplateURLPrepopulateDataTest, GetLogoURLUnknown) {
+  TemplateURLData data;
+  data.SetURL("http://webalta.ru/");
+  TemplateURL turl(NULL, data);
+  GURL logo_url = TemplateURLPrepopulateData::GetLogoURL(
+      turl, TemplateURLPrepopulateData::LOGO_100_PERCENT);
+
+  EXPECT_TRUE(logo_url.is_empty());
+}
+
+TEST(TemplateURLPrepopulateDataTest, GetLogoURLInvalid) {
+  TemplateURLData data;
+  data.SetURL("http://invalid:search:url/");
+  TemplateURL turl(NULL, data);
+  GURL logo_url = TemplateURLPrepopulateData::GetLogoURL(
+      turl, TemplateURLPrepopulateData::LOGO_100_PERCENT);
+
+  EXPECT_TRUE(logo_url.is_empty());
+}