Safe browsing subresource filter bubble UI skeleton.

chrome/browser/ui/content_settings/content_setting_image_model.h

Index: chrome/browser/ui/content_settings/content_setting_image_model.h
diff --git a/chrome/browser/ui/content_settings/content_setting_image_model.h b/chrome/browser/ui/content_settings/content_setting_image_model.h
index 7daf444a9918160213a16175806118d14ff7b84d..33011cbe5baac6e43fecf8e238f6a9eb126fff13 100644
--- a/chrome/browser/ui/content_settings/content_setting_image_model.h
+++ b/chrome/browser/ui/content_settings/content_setting_image_model.h
@@ -65,6 +65,9 @@ class ContentSettingImageModel {
   int explanatory_string_id() const { return explanatory_string_id_; }
   const base::string16& get_tooltip() const { return tooltip_; }
 
+  static std::unique_ptr<ContentSettingImageModel>
+  CreateSubresourceFilterImageModelForTesting();

[msramek, 2016/07/27 09:46:22]

I think this fundamentally doesn't belong here. We should probably pull out the
class declaration here to the .cc file, same way as we did for
ContentSettingSimpleImageModel.

Basically, we could get away with hiding most class declarations in the .cc file
because ContentSettingSimpleImageModel knows how to construct them from
ContentSettingsType, but that's not possible here. So I believe your class
declaration should be public in the .h file.

[melandory, 2016/07/27 12:53:41]

Done.

+
  protected:
   ContentSettingImageModel();
   void SetIconByResourceId(int id);