Add a getter for search provider logos.

chrome/browser/search_engines/template_url_prepopulate_data.h

 // 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.
 
 #ifndef CHROME_BROWSER_SEARCH_ENGINES_TEMPLATE_URL_PREPOPULATE_DATA_H_
 #define CHROME_BROWSER_SEARCH_ENGINES_TEMPLATE_URL_PREPOPULATE_DATA_H_
 
 #include <stddef.h>
 #include <string>
 #include <vector>
@@ skipping 39 matching lines @@
 // not used.
 TemplateURL* GetPrepopulatedDefaultSearch(Profile* profile);
 
 // Returns the type of the matching engine, or SEARCH_ENGINE_OTHER if no engines
 // match.  This uses same-origin checks unless the |url| is a Google seach URL,
 // in which case we'll identify any valid Google hostname as "Google".
 //
 // NOTE: Must be called on the UI thread.
 SearchEngineType GetEngineType(const std::string& url);
 
+// Returns the logo at the specified |size| for |template_url|.  If no logo is
+// known, this will return an empty GURL.
+//
+// NOTE: Must be called on the UI thread.
+enum LogoSize {

[msw, 2012/08/28 18:22:00]

nit: define the enum at line 24, keep the comment with the function.

[samarth, 2012/08/28 21:22:46]

Done.

+  SIZE_100_PERCENT,

[msw, 2012/08/28 18:22:00]

nit: TemplateURLPrepopulateData::SIZE_100_PERCENT doesn't convey context for the
size, consider renaming the enum values LOGO_[1|2]00_PERCENT.

[samarth, 2012/08/28 21:22:46]

Done.

+  SIZE_200_PERCENT,
+};
+GURL GetLogoURL(const TemplateURL& url, LogoSize size);

[msw, 2012/08/28 18:22:00]

nit: s/url/template_url/ or change the comment's use of "|template_url|".

[samarth, 2012/08/28 21:22:46]

Done.

+
 }  // namespace TemplateURLPrepopulateData
 
 #endif  // CHROME_BROWSER_SEARCH_ENGINES_TEMPLATE_URL_PREPOPULATE_DATA_H_