Chromium Code Reviews| Index: chrome/browser/extensions/extension_context_menu_model.cc |
| diff --git a/chrome/browser/extensions/extension_context_menu_model.cc b/chrome/browser/extensions/extension_context_menu_model.cc |
| index 3334fd9ccc063a8409a9b146992717331460d3b8..848f9395943043a1f5bd052872c95628cd4557c5 100644 |
| --- a/chrome/browser/extensions/extension_context_menu_model.cc |
| +++ b/chrome/browser/extensions/extension_context_menu_model.cc |
| @@ -6,6 +6,7 @@ |
| #include "base/prefs/pref_service.h" |
| #include "base/strings/utf_string_conversions.h" |
| +#include "chrome/browser/extensions/active_script_controller.h" |
| #include "chrome/browser/extensions/api/extension_action/extension_action_api.h" |
| #include "chrome/browser/extensions/extension_action.h" |
| #include "chrome/browser/extensions/extension_action_manager.h" |
| @@ -26,6 +27,7 @@ |
| #include "extensions/browser/management_policy.h" |
| #include "extensions/browser/uninstall_reason.h" |
| #include "extensions/common/extension.h" |
| +#include "extensions/common/manifest_constants.h" |
| #include "grit/chromium_strings.h" |
| #include "grit/generated_resources.h" |
| #include "ui/base/l10n/l10n_util.h" |
| @@ -113,6 +115,13 @@ void ExtensionContextMenuModel::ExecuteCommand(int command_id, |
| browser_->OpenURL(params); |
| break; |
| } |
| + case ALWAYS_ALLOW: { |
| + content::WebContents* web_contents = |
| + browser_->tab_strip_model()->GetActiveWebContents(); |
| + extensions::ActiveScriptController::GetForWebContents(web_contents) |
| + ->AddPersistedPermission(extension); |
| + break; |
| + } |
| case CONFIGURE: |
| DCHECK(!extensions::ManifestURL::GetOptionsPage(extension).is_empty()); |
| extensions::ExtensionTabUtil::OpenOptionsPage(extension, browser_); |
| @@ -177,6 +186,15 @@ void ExtensionContextMenuModel::InitMenu(const Extension* extension) { |
| base::ReplaceChars(extension_name, "&", "&&", &extension_name); |
| AddItem(NAME, base::UTF8ToUTF16(extension_name)); |
| AddSeparator(ui::NORMAL_SEPARATOR); |
| + |
| + // Add the "Always Allow" item for adding persisted permissions for script |
| + // injections if there is an active action for this extension (this action). |
| + content::WebContents* web_contents = |
| + browser_->tab_strip_model()->GetActiveWebContents(); |
| + if (extensions::ActiveScriptController::GetForWebContents(web_contents) |
| + ->HasActiveScriptAction(extension)) |
| + AddItemWithStringId(ALWAYS_ALLOW, IDS_EXTENSIONS_ALWAYS_ALLOW); |
|
not at google - send to devlin
2014/08/01 18:15:12
nit: the condition is 2 lines, so the body should
gpdavis
2014/08/01 20:06:06
Done.
|
| + |
| AddItemWithStringId(CONFIGURE, IDS_EXTENSIONS_OPTIONS_MENU_ITEM); |
| AddItem(UNINSTALL, l10n_util::GetStringUTF16(IDS_EXTENSIONS_UNINSTALL)); |
| if (extension_action_manager->GetBrowserAction(*extension)) |