Convert OSX Page Info Bubble to Material Design

chrome/browser/ui/website_settings/website_settings_ui.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_WEBSITE_SETTINGS_WEBSITE_SETTINGS_UI_H_
 #define CHROME_BROWSER_UI_WEBSITE_SETTINGS_WEBSITE_SETTINGS_UI_H_
 
 #include <memory>
 #include <string>
 #include <vector>
@@ skipping 26 matching lines @@
 class WebsiteSettingsUI {
  public:
   // The Website Settings UI contains several tabs. Each tab is associated with
   // a unique tab id. The enum |TabId| contains all the ids for the tabs.
   enum TabId {
     TAB_ID_PERMISSIONS = 0,
     TAB_ID_CONNECTION,
     NUM_TAB_IDS,
   };
 
+  // The security summary is styled depending on the security state. At the
+  // moment, the only styling we apply is color, but it could also include e.g.
+  // bolding.
+  enum SecuritySummaryStyle {
+    STYLE_DEFAULT = 0,
+    STYLE_GREEN,
+    STYLE_YELLOW,
+    STYLE_RED,
+  };
+
+  struct SecurityDescription {
+    // A one-line summary of the security state.
+    base::string16 summary;
+    // How to style the summary.
+    SecuritySummaryStyle summary_style;
+    // A short paragraph with more details about the state, and how
+    // the user should treat it.
+    base::string16 details;
+  };
+
   // |CookieInfo| contains information about the cookies from a specific source.
   // A source can for example be a specific origin or an entire wildcard domain.
   struct CookieInfo {
     CookieInfo();
 
     // The number of allowed cookies.
     int allowed;
     // The number of blocked cookies.
     int blocked;
 
@@ skipping 35 matching lines @@
   struct IdentityInfo {
     IdentityInfo();
     ~IdentityInfo();
 
     // The site's identity: the certificate's Organization Name for sites with
     // Extended Validation certificates, or the URL's hostname for all other
     // sites.
     std::string site_identity;
     // Status of the site's identity.
     WebsiteSettings::SiteIdentityStatus identity_status;
-    // Helper to get the status text to display to the user.
+    // Helper to get security description info to display to the user.
+    std::unique_ptr<SecurityDescription> GetSecurityDescription() const;
+    // Deprecated method go get just the summary from GetSecurityDescription().
     base::string16 GetSecuritySummary() const;
-    // Textual description of the site's identity status that is displayed to
-    // the user.
+    // Helper to get the status text style (i.e. color).

[felt, 2016/06/30 05:39:24]

This comment seems mismatched, it is not above a helper method?

[lgarron, 2016/07/08 17:47:08]

Woops, I deleted the old comment instead of a temporary new one. Fixed.

     std::string identity_status_description;
     // The ID is the server certificate of a secure connection or 0.
     int cert_id;
     // Status of the site's connection.
     WebsiteSettings::SiteConnectionStatus connection_status;
     // Textual description of the site's connection status that is displayed to
     // the user.
     std::string connection_status_description;
     // Set when the user has explicitly bypassed an SSL error for this host and
     // has a flag set to remember ssl decisions (explicit flag or in the
@@ skipping 71 matching lines @@
 
   // Selects the tab with the given |tab_id|.
   virtual void SetSelectedTab(TabId tab_id) = 0;
 };
 
 typedef WebsiteSettingsUI::CookieInfoList CookieInfoList;
 typedef WebsiteSettingsUI::PermissionInfoList PermissionInfoList;
 typedef WebsiteSettingsUI::ChosenObjectInfoList ChosenObjectInfoList;
 
 #endif  // CHROME_BROWSER_UI_WEBSITE_SETTINGS_WEBSITE_SETTINGS_UI_H_