Permissions: Show the reason for permission decisions made on the user's behalf.

chrome/browser/ui/cocoa/page_info/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/page_info/website_settings_bubble_controller.h"
 
 #import <AppKit/AppKit.h>
 
 #include <cmath>
 
@@ skipping 77 matching lines @@
 
 // Minimum distance between the label and its corresponding menu.
 const CGFloat kMinSeparationBetweenLabelAndMenu = 16;
 
 // Square size of the permission delete button image.
 const CGFloat kPermissionDeleteImageSize = 16;
 
 // The spacing between individual permissions.
 const CGFloat kPermissionsVerticalSpacing = 16;
 
+// The spacing between permissions and their decision description labels.
+const CGFloat kPermissionsDecisionVerticalSpacing = 4;
+
 // Amount to lower each permission icon to align the icon baseline with the
 // label text.
 const CGFloat kPermissionIconYAdjustment = 1;
 
 // Amount to lower each permission popup button to make its text align with the
 // permission label.
 const CGFloat kPermissionPopupButtonYAdjustment = 3;
 
 // Internal Page Bubble --------------------------------------------------------
 
@@ skipping 796 matching lines @@
 
   // Align the icon with the text.
   [self alignPermissionIcon:imageView withTextField:label];
 
   // Permissions specified by policy or an extension cannot be changed.
   if (permissionInfo.source == content_settings::SETTING_SOURCE_EXTENSION ||
       permissionInfo.source == content_settings::SETTING_SOURCE_POLICY) {
     [button setEnabled:NO];
   }
 
+  // Show the permission decision reason, if it was not the user.

[dominickn, 2017/03/26 23:46:24]

"Show the reason for the permission decision in a new row if it did not come
from the user"?

[Patti Lor, 2017/03/27 05:52:40]

Done.

   NSRect buttonFrame = [button frame];
-  return NSMakePoint(NSMaxX(buttonFrame), NSMaxY(buttonFrame));
+  int maxY = NSMaxY(buttonFrame);
+  base::string16 reason = WebsiteSettingsUI::PermissionDecisionReasonToString(
+      [self profile], permissionInfo, url_);
+  if (!reason.empty()) {
+    point.y = position.y + kPermissionsDecisionVerticalSpacing +
+              label.frame.size.height;
+    label = [self addText:reason
+                 withSize:[NSFont systemFontSize]
+                     bold:NO
+                   toView:view
+                  atPoint:point];
+    label.textColor = skia::SkColorToSRGBNSColor(SK_ColorGRAY);
+    maxY += label.frame.size.height;
+  }
+
+  return NSMakePoint(NSMaxX(buttonFrame), maxY);
 }
 
 // Adds a new row to the UI listing the permissions. Returns the NSPoint of the
 // last UI element added (either the permission button, in LTR, or the text
 // label, in RTL).
 - (NSPoint)addChosenObject:(ChosenObjectInfoPtr)objectInfo
                     toView:(NSView*)view
                    atPoint:(NSPoint)point {
   base::string16 labelText = l10n_util::GetStringFUTF16(
       objectInfo->ui_info.label_string_id,
@@ skipping 286 matching lines @@
     const CookieInfoList& cookie_info_list) {
   [bubble_controller_ setCookieInfo:cookie_info_list];
 }
 
 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)];
 }