diff options
author | rsesek@chromium.org <rsesek@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-02 19:08:44 +0000 |
---|---|---|
committer | rsesek@chromium.org <rsesek@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-02 19:08:44 +0000 |
commit | 7fef9717afc4570883a9207fb8ea70f295685488 (patch) | |
tree | 232d97320ae035c7cad778335e6b32c1ce1907db /chrome/browser/cocoa/page_info_window_mac.mm | |
parent | 4c7003dc599960bb569dd586e3065f755879deb4 (diff) | |
download | chromium_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.mm | 16 |
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()]; |