Cocoa Page Info: Implement expanding all permissions

chrome/browser/ui/cocoa/website_settings/website_settings_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/website_settings/website_settings_bubble_controller.h"
 
 #import <AppKit/AppKit.h>
 
 #include <cmath>
 
@@ skipping 52 matching lines @@
 const CGFloat kDefaultWindowWidth = 320;
 
 // Padding around each section
 const CGFloat kSectionVerticalPadding = 20;
 const CGFloat kSectionHorizontalPadding = 16;
 
 // Links are buttons with invisible padding, so we need to move them back to
 // align with other text.
 const CGFloat kLinkButtonXAdjustment = 1;
 
+// Built-in margin for NSButton to take into account.
+const CGFloat kNSButtonBuiltinMargin = 4;
+
 // Security Section ------------------------------------------------------------
 
 // Spacing between security summary, security details, and cert decisions text.
 const CGFloat kSecurityParagraphSpacing = 12;
 
 // Site Settings Section -------------------------------------------------------
 
 // Square size of the permission images.
 const CGFloat kPermissionImageSize = 16;
 
@@ skipping 108 matching lines @@
          url.SchemeIs(content::kViewSourceScheme);
 }
 
 - (id)initWithParentWindow:(NSWindow*)parentWindow
     websiteSettingsUIBridge:(WebsiteSettingsUIBridge*)bridge
                 webContents:(content::WebContents*)webContents
                         url:(const GURL&)url {
   DCHECK(parentWindow);
 
   webContents_ = webContents;
-  permissionsPresent_ = NO;
   url_ = url;
 
   // Use an arbitrary height; it will be changed in performLayout.
   NSRect contentRect = NSMakeRect(0, 0, [self defaultWindowWidth], 1);
   // Create an empty window into which content is placed.
   base::scoped_nsobject<InfoBubbleWindow> window(
       [[InfoBubbleWindow alloc] initWithContentRect:contentRect
                                           styleMask:NSBorderlessWindowMask
                                             backing:NSBackingStoreBuffered
                                               defer:NO]);
@@ skipping 157 matching lines @@
       [[[FlippedView alloc] initWithFrame:[superview frame]] autorelease];
   [cookiesView_ setAutoresizingMask:NSViewWidthSizable];
   [siteSettingsSectionView addSubview:cookiesView_];
 
   permissionsView_ =
       [[[FlippedView alloc] initWithFrame:[superview frame]] autorelease];
   [siteSettingsSectionView addSubview:permissionsView_];
 
   // 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_];
+  NSString* permissionButtonText =
+      l10n_util::GetNSString(IDS_PAGE_INFO_SHOW_ALL_PERMISSIONS_LINK);
+  permissionButton_ = [self addButtonWithText:permissionButtonText
+                                       toView:siteSettingsSectionView];
+  [GTMUILocalizerAndLayoutTweaker sizeToFitView:permissionButton_];
 
-  [siteSettingsButton_ setTarget:self];
-  [siteSettingsButton_ setAction:@selector(showSiteSettingsData:)];
+  [permissionButton_ setTarget:self];
+  [permissionButton_ setAction:@selector(showAllPermissions:)];
 
   return siteSettingsSectionView.get();
 }
 
 // Handler for the link button below the list of cookies.
 - (void)showCookiesAndSiteData:(id)sender {
   DCHECK(webContents_);
   DCHECK(presenter_);
   presenter_->RecordWebsiteSettingsAction(
       WebsiteSettings::WEBSITE_SETTINGS_COOKIES_DIALOG_OPENED);
   TabDialogs::FromWebContents(webContents_)->ShowCollectedCookies();
 }
 
 // Handler for the site settings button below the list of permissions.
+- (void)showAllPermissions:(id)sender {
+  DCHECK(presenter_);
+  // TODO(crbug.com/695670): Place this code in a cross-platform location.
+  presenter_->RecordWebsiteSettingsAction(
+      WebsiteSettings::WEBSITE_SETTINGS_SHOW_ALL_PERMISSIONS_PRESSED);
+  // TODO(crbug.com/695723): Avoid hiding a permission when it is changed to the
+  // default.
+  presenter_->PresentAllSitePermissions();
+}
+
+// Handler for the site settings button below the list of permissions.
 - (void)showSiteSettingsData:(id)sender {
   DCHECK(webContents_);
   DCHECK(presenter_);
   presenter_->RecordWebsiteSettingsAction(
       WebsiteSettings::WEBSITE_SETTINGS_SITE_SETTINGS_OPENED);
   webContents_->OpenURL(content::OpenURLParams(
       GURL(chrome::kChromeUIContentSettingsURL), content::Referrer(),
       WindowOpenDisposition::NEW_FOREGROUND_TAB, ui::PAGE_TRANSITION_LINK,
       false));
 }
@@ skipping 23 matching lines @@
 // Handler for the link button to revoke user certificate decisions.
 - (void)resetCertificateDecisions:(id)sender {
   DCHECK(resetDecisionsButton_);
   presenter_->OnRevokeSSLErrorBypassButtonPressed();
   [self close];
 }
 
 - (CGFloat)layoutViewAtRTLStart:(NSView*)view withYPosition:(CGFloat)yPos {
   CGFloat xPos;
   if (base::i18n::IsRTL()) {
-    xPos =
-        kDefaultWindowWidth - kSectionHorizontalPadding - NSWidth([view frame]);
+    xPos = kDefaultWindowWidth - kSectionHorizontalPadding -
+           NSWidth([view frame]) + kNSButtonBuiltinMargin;
   } else {
-    xPos = kSectionHorizontalPadding;
+    xPos = kSectionHorizontalPadding - kNSButtonBuiltinMargin;
   }
-  [view setFrameOrigin:NSMakePoint(xPos, yPos)];
-  return yPos + NSHeight([view frame]);
+  [view setFrameOrigin:NSMakePoint(xPos, yPos - kNSButtonBuiltinMargin)];
+  return yPos + NSHeight([view frame]) - kNSButtonBuiltinMargin;
 }
 
 // Set the Y position of |view| to the given position, and return the position
 // of its bottom edge.
 - (CGFloat)setYPositionOfView:(NSView*)view to:(CGFloat)position {
   NSRect frame = [view frame];
   frame.origin.y = position;
   [view setFrame:frame];
   return position + NSHeight(frame);
 }
@@ skipping 64 matching lines @@
   }
 
   // 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;
 
-  yPos =
-      [self setYPositionOfView:cookiesView_ to:yPos + kSectionVerticalPadding];
-
-  if (permissionsPresent_) {
-    // Put the permission info just below the link button.
-    yPos = [self setYPositionOfView:permissionsView_ to:yPos];
-  }
-
-  yPos = [self layoutViewAtRTLStart:siteSettingsButton_ withYPosition:yPos];
+  yPos += kSectionVerticalPadding;
+  yPos = [self setYPositionOfView:cookiesView_ to:yPos];
+  yPos = [self setYPositionOfView:permissionsView_ to:yPos];
+  yPos = [self layoutViewAtRTLStart:permissionButton_ withYPosition:yPos];
 
   // Resize the height based on contents.
   [self setHeightOfView:siteSettingsSectionView_
                      to:yPos + kSectionVerticalPadding];
 }
 
 // Adjust the size of the window to match the size of the content, and position
 // the bubble anchor appropriately.
 - (void)sizeAndPositionWindow {
   NSRect windowFrame = [contentView_ frame];
@@ skipping 537 matching lines @@
       setFrameSize:NSMakeSize(NSWidth([cookiesView_ frame]), controlOrigin.y)];
 
   [self performLayout];
 }
 
 - (void)setPermissionInfo:(const PermissionInfoList&)permissionInfoList
          andChosenObjects:(ChosenObjectInfoList)chosenObjectInfoList {
   [permissionsView_ setSubviews:[NSArray array]];
   NSPoint controlOrigin = NSMakePoint(kSectionHorizontalPadding, 0);
 
-  permissionsPresent_ = YES;
+  for (const auto& permission : permissionInfoList) {
+    controlOrigin.y += kPermissionsVerticalSpacing;
+    NSPoint rowBottomRight = [self addPermission:permission
+                                          toView:permissionsView_
+                                         atPoint:controlOrigin];
+    controlOrigin.y = rowBottomRight.y;
+  }
 
-  if (permissionInfoList.size() > 0 || chosenObjectInfoList.size() > 0) {
-    base::string16 sectionTitle = l10n_util::GetStringUTF16(
-        IDS_WEBSITE_SETTINGS_TITLE_SITE_PERMISSIONS);
-
-    for (const auto& permission : permissionInfoList) {
-      controlOrigin.y += kPermissionsVerticalSpacing;
-      NSPoint rowBottomRight = [self addPermission:permission
+  for (auto& object : chosenObjectInfoList) {
+    controlOrigin.y += kPermissionsVerticalSpacing;
+    NSPoint rowBottomRight = [self addChosenObject:std::move(object)
                                             toView:permissionsView_
                                            atPoint:controlOrigin];
-      controlOrigin.y = rowBottomRight.y;
-    }
-
-    for (auto& object : chosenObjectInfoList) {
-      controlOrigin.y += kPermissionsVerticalSpacing;
-      NSPoint rowBottomRight = [self addChosenObject:std::move(object)
-                                              toView:permissionsView_
-                                             atPoint:controlOrigin];
-      controlOrigin.y = rowBottomRight.y;
-    }
-
-    controlOrigin.y += kPermissionsVerticalSpacing;
+    controlOrigin.y = rowBottomRight.y;
   }
 
+  controlOrigin.y += kPermissionsVerticalSpacing;
   [permissionsView_ setFrameSize:
       NSMakeSize(NSWidth([permissionsView_ frame]), controlOrigin.y)];
   [self performLayout];
 }
 
+- (void)updatePermissionButton:
+    (WebsiteSettingsUI::VisiblePermissions)visiblePermissions {
+  base::string16 title = bridge_->GetPermissionButtonString(visiblePermissions);
+  [permissionButton_ setTitle:base::SysUTF16ToNSString(title)];
+  if (visiblePermissions == WebsiteSettingsUI::VISIBLE_PERMISSIONS_ALL) {
+    [permissionButton_ setAction:@selector(showSiteSettingsData:)];
+  } else {
+    [permissionButton_ setAction:@selector(showAllPermissions:)];
+  }
+}
+
 @end
 
 WebsiteSettingsUIBridge::WebsiteSettingsUIBridge(
     content::WebContents* web_contents)
     : content::WebContentsObserver(web_contents),
       web_contents_(web_contents),
       bubble_controller_(nil) {
   DCHECK(!g_is_popup_showing);
   g_is_popup_showing = true;
 }
@@ skipping 71 matching lines @@
 
 void WebsiteSettingsUIBridge::SetPermissionInfo(
     const PermissionInfoList& permission_info_list,
     ChosenObjectInfoList chosen_object_info_list) {
   [bubble_controller_ setPermissionInfo:permission_info_list
                        andChosenObjects:std::move(chosen_object_info_list)];
 }
 
 void WebsiteSettingsUIBridge::UpdatePermissionButton(
     WebsiteSettingsUI::VisiblePermissions visible_permissions) {
-  // TODO(crbug.com/657267)
+  [bubble_controller_ updatePermissionButton:visible_permissions];
 }