Add a Certificate Viewer link to the Page Info dropdown

chrome/browser/ui/cocoa/page_info/page_info_bubble_controller.mm

 // 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 "chrome/browser/ui/cocoa/page_info/page_info_bubble_controller.h"
 
 #import <AppKit/AppKit.h>
 
 #include <cmath>
 
@@ skipping 121 matching lines @@
     if (location_bar) {
       NSPoint bubble_point = location_bar->GetPageInfoBubblePoint();
       origin = ui::ConvertPointFromWindowToScreen(parent, bubble_point);
     }
   }
   return origin;
 }
 
 }  // namespace
 
+// The |InspectLinkView| objects are used to show the Cookie and Certificate
+// status and a link to inspect the underlying data.
+@interface InspectLinkView : FlippedView
+@end
+
+@implementation InspectLinkView {
+  NSButton* actionLink_;
+}
+
+- (id)initWithFrame:(NSRect)frame {
+  if (self = [super initWithFrame:frame]) {
+    [self setAutoresizingMask:NSViewWidthSizable];
+  }
+  return self;
+}
+
+- (void)setActionLink:(NSButton*)actionLink {
+  actionLink_ = actionLink;
+}
+
+- (void)setLinkText:(NSString*)linkText {
+  [actionLink_ setTitle:linkText];
+  [GTMUILocalizerAndLayoutTweaker sizeToFitView:actionLink_];
+}
+
+- (void)setLinkToolTip:(NSString*)linkToolTip {
+  [actionLink_ setToolTip:linkToolTip];
+}
+
+- (void)setLinkTarget:(NSObject*)target withAction:(SEL)action {
+  [actionLink_ setTarget:target];
+  [actionLink_ setAction:action];
+}
+@end
+
 @interface ChosenObjectDeleteButton : HoverImageButton {
  @private
   ChosenObjectInfoPtr objectInfo_;
   ChosenObjectDeleteCallback callback_;
 }
 
 // Designated initializer. Takes ownership of |objectInfo|.
 - (instancetype)initWithChosenObject:(ChosenObjectInfoPtr)objectInfo
                              atPoint:(NSPoint)point
                         withCallback:(ChosenObjectDeleteCallback)callback;
@@ skipping 210 matching lines @@
 
 // Create and return a subview for the site settings and add it to the given
 // |superview|. |superview| retains the new view.
 - (NSView*)addSiteSettingsSectionToView:(NSView*)superview {
   base::scoped_nsobject<NSView> siteSettingsSectionView(
       [[FlippedView alloc] initWithFrame:[superview frame]]);
   [superview addSubview:siteSettingsSectionView];
 
   // Initialize the two containers that hold the controls. The initial frames
   // are arbitrary, and will be adjusted after the controls are laid out.
-  cookiesView_ =
-      [[[FlippedView alloc] initWithFrame:[superview frame]] autorelease];
-  [cookiesView_ setAutoresizingMask:NSViewWidthSizable];
-  [siteSettingsSectionView addSubview:cookiesView_];
+  PageInfoUI::PermissionInfo info;
+  info.type = CONTENT_SETTINGS_TYPE_COOKIES;
+  info.setting = CONTENT_SETTING_ALLOW;
+  cookiesView_ = [self
+      addInspectLinkToView:siteSettingsSectionView
+               sectionIcon:PageInfoUI::GetPermissionIcon(info).ToNSImage()
+              sectionTitle:l10n_util::GetStringUTF16(IDS_PAGE_INFO_COOKIES)
+                  linkText:l10n_util::GetPluralNSStringF(
+                               IDS_PAGE_INFO_NUM_COOKIES, 0)];
+  [cookiesView_ setLinkTarget:self
+                   withAction:@selector(showCookiesAndSiteData:)];
 
   permissionsView_ =
       [[[FlippedView alloc] initWithFrame:[superview frame]] autorelease];
   [siteSettingsSectionView addSubview:permissionsView_];
 
+  // The certificate section is created on demand.
+  certificateView_ = nil;
+
   // Create the link button to view site settings. Its position will be set in
   // performLayout.
   NSString* siteSettingsButtonText =
       l10n_util::GetNSString(IDS_PAGE_INFO_SITE_SETTINGS_LINK);
   siteSettingsButton_ = [self addButtonWithText:siteSettingsButtonText
                                          toView:siteSettingsSectionView];
   [GTMUILocalizerAndLayoutTweaker sizeToFitView:siteSettingsButton_];
 
   [siteSettingsButton_ setTarget:self];
   [siteSettingsButton_ setAction:@selector(showSiteSettingsData:)];
 
   return siteSettingsSectionView.get();
 }
 
+- (InspectLinkView*)addInspectLinkToView:(NSView*)superview
+                             sectionIcon:(NSImage*)imageIcon
+                            sectionTitle:(const base::string16&)titleText
+                                linkText:(NSString*)linkText {
+  // Create the subview.
+  base::scoped_nsobject<InspectLinkView> newView(
+      [[InspectLinkView alloc] initWithFrame:[superview frame]]);
+  [superview addSubview:newView];
+
+  bool isRTL = base::i18n::IsRTL();
+  NSPoint controlOrigin = NSMakePoint(kSectionHorizontalPadding, 0);
+
+  CGFloat viewWidth = NSWidth([newView frame]);
+
+  // Reset X for the icon.
+  if (isRTL) {
+    controlOrigin.x =
+        viewWidth - kPermissionImageSize - kSectionHorizontalPadding;
+  }
+
+  NSImageView* imageView = [self addImageWithSize:[imageIcon size]
+                                           toView:newView
+                                          atPoint:controlOrigin];
+  [imageView setImage:imageIcon];
+
+  NSButton* actionLink = [self addLinkButtonWithText:linkText toView:newView];
+  [newView setActionLink:actionLink];
+
+  if (isRTL) {
+    controlOrigin.x -= kPermissionImageSpacing;
+    NSTextField* sectionTitle = [self addText:titleText
+                                     withSize:[NSFont systemFontSize]
+                                         bold:NO
+                                       toView:newView
+                                      atPoint:controlOrigin];
+    [sectionTitle sizeToFit];
+
+    NSPoint sectionTitleOrigin = [sectionTitle frame].origin;
+    sectionTitleOrigin.x -= NSWidth([sectionTitle frame]);
+    [sectionTitle setFrameOrigin:sectionTitleOrigin];
+
+    // Align the icon with the text.
+    [self alignPermissionIcon:imageView withTextField:sectionTitle];
+
+    controlOrigin.y +=
+        NSHeight([sectionTitle frame]) + kPermissionLabelBottomPadding;
+    controlOrigin.x -= NSWidth([actionLink frame]) - kLinkButtonXAdjustment;
+    [actionLink setFrameOrigin:controlOrigin];
+  } else {
+    controlOrigin.x += kPermissionImageSize + kPermissionImageSpacing;
+    NSTextField* sectionTitle = [self addText:titleText
+                                     withSize:[NSFont systemFontSize]
+                                         bold:NO
+                                       toView:newView
+                                      atPoint:controlOrigin];
+    [sectionTitle sizeToFit];
+
+    // Align the icon with the text.
+    [self alignPermissionIcon:imageView withTextField:sectionTitle];
+
+    controlOrigin.y +=
+        NSHeight([sectionTitle frame]) + kPermissionLabelBottomPadding;
+    controlOrigin.x -= kLinkButtonXAdjustment;
+    [actionLink setFrameOrigin:controlOrigin];
+  }
+
+  controlOrigin.y += NSHeight([actionLink frame]);
+  [newView setFrameSize:NSMakeSize(NSWidth([newView frame]), controlOrigin.y)];
+
+  return newView.get();
+}
+
 // Handler for the link button below the list of cookies.
 - (void)showCookiesAndSiteData:(id)sender {
   DCHECK(webContents_);
   DCHECK(presenter_);
   presenter_->RecordPageInfoAction(PageInfo::PAGE_INFO_COOKIES_DIALOG_OPENED);
   TabDialogs::FromWebContents(webContents_)->ShowCollectedCookies();
 }
 
 // Handler for the site settings button below the list of permissions.
 - (void)showSiteSettingsData:(id)sender {
@@ skipping 118 matching lines @@
                                    yPos)];
     yPos = NSMaxY([resetDecisionsButton_ frame]);
   }
 
   // Resize the height based on contents.
   [self setHeightOfView:securitySectionView_ to:yPos + kSectionVerticalPadding];
 }
 
 - (void)layoutSiteSettingsSection {
   // Start the layout with the first element. Margins are handled by the caller.
-  CGFloat yPos = 0;
+  CGFloat yPos = kSectionVerticalPadding;
 
-  yPos =
-      [self setYPositionOfView:cookiesView_ to:yPos + kSectionVerticalPadding];
+  if (certificateView_) {
+    yPos = [self setYPositionOfView:certificateView_ to:yPos] +
+           kPermissionsVerticalSpacing;
+  }
+
+  yPos = [self setYPositionOfView:cookiesView_ to:yPos];
 
   if (permissionsPresent_) {
     // Put the permission info just below the link button.
     yPos = [self setYPositionOfView:permissionsView_ to:yPos];
   }
 
   yPos = [self layoutViewAtRTLStart:siteSettingsButton_ withYPosition:yPos];
 
   // Resize the height based on contents.
   [self setHeightOfView:siteSettingsSectionView_
@@ skipping 132 matching lines @@
   buttonFrame.size.width = maxTitleWidth;
 
   [button setFrame:buttonFrame];
   [button setButtonType:NSMomentaryPushInButton];
   [button setBezelStyle:NSRegularSquareBezelStyle];
   [view addSubview:button.get()];
 
   return button.get();
 }
 
-// Set the content of the identity and identity status fields.
+// Set the content of the identity and identity status fields, and add the
+// Certificate view if applicable.
 - (void)setIdentityInfo:(const PageInfoUI::IdentityInfo&)identityInfo {
   std::unique_ptr<PageInfoUI::SecurityDescription> security_description =
       identityInfo.GetSecurityDescription();
   [securitySummaryField_
-      setStringValue:base::SysUTF16ToNSString(security_description->summary)];
+      setStringValue:SysUTF16ToNSString(security_description->summary)];
 
   [securityDetailsField_
       setStringValue:SysUTF16ToNSString(security_description->details)];
 
   certificate_ = identityInfo.certificate;
 
-  if (certificate_ && identityInfo.show_ssl_decision_revoke_button) {
-    resetDecisionsField_ =
-        [self addText:base::string16()
-             withSize:[NSFont smallSystemFontSize]
-                 bold:NO
-               toView:securitySectionView_
-              atPoint:NSMakePoint(kSectionHorizontalPadding, 0)];
-    [resetDecisionsField_
-        setStringValue:l10n_util::GetNSString(
-                           IDS_PAGEINFO_INVALID_CERTIFICATE_DESCRIPTION)];
-    [self sizeTextFieldHeightToFit:resetDecisionsField_];
+  if (certificate_) {
+    if (identityInfo.show_ssl_decision_revoke_button) {
+      resetDecisionsField_ =
+          [self addText:base::string16()
+               withSize:[NSFont smallSystemFontSize]
+                   bold:NO
+                 toView:securitySectionView_
+                atPoint:NSMakePoint(kSectionHorizontalPadding, 0)];
+      [resetDecisionsField_
+          setStringValue:l10n_util::GetNSString(
+                             IDS_PAGEINFO_INVALID_CERTIFICATE_DESCRIPTION)];
+      [self sizeTextFieldHeightToFit:resetDecisionsField_];
 
-    resetDecisionsButton_ =
-        [self addLinkButtonWithText:
-                  l10n_util::GetNSString(
-                      IDS_PAGEINFO_RESET_INVALID_CERTIFICATE_DECISIONS_BUTTON)
-                             toView:securitySectionView_];
-    [resetDecisionsButton_ setTarget:self];
-    [resetDecisionsButton_ setAction:@selector(resetCertificateDecisions:)];
+      resetDecisionsButton_ =
+          [self addLinkButtonWithText:
+                    l10n_util::GetNSString(
+                        IDS_PAGEINFO_RESET_INVALID_CERTIFICATE_DECISIONS_BUTTON)
+                               toView:securitySectionView_];
+      [resetDecisionsButton_ setTarget:self];
+      [resetDecisionsButton_ setAction:@selector(resetCertificateDecisions:)];
+    }
+
+    if (PageInfoUI::ShouldShowCertificateLink()) {
+      bool isValid = (identityInfo.identity_status !=
+                      PageInfo::SITE_IDENTITY_STATUS_ERROR);
+      NSString* linkText = l10n_util::GetNSString(
+          isValid ? IDS_PAGE_INFO_CERTIFICATE_VALID_LINK
+                  : IDS_PAGE_INFO_CERTIFICATE_INVALID_LINK);
+
+      certificateView_ = [self
+          addInspectLinkToView:siteSettingsSectionView_
+                   sectionIcon:PageInfoUI::GetCertificateIcon().ToNSImage()
+                  sectionTitle:l10n_util::GetStringUTF16(
+                                   IDS_PAGE_INFO_CERTIFICATE)
+                      linkText:linkText];
+      if (isValid) {
+        [certificateView_
+            setLinkToolTip:l10n_util::GetNSStringF(
+                               IDS_PAGE_INFO_CERTIFICATE_VALID_LINK_TOOLTIP,
+                               base::UTF8ToUTF16(
+                                   certificate_->issuer().GetDisplayName()))];
+      }
+
+      [certificateView_ setLinkTarget:self
+                           withAction:@selector(showCertificateInfo:)];
+    }
   }
 
   [self performLayout];
 }
 
 // Add a pop-up button for |permissionInfo| to the given view.
 - (NSPopUpButton*)addPopUpButtonForPermission:
                       (const PageInfoUI::PermissionInfo&)permissionInfo
                                        toView:(NSView*)view
                                       atPoint:(NSPoint)point {
@@ skipping 276 matching lines @@
 // Align an image with a text field by vertically centering the image on
 // the cap height of the first line of text.
 - (void)alignPermissionIcon:(NSImageView*)imageView
               withTextField:(NSTextField*)textField {
   NSRect frame = [imageView frame];
   frame.origin.y += kPermissionIconYAdjustment;
   [imageView setFrame:frame];
 }
 
 - (void)setCookieInfo:(const CookieInfoList&)cookieInfoList {
-  // A result of re-ordering of the permissions (crbug.com/444244) is
-  // that sometimes permissions may not be displayed at all, so it's
-  // incorrect to check they are set before the cookie info.
-
   // |cookieInfoList| should only ever have 2 items: first- and third-party
   // cookies.
   DCHECK_EQ(cookieInfoList.size(), 2u);
 
   int totalAllowed = 0;
   for (const auto& i : cookieInfoList) {
     totalAllowed += i.allowed;
   }
-  base::string16 label_text =
-      l10n_util::GetPluralStringFUTF16(IDS_PAGE_INFO_NUM_COOKIES, totalAllowed);
 
-  base::string16 sectionTitle =
-      l10n_util::GetStringUTF16(IDS_PAGE_INFO_TITLE_SITE_DATA);
-  bool isRTL = base::i18n::IsRTL();
-
-  [cookiesView_ setSubviews:[NSArray array]];
-  NSPoint controlOrigin = NSMakePoint(kSectionHorizontalPadding, 0);
-
-  CGFloat viewWidth = NSWidth([cookiesView_ frame]);
-
-  // Reset X for the cookie image.
-  if (isRTL) {
-    controlOrigin.x = viewWidth - kPermissionImageSize -
-                      kPermissionImageSpacing - kSectionHorizontalPadding;
-  }
-
-  PageInfoUI::PermissionInfo info;
-  info.type = CONTENT_SETTINGS_TYPE_COOKIES;
-  info.setting = CONTENT_SETTING_ALLOW;
-  // info.default_setting, info.source, and info.is_incognito have not been set,
-  // but GetPermissionIcon doesn't use any of those.
-  NSImage* image = PageInfoUI::GetPermissionIcon(info).ToNSImage();
-  NSImageView* imageView = [self addImageWithSize:[image size]
-                                           toView:cookiesView_
-                                          atPoint:controlOrigin];
-  [imageView setImage:image];
-
-  NSButton* cookiesButton =
-      [self addLinkButtonWithText:base::SysUTF16ToNSString(label_text)
-                           toView:cookiesView_];
-  [cookiesButton setTarget:self];
-  [cookiesButton setAction:@selector(showCookiesAndSiteData:)];
-
-  if (isRTL) {
-    controlOrigin.x -= kPermissionImageSpacing;
-    NSTextField* cookiesLabel =
-        [self addText:l10n_util::GetStringUTF16(IDS_PAGE_INFO_COOKIES)
-             withSize:[NSFont systemFontSize]
-                 bold:NO
-               toView:cookiesView_
-              atPoint:controlOrigin];
-    [cookiesLabel sizeToFit];
-
-    NSPoint cookiesLabelOrigin = [cookiesLabel frame].origin;
-    cookiesLabelOrigin.x -= NSWidth([cookiesLabel frame]);
-    [cookiesLabel setFrameOrigin:cookiesLabelOrigin];
-
-    // Align the icon with the text.
-    [self alignPermissionIcon:imageView withTextField:cookiesLabel];
-
-    controlOrigin.y +=
-        NSHeight([cookiesLabel frame]) + kPermissionLabelBottomPadding;
-    controlOrigin.x -= NSWidth([cookiesButton frame]) - kLinkButtonXAdjustment;
-    [cookiesButton setFrameOrigin:controlOrigin];
-  } else {
-    controlOrigin.x += kPermissionImageSize + kPermissionImageSpacing;
-    NSTextField* cookiesLabel =
-        [self addText:l10n_util::GetStringUTF16(IDS_PAGE_INFO_COOKIES)
-             withSize:[NSFont systemFontSize]
-                 bold:NO
-               toView:cookiesView_
-              atPoint:controlOrigin];
-    [cookiesLabel sizeToFit];
-
-    controlOrigin.y +=
-        NSHeight([cookiesLabel frame]) + kPermissionLabelBottomPadding;
-    controlOrigin.x -= kLinkButtonXAdjustment;
-    [cookiesButton setFrameOrigin:controlOrigin];
-
-    // Align the icon with the text.
-    [self alignPermissionIcon:imageView withTextField:cookiesLabel];
-  }
-
-  controlOrigin.y += NSHeight([cookiesButton frame]);
-  [cookiesView_
-      setFrameSize:NSMakeSize(NSWidth([cookiesView_ frame]), controlOrigin.y)];
-
+  [cookiesView_ setLinkText:l10n_util::GetPluralNSStringF(
+                                IDS_PAGE_INFO_NUM_COOKIES, totalAllowed)];
   [self performLayout];
 }
 
 - (void)setPermissionInfo:(const PermissionInfoList&)permissionInfoList
          andChosenObjects:(ChosenObjectInfoList)chosenObjectInfoList {
   [permissionsView_ setSubviews:[NSArray array]];
   NSPoint controlOrigin = NSMakePoint(kSectionHorizontalPadding, 0);
 
   permissionsPresent_ = YES;
 
   if (permissionInfoList.size() > 0 || chosenObjectInfoList.size() > 0) {
-    base::string16 sectionTitle =
-        l10n_util::GetStringUTF16(IDS_PAGE_INFO_TITLE_SITE_PERMISSIONS);
-
     for (const auto& permission : permissionInfoList) {
       controlOrigin.y += kPermissionsVerticalSpacing;
       NSPoint rowBottomRight = [self addPermission:permission
                                             toView:permissionsView_
                                            atPoint:controlOrigin];
       controlOrigin.y = rowBottomRight.y;
     }
 
     for (auto& object : chosenObjectInfoList) {
       controlOrigin.y += kPermissionsVerticalSpacing;
@@ skipping 88 matching lines @@
 void PageInfoUIBridge::SetCookieInfo(const CookieInfoList& cookie_info_list) {
   [bubble_controller_ setCookieInfo:cookie_info_list];
 }
 
 void PageInfoUIBridge::SetPermissionInfo(
     const PermissionInfoList& permission_info_list,
     ChosenObjectInfoList chosen_object_info_list) {
   [bubble_controller_ setPermissionInfo:permission_info_list
                        andChosenObjects:std::move(chosen_object_info_list)];
 }