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

chrome/browser/ui/views/page_info/permission_selector_row.cc

 // 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.
 
 #include "chrome/browser/ui/views/page_info/permission_selector_row.h"
 
 #include "base/i18n/rtl.h"
 #include "base/macros.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/ui/page_info/permission_menu_model.h"
@@ skipping 235 matching lines @@
   label_ = new views::Label(
       WebsiteSettingsUI::PermissionTypeToUIString(permission.type));
   layout->AddView(label_, 1, 1, views::GridLayout::LEADING,
                   views::GridLayout::CENTER);
   // Create the menu model.
   menu_model_.reset(new PermissionMenuModel(
       profile, url, permission,
       base::Bind(&PermissionSelectorRow::PermissionChanged,
                  base::Unretained(this))));
 
-  // Create the permission menu button.
+// Create the permission menu button.
 #if defined(OS_MACOSX)
   bool use_real_combobox = true;
 #else
   bool use_real_combobox =
       ui::MaterialDesignController::IsSecondaryUiMaterial();
 #endif
   if (use_real_combobox)
     InitializeComboboxView(layout, permission);
   else
     InitializeMenuButtonView(layout, permission);
+
+  // Show the permission decision reason, if it was not the user.

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

Same comment as above

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

Done.

+  base::string16 reason = WebsiteSettingsUI::PermissionDecisionReasonToString(
+      profile, permission, url);
+  if (!reason.empty()) {
+    layout->StartRow(1, 1);
+    layout->SkipColumns(1);
+    views::Label* permission_decision_reason = new views::Label(reason);
+    permission_decision_reason->SetEnabledColor(SK_ColorGRAY);
+    // Long labels should span the remaining width of the row.
+    views::ColumnSet* column_set = layout->GetColumnSet(1);
+    DCHECK(!!column_set);
+    layout->AddView(permission_decision_reason, column_set->num_columns() - 2,
+                    1, views::GridLayout::LEADING, views::GridLayout::CENTER);
+  }
 }
 
 void PermissionSelectorRow::AddObserver(
     PermissionSelectorRowObserver* observer) {
   observer_list_.AddObserver(observer);
 }
 
 PermissionSelectorRow::~PermissionSelectorRow() {
   // Gross. On paper the Combobox and the ComboboxModelAdapter are both owned by
   // this class, but actually, the Combobox is owned by View and will be
@@ skipping 65 matching lines @@
     observer.OnPermissionChanged(permission);
 }
 
 views::View* PermissionSelectorRow::button() {
   // These casts are required because the two arms of a ?: cannot have different
   // types T1 and T2, even if the resulting value of the ?: is about to be a T
   // and T1 and T2 are both subtypes of T.
   return menu_button_ ? static_cast<views::View*>(menu_button_)
                       : static_cast<views::View*>(combobox_);
 }