Add a getter for search provider logos.

chrome/browser/search_engines/template_url_prepopulate_data.h

Index: chrome/browser/search_engines/template_url_prepopulate_data.h
diff --git a/chrome/browser/search_engines/template_url_prepopulate_data.h b/chrome/browser/search_engines/template_url_prepopulate_data.h
index 4d676fb6fe841c313695f930bd56e1f9baf00ef7..5719adfef2e2a67f28e61c7ec9f70f95d546c450 100644
--- a/chrome/browser/search_engines/template_url_prepopulate_data.h
+++ b/chrome/browser/search_engines/template_url_prepopulate_data.h
@@ -57,6 +57,16 @@ TemplateURL* GetPrepopulatedDefaultSearch(Profile* profile);
 // 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_