Convert OSX Page Info Bubble to Material Design

chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.h

 // Copyright 2014 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.
 
 #import <Cocoa/Cocoa.h>
 
 #include <memory>
 
 #include "base/mac/scoped_nsobject.h"
 #include "base/macros.h"
@@ skipping 15 matching lines @@
 // to OriginInfo*.
 @interface WebsiteSettingsBubbleController : BaseBubbleController {
  @private
   content::WebContents* webContents_;
 
   base::scoped_nsobject<NSView> contentView_;
 
   // The main content view for the Permissions tab.
   NSView* securitySectionView_;
 
-  // Displays the web site identity.
-  NSTextField* identityField_;
+  // Displays the short security summary for the page
+  // (private/not private/etc.).
+  NSTextField* securitySummaryField_;
 
-  // Displays the 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.
   NSButton* securityDetailsButton_;
 
   // 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.

[Avi (use Gerrit), 2016/08/24 04:02:41]

active

[lgarron, 2016/08/26 23:27:04]

Whoops, it seems I accidentally uploaded the latest patch to a new CL instead of
this one. All three comments should be addressed in patch #5; thanks for
noticing!

+  NSTextField* resetDecisionsField_;
+
+  // The link button for revoking certificate decisions.
+  // This link only shows when there is an acrive certificate exception.

[Avi (use Gerrit), 2016/08/24 04:02:41]

active

+  NSButton* resetDecisionsButton_;
+
   // Whether DevTools is disabled for the relevant profile.
   BOOL isDevToolsDisabled_;
 
   // The ID of the server certificate from the identity info. This should
   // always be non-zero on a cryptographic connection, and 0 otherwise.
   int certificateId_;
 
-  // The link button for revoking certificate decisions.
-  NSButton* resetDecisionsButton_;
-
   // Separator line.
   NSView* separatorAfterSecuritySection_;
 
   // Container for the site settings section.
   NSView* siteSettingsSectionView_;
 
   // Container for cookies info in the site settings section.
   NSView* cookiesView_;
 
   // Container for permission info in the site settings section.
@@ skipping 67 matching lines @@
 
  private:
   // The WebContents the bubble UI is attached to.
   content::WebContents* web_contents_;
 
   // The Cocoa controller for the bubble UI.
   WebsiteSettingsBubbleController* bubble_controller_;
 
   DISALLOW_COPY_AND_ASSIGN(WebsiteSettingsUIBridge);
 };