summaryrefslogtreecommitdiffstats
path: root/chrome/browser/cocoa/page_info_window_mac.mm
diff options
context:
space:
mode:
authorrsesek@chromium.org <rsesek@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-10-02 19:08:44 +0000
committerrsesek@chromium.org <rsesek@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-10-02 19:08:44 +0000
commit7fef9717afc4570883a9207fb8ea70f295685488 (patch)
tree232d97320ae035c7cad778335e6b32c1ce1907db /chrome/browser/cocoa/page_info_window_mac.mm
parent4c7003dc599960bb569dd586e3065f755879deb4 (diff)
downloadchromium_src-7fef9717afc4570883a9207fb8ea70f295685488.zip
chromium_src-7fef9717afc4570883a9207fb8ea70f295685488.tar.gz
chromium_src-7fef9717afc4570883a9207fb8ea70f295685488.tar.bz2
Refactor the code for loading icons into the PageInfoModel.
This code is basically duplicated twice on each platform. This consolidates all that into PageInfoModel::GetIconImage() and renames the SectionInfo.state to icon_id. BUG=none TEST=Page info windows and bubbles work as they did before across all platforms. Review URL: http://codereview.chromium.org/3560004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61308 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/cocoa/page_info_window_mac.mm')
-rw-r--r--chrome/browser/cocoa/page_info_window_mac.mm16
1 files changed, 1 insertions, 15 deletions
diff --git a/chrome/browser/cocoa/page_info_window_mac.mm b/chrome/browser/cocoa/page_info_window_mac.mm
index 8ca4c1b..e246c0a 100644
--- a/chrome/browser/cocoa/page_info_window_mac.mm
+++ b/chrome/browser/cocoa/page_info_window_mac.mm
@@ -67,7 +67,6 @@ PageInfoWindowMac::PageInfoWindowMac(PageInfoWindowController* controller,
: controller_(controller),
model_(new PageInfoModel(profile, url, ssl, show_history, this)),
cert_id_(ssl.cert_id()) {
- Init();
}
PageInfoWindowMac::PageInfoWindowMac(PageInfoWindowController* controller,
@@ -75,7 +74,6 @@ PageInfoWindowMac::PageInfoWindowMac(PageInfoWindowController* controller,
: controller_(controller),
model_(model),
cert_id_(0) {
- Init();
}
// static
@@ -98,17 +96,6 @@ void PageInfoWindowMac::ShowPageInfo(gfx::NativeWindow parent,
page_info->Show();
}
-void PageInfoWindowMac::Init() {
- // Load the image refs.
- ResourceBundle& rb = ResourceBundle::GetSharedInstance();
- good_image_.reset([rb.GetNSImageNamed(IDR_PAGEINFO_GOOD) retain]);
- DCHECK_GE(kImageSize, [good_image_ size].width);
- DCHECK_GE(kImageSize, [good_image_ size].height);
- bad_image_.reset([rb.GetNSImageNamed(IDR_PAGEINFO_BAD) retain]);
- DCHECK_GE(kImageSize, [bad_image_ size].width);
- DCHECK_GE(kImageSize, [bad_image_ size].height);
-}
-
PageInfoWindowMac::~PageInfoWindowMac() {
}
@@ -213,8 +200,7 @@ void PageInfoWindowMac::LayoutSections() {
scoped_nsobject<NSImageView> image_view(
[[NSImageView alloc] initWithFrame:image_view_rect]);
[image_view setImageFrameStyle:NSImageFrameNone];
- [image_view setImage:(info.state == PageInfoModel::SECTION_STATE_OK) ?
- good_image_.get() : bad_image_.get()];
+ [image_view setImage:model_->GetIconImage(info.icon_id)];
// Add the box to the list of new subviews.
[box addSubview:image_view.get()];