Add icons to the views Website Settings UI.

chrome/browser/ui/website_settings/website_settings_ui.h

Index: chrome/browser/ui/website_settings/website_settings_ui.h
diff --git a/chrome/browser/ui/website_settings/website_settings_ui.h b/chrome/browser/ui/website_settings/website_settings_ui.h
index 648e5c73d7babf0f1525ed4aafc4c9909ca59fe5..576f086b393d43acb6e4e90dd8228d037439e924 100644
--- a/chrome/browser/ui/website_settings/website_settings_ui.h
+++ b/chrome/browser/ui/website_settings/website_settings_ui.h
@@ -25,6 +25,10 @@ namespace content {
 struct SSLStatus;
 }
 
+namespace gfx {
+class Image;
+}
+
 // The class |WebsiteSettingsUI| specifies the platform independent
 // interface of the website settings UI. The website settings UI displays
 // information and controls for site specific data (local stored objects like
@@ -88,6 +92,18 @@ class WebsiteSettingsUI {
   // Returns the resource ID of the UI string for the given permission |value|.
   static int PermissionValueToUIStringID(ContentSetting value);
 
+  // Returns the icon for the given permission |type| and |setting|.
+  static gfx::Image* GetPermissionIcon(ContentSettingsType type,

[sky, 2012/06/14 15:48:40]

Why are you returning pointers here? Ownership is clearer if you don't and it
also means folks don't have to worry about NULL.

[markusheintz_, 2012/06/14 16:41:13]

I changed it to const refs.

+                                       ContentSetting setting);
+
+  // Returns the identity icon for the given identity |status|.
+  static gfx::Image* GetIdentityIcon(
+      WebsiteSettings::SiteIdentityStatus status);
+
+  // Returns the connection icon for the given connection |status|.
+  static gfx::Image* GetConnectionIcon(
+      WebsiteSettings::SiteConnectionStatus status);
+
   // Sets cookie information.
   virtual void SetCookieInfo(const CookieInfoList& cookie_info_list) = 0;