| 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 aec4bb63562618170ecb7972e477b52551c6e8b7..254fa9a7413113cc5341d94bb904244022b048db 100644
|
| --- a/chrome/browser/ui/website_settings/permission_menu_model.cc
|
| +++ b/chrome/browser/ui/website_settings/permission_menu_model.cc
|
| @@ -4,6 +4,7 @@
|
|
|
| #include "chrome/browser/ui/website_settings/permission_menu_model.h"
|
|
|
| +#include "chrome/browser/plugins/plugins_field_trial.h"
|
| #include "chrome/grit/generated_resources.h"
|
| #include "ui/base/l10n/l10n_util.h"
|
|
|
| @@ -14,7 +15,15 @@ PermissionMenuModel::PermissionMenuModel(
|
| : ui::SimpleMenuModel(this), permission_(info), callback_(callback) {
|
| DCHECK(!callback_.is_null());
|
| base::string16 label;
|
| - switch (permission_.default_setting) {
|
| +
|
| + ContentSetting effective_setting = permission_.setting;
|
| +
|
| +#if defined(ENABLE_PLUGINS)
|
| + effective_setting = PluginsFieldTrial::EffectiveContentSetting(
|
| + permission_.type, permission_.setting);
|
| +#endif // defined(ENABLE_PLUGINS)
|
| +
|
| + switch (effective_setting) {
|
| case CONTENT_SETTING_ALLOW:
|
| label = l10n_util::GetStringUTF16(
|
| IDS_WEBSITE_SETTINGS_MENU_ITEM_DEFAULT_ALLOW);
|
| @@ -24,11 +33,8 @@ PermissionMenuModel::PermissionMenuModel(
|
| IDS_WEBSITE_SETTINGS_MENU_ITEM_DEFAULT_BLOCK);
|
| break;
|
| case CONTENT_SETTING_ASK:
|
| - // For Plugins, ASK is obsolete. Show as BLOCK to reflect actual behavior.
|
| - label = l10n_util::GetStringUTF16(
|
| - permission_.type == CONTENT_SETTINGS_TYPE_PLUGINS
|
| - ? IDS_WEBSITE_SETTINGS_MENU_ITEM_DEFAULT_BLOCK
|
| - : IDS_WEBSITE_SETTINGS_MENU_ITEM_DEFAULT_ASK);
|
| + label =
|
| + l10n_util::GetStringUTF16(IDS_WEBSITE_SETTINGS_MENU_ITEM_DEFAULT_ASK);
|
| break;
|
| case CONTENT_SETTING_DETECT_IMPORTANT_CONTENT:
|
| label = l10n_util::GetStringUTF16(
|
| @@ -90,13 +96,14 @@ PermissionMenuModel::PermissionMenuModel(const GURL& url,
|
| PermissionMenuModel::~PermissionMenuModel() {}
|
|
|
| bool PermissionMenuModel::IsCommandIdChecked(int command_id) const {
|
| - // For Plugins, ASK is obsolete. Show as BLOCK to reflect actual behavior.
|
| - if (permission_.type == CONTENT_SETTINGS_TYPE_PLUGINS &&
|
| - permission_.setting == CONTENT_SETTING_ASK &&
|
| - command_id == CONTENT_SETTING_BLOCK) {
|
| - return true;
|
| - }
|
| - return permission_.setting == command_id;
|
| + ContentSetting setting = permission_.setting;
|
| +
|
| +#if defined(ENABLE_PLUGINS)
|
| + setting = PluginsFieldTrial::EffectiveContentSetting(permission_.type,
|
| + permission_.setting);
|
| +#endif // defined(ENABLE_PLUGINS)
|
| +
|
| + return setting == command_id;
|
| }
|
|
|
| bool PermissionMenuModel::IsCommandIdEnabled(int command_id) const {
|
|
|