Views/Permissions: Update desktop UI to display BLOCK for embargoed permissions.

chrome/browser/permissions/permission_util.h

Index: chrome/browser/permissions/permission_util.h
diff --git a/chrome/browser/permissions/permission_util.h b/chrome/browser/permissions/permission_util.h
index b965ceb2169cca2bdf48b1eaa916f64d6fb2d141..e133b35ebf742d322ae0006aa64c5ca22cd9be53 100644
--- a/chrome/browser/permissions/permission_util.h
+++ b/chrome/browser/permissions/permission_util.h
@@ -61,6 +61,9 @@ class PermissionUtil {
   static bool GetPermissionType(ContentSettingsType type,
                                 content::PermissionType* out);
 
+  // Checks whether the given ContentSettingsType is a permission.
+  static bool IsPermission(ContentSettingsType type);

[dominickn, 2017/03/03 03:31:36]

Perhaps move this method to the anonymous namespace in
permission_selector_row.cc? We're still working out what the best way of
handling this is, so for now it might be best to avoid polluting too many files.
:)

[Patti Lor, 2017/03/03 05:16:32]

I used this in website_settings.cc as well D:

What about getting rid of it in permission_selector_row.cc by adding a new
SettingSource::KILL_SWITCH or similar? I'm not sure if overlapping
PermissionStatusSource and SettingSource is what you want though.

Alternatively we could just use GetPermissionType() from above which already has
a TODO on it and refactor all of this when a better way to distinguish between a
permission / normal content setting is worked out?

+
   static bool ShouldShowPersistenceToggle();
 
   // A scoped class that will check the current resolved content setting on