Index: chrome/browser/ui/website_settings/permission_menu_model.cc |
diff --git a/chrome/browser/ui/website_settings/permission_menu_model.cc b/chrome/browser/ui/website_settings/permission_menu_model.cc |
index 74e94e246531b82d4c9db2f6d82a0e8eb988c8dc..1c7632ce90155346deb03fdb20b365a88a531005 100644 |
--- a/chrome/browser/ui/website_settings/permission_menu_model.cc |
+++ b/chrome/browser/ui/website_settings/permission_menu_model.cc |
@@ -9,11 +9,13 @@ |
PermissionMenuModel::PermissionMenuModel( |
Delegate* delegate, |
+ const GURL& url, |
ContentSettingsType type, |
ContentSetting default_setting, |
ContentSetting current_setting) |
: ALLOW_THIS_IN_INITIALIZER_LIST(ui::SimpleMenuModel(this)), |
- delegate_(delegate) { |
+ delegate_(delegate), |
+ site_url_(url) { |
string16 label; |
switch (default_setting) { |
case CONTENT_SETTING_ALLOW: |
@@ -33,8 +35,9 @@ PermissionMenuModel::PermissionMenuModel( |
} |
AddCheckItem(COMMAND_SET_TO_DEFAULT, label); |
- // TODO(xians): Media should support COMMAND_SET_TO_ALLOW for https. |
- if (type != CONTENT_SETTINGS_TYPE_MEDIASTREAM) { |
+ // Media only support COMMAND_SET_TO_ALLOW for https. |
+ if (type != CONTENT_SETTINGS_TYPE_MEDIASTREAM || |
+ url.SchemeIsSecure()) { |
label = l10n_util::GetStringUTF16( |
IDS_WEBSITE_SETTINGS_MENU_ITEM_ALLOW); |
AddCheckItem(COMMAND_SET_TO_ALLOW, label); |