Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(823)

Unified Diff: chrome/browser/cocoa/location_bar_view_mac.h

Issue 1056002: Omnibox M5 work, part 1: Security changes... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/cocoa/location_bar_view_mac.h
===================================================================
--- chrome/browser/cocoa/location_bar_view_mac.h (revision 42491)
+++ chrome/browser/cocoa/location_bar_view_mac.h (working copy)
@@ -177,31 +177,24 @@
// current URL's scheme is https.
class SecurityImageView : public LocationBarImageView {
public:
- enum Image {
- LOCK = 0,
- WARNING
- };
-
SecurityImageView(LocationBarViewMac* owner,
Profile* profile,
ToolbarModel* model);
virtual ~SecurityImageView();
// Sets the image to the appropriate icon.
- void SetImageShown(Image image);
+ void SetImageShown(int resource_id);
// Shows the page info dialog.
virtual void OnMousePressed(NSRect bounds);
private:
- // The lock icon shown when using HTTPS. Loaded lazily, the first time it's
- // needed.
- scoped_nsobject<NSImage> lock_icon_;
+ // Icons for various states. Lazily loaded the first time they're needed.
+ scoped_nsobject<NSImage> ev_secure_icon_;
+ scoped_nsobject<NSImage> secure_icon_;
+ scoped_nsobject<NSImage> security_warning_icon_;
+ scoped_nsobject<NSImage> security_error_icon_;
- // The warning icon shown when HTTPS is broken. Loaded lazily, the first
- // time it's needed.
- scoped_nsobject<NSImage> warning_icon_;
-
// The location bar view that owns us.
LocationBarViewMac* owner_;
@@ -382,7 +375,7 @@
private:
// Sets the SSL icon we should be showing.
- void SetSecurityIcon(ToolbarModel::Icon icon);
+ void SetSecurityIcon(int resource_id);
// Sets the label for the SSL icon.
void SetSecurityIconLabel();
« no previous file with comments | « chrome/browser/cocoa/autocomplete_text_field_unittest.mm ('k') | chrome/browser/cocoa/location_bar_view_mac.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698