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

Unified Diff: chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.h

Issue 2285683003: Material Page Info (Mac, 2/3): Update security section. (Closed)
Patch Set: Remove obsolete tests. Created 4 years, 3 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
« no previous file with comments | « no previous file | chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.h
diff --git a/chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.h b/chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.h
index 0112b9d86dac6637fe2253d50379fb07b77e89d3..e354afe73d7c5d56011dd426fe0fa6c51f9a4913 100644
--- a/chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.h
+++ b/chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.h
@@ -37,12 +37,14 @@ class X509Certificate;
// The main content view for the Permissions tab.
NSView* securitySectionView_;
- // Displays the web site identity.
- NSTextField* identityField_;
-
- // Displays the security summary for the page (private/not private/etc.).
+ // Displays the short security summary for the page
+ // (private/not private/etc.).
NSTextField* securitySummaryField_;
+ // Displays a longer explanation of the page's security state, and how the
+ // user should treat it.
+ NSTextField* securityDetailsField_;
+
// The link button for opening security details for the page. This is the
// DevTools Security panel for most users, but may be the certificate viewer
// for enterprise users with DevTools disabled.
@@ -51,6 +53,16 @@ class X509Certificate;
// URL of the page for which the bubble is shown.
GURL url_;
+ // Displays a paragraph to accompany the reset decisions button, explaining
+ // that the user has made a decision to trust an invalid security certificate
+ // for the current site.
+ // This field only shows when there is an acrive certificate exception.
+ NSTextField* resetDecisionsField_;
+
+ // The link button for revoking certificate decisions.
+ // This link only shows when there is an acrive certificate exception.
+ NSButton* resetDecisionsButton_;
+
// Whether DevTools is disabled for the relevant profile.
BOOL isDevToolsDisabled_;
@@ -58,9 +70,6 @@ class X509Certificate;
// non-null on a cryptographic connection, and null otherwise.
scoped_refptr<net::X509Certificate> certificate_;
- // The link button for revoking certificate decisions.
- NSButton* resetDecisionsButton_;
-
// Separator line.
NSView* separatorAfterSecuritySection_;
« no previous file with comments | « no previous file | chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698