#include "base/strings/string_split.h"
#include "base/strings/string_util.h"
#include "base/strings/utf_string_conversions.h"
-#include "chrome/browser/chrome_notification_types.h"
+#include "chrome/app/chrome_command_ids.h"
#include "chrome/browser/extensions/api/commands/commands.h"
#include "chrome/browser/extensions/extension_commands_global_registry.h"
-#include "chrome/browser/extensions/extension_function_registry.h"
#include "chrome/browser/extensions/extension_keybinding_registry.h"
-#include "chrome/browser/extensions/extension_service.h"
-#include "chrome/browser/extensions/extension_system.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/ui/accelerator_utils.h"
#include "chrome/common/extensions/api/commands/commands_handler.h"
+#include "chrome/common/extensions/manifest_handlers/ui_overrides_handler.h"
#include "chrome/common/pref_names.h"
-#include "components/user_prefs/pref_registry_syncable.h"
+#include "components/pref_registry/pref_registry_syncable.h"
#include "content/public/browser/notification_details.h"
#include "content/public/browser/notification_service.h"
+#include "extensions/browser/extension_function_registry.h"
+#include "extensions/browser/extension_prefs.h"
+#include "extensions/browser/extension_registry.h"
+#include "extensions/browser/extension_system.h"
+#include "extensions/browser/notification_types.h"
#include "extensions/common/feature_switch.h"
#include "extensions/common/manifest_constants.h"
+#include "extensions/common/permissions/permissions_data.h"
-using extensions::Extension;
-using extensions::ExtensionPrefs;
-
+namespace extensions {
namespace {
const char kExtension[] = "extension";
const char kCommandName[] = "command_name";
const char kGlobal[] = "global";
+// A preference that stores keybinding state associated with extension commands.
+const char kCommands[] = "commands";
+
+// Preference key name for saving the extension-suggested key.
+const char kSuggestedKey[] = "suggested_key";
+
+// Preference key name for saving whether the extension-suggested key was
+// actually assigned.
+const char kSuggestedKeyWasAssigned[] = "was_assigned";
+
// A preference that indicates that the initial keybindings for the given
// extension have been set.
const char kInitialBindingsHaveBeenAssigned[] = "initial_keybindings_set";
std::string GetPlatformKeybindingKeyForAccelerator(
const ui::Accelerator& accelerator, const std::string extension_id) {
- std::string key = extensions::Command::CommandPlatform() + ":" +
- extensions::Command::AcceleratorToString(accelerator);
+ std::string key = Command::CommandPlatform() + ":" +
+ Command::AcceleratorToString(accelerator);
// Media keys have a 1-to-many relationship with targets, unlike regular
// shortcut (1-to-1 relationship). That means two or more extensions can
// register for the same media key so the extension ID needs to be added to
// the key to make sure the key is unique.
- if (extensions::CommandService::IsMediaKey(accelerator))
+ if (Command::IsMediaKey(accelerator))
key += ":" + extension_id;
return key;
}
bool IsForCurrentPlatform(const std::string& key) {
- return StartsWithASCII(
- key, extensions::Command::CommandPlatform() + ":", true);
+ return StartsWithASCII(key, Command::CommandPlatform() + ":", true);
}
void SetInitialBindingsHaveBeenAssigned(
return assigned;
}
-bool IsWhitelistedGlobalShortcut(const extensions::Command& command) {
- // Non-global shortcuts are always allowed.
- if (!command.global())
- return true;
- // Global shortcuts must be (Ctrl|Command)-Shift-[0-9].
-#if defined OS_MACOSX
- if (!command.accelerator().IsCmdDown())
- return false;
-#else
- if (!command.accelerator().IsCtrlDown())
- return false;
-#endif
- if (!command.accelerator().IsShiftDown())
- return false;
- return (command.accelerator().key_code() >= ui::VKEY_0 &&
- command.accelerator().key_code() <= ui::VKEY_9);
+// Merge |suggested_key_prefs| into the saved preferences for the extension. We
+// merge rather than overwrite to preserve existing was_assigned preferences.
+void MergeSuggestedKeyPrefs(
+ const std::string& extension_id,
+ ExtensionPrefs* extension_prefs,
+ scoped_ptr<base::DictionaryValue> suggested_key_prefs) {
+ const base::DictionaryValue* current_prefs;
+ if (extension_prefs->ReadPrefAsDictionary(extension_id,
+ kCommands,
+ ¤t_prefs)) {
+ scoped_ptr<base::DictionaryValue> new_prefs(current_prefs->DeepCopy());
+ new_prefs->MergeDictionary(suggested_key_prefs.get());
+ suggested_key_prefs.reset(new_prefs.release());
+ }
+
+ extension_prefs->UpdateExtensionPref(extension_id,
+ kCommands,
+ suggested_key_prefs.release());
}
} // namespace
-namespace extensions {
-
// static
void CommandService::RegisterProfilePrefs(
user_prefs::PrefRegistrySyncable* registry) {
user_prefs::PrefRegistrySyncable::SYNCABLE_PREF);
}
-CommandService::CommandService(Profile* profile)
- : profile_(profile) {
+CommandService::CommandService(content::BrowserContext* context)
+ : profile_(Profile::FromBrowserContext(context)),
+ extension_registry_observer_(this) {
ExtensionFunctionRegistry::GetInstance()->
RegisterFunction<GetAllCommandsFunction>();
- registrar_.Add(this, chrome::NOTIFICATION_EXTENSION_INSTALLED,
- content::Source<Profile>(profile));
- registrar_.Add(this, chrome::NOTIFICATION_EXTENSION_UNINSTALLED,
- content::Source<Profile>(profile));
+ extension_registry_observer_.Add(ExtensionRegistry::Get(profile_));
}
CommandService::~CommandService() {
}
-static base::LazyInstance<ProfileKeyedAPIFactory<CommandService> >
-g_factory = LAZY_INSTANCE_INITIALIZER;
+static base::LazyInstance<BrowserContextKeyedAPIFactory<CommandService> >
+ g_factory = LAZY_INSTANCE_INITIALIZER;
// static
-ProfileKeyedAPIFactory<CommandService>* CommandService::GetFactoryInstance() {
+BrowserContextKeyedAPIFactory<CommandService>*
+CommandService::GetFactoryInstance() {
return g_factory.Pointer();
}
// static
-CommandService* CommandService::Get(Profile* profile) {
- return ProfileKeyedAPIFactory<CommandService>::GetForProfile(profile);
+CommandService* CommandService::Get(content::BrowserContext* context) {
+ return BrowserContextKeyedAPIFactory<CommandService>::Get(context);
}
// static
-bool CommandService::IsMediaKey(const ui::Accelerator& accelerator) {
- if (accelerator.modifiers() != 0)
- return false;
+bool CommandService::RemovesBookmarkShortcut(const Extension* extension) {
+ return UIOverrides::RemovesBookmarkShortcut(extension) &&
+ (extension->permissions_data()->HasAPIPermission(
+ APIPermission::kBookmarkManagerPrivate) ||
+ FeatureSwitch::enable_override_bookmarks_ui()->IsEnabled());
+}
- return (accelerator.key_code() == ui::VKEY_MEDIA_NEXT_TRACK ||
- accelerator.key_code() == ui::VKEY_MEDIA_PREV_TRACK ||
- accelerator.key_code() == ui::VKEY_MEDIA_PLAY_PAUSE ||
- accelerator.key_code() == ui::VKEY_MEDIA_STOP);
+// static
+bool CommandService::RemovesBookmarkOpenPagesShortcut(
+ const Extension* extension) {
+ return UIOverrides::RemovesBookmarkOpenPagesShortcut(extension) &&
+ (extension->permissions_data()->HasAPIPermission(
+ APIPermission::kBookmarkManagerPrivate) ||
+ FeatureSwitch::enable_override_bookmarks_ui()->IsEnabled());
}
-bool CommandService::GetBrowserActionCommand(
- const std::string& extension_id,
- QueryType type,
- extensions::Command* command,
- bool* active) {
+bool CommandService::GetBrowserActionCommand(const std::string& extension_id,
+ QueryType type,
+ Command* command,
+ bool* active) const {
return GetExtensionActionCommand(
extension_id, type, command, active, BROWSER_ACTION);
}
-bool CommandService::GetPageActionCommand(
- const std::string& extension_id,
- QueryType type,
- extensions::Command* command,
- bool* active) {
+bool CommandService::GetPageActionCommand(const std::string& extension_id,
+ QueryType type,
+ Command* command,
+ bool* active) const {
return GetExtensionActionCommand(
extension_id, type, command, active, PAGE_ACTION);
}
bool CommandService::GetNamedCommands(const std::string& extension_id,
QueryType type,
CommandScope scope,
- extensions::CommandMap* command_map) {
- ExtensionService* extension_service =
- ExtensionSystem::Get(profile_)->extension_service();
- if (!extension_service)
- return false; // Can occur during testing.
- const ExtensionSet* extensions = extension_service->extensions();
- const Extension* extension = extensions->GetByID(extension_id);
+ CommandMap* command_map) const {
+ const ExtensionSet& extensions =
+ ExtensionRegistry::Get(profile_)->enabled_extensions();
+ const Extension* extension = extensions.GetByID(extension_id);
CHECK(extension);
command_map->clear();
- const extensions::CommandMap* commands =
- CommandsInfo::GetNamedCommands(extension);
+ const CommandMap* commands = CommandsInfo::GetNamedCommands(extension);
if (!commands)
return false;
- extensions::CommandMap::const_iterator iter = commands->begin();
- for (; iter != commands->end(); ++iter) {
+ for (CommandMap::const_iterator iter = commands->begin();
+ iter != commands->end(); ++iter) {
// Look up to see if the user has overridden how the command should work.
- extensions::Command saved_command =
+ Command saved_command =
FindCommandByName(extension_id, iter->second.command_name());
ui::Accelerator shortcut_assigned = saved_command.accelerator();
if (type == ACTIVE_ONLY && shortcut_assigned.key_code() == ui::VKEY_UNKNOWN)
continue;
- extensions::Command command = iter->second;
+ Command command = iter->second;
if (scope != ANY_SCOPE && ((scope == GLOBAL) != saved_command.global()))
continue;
if (accelerator.key_code() == ui::VKEY_UNKNOWN)
return false;
+ // Nothing needs to be done if the existing command is the same as the desired
+ // new one.
+ Command existing_command = FindCommandByName(extension_id, command_name);
+ if (existing_command.accelerator() == accelerator &&
+ existing_command.global() == global)
+ return true;
+
// Media Keys are allowed to be used by named command only.
- DCHECK(!IsMediaKey(accelerator) ||
+ DCHECK(!Command::IsMediaKey(accelerator) ||
(command_name != manifest_values::kPageActionCommandEvent &&
command_name != manifest_values::kBrowserActionCommandEvent));
RemoveKeybindingPrefs(old_extension_id, old_command_name);
}
+ // If the command that is taking a new shortcut already has a shortcut, remove
+ // it before assigning the new one.
+ if (existing_command.accelerator().key_code() != ui::VKEY_UNKNOWN)
+ RemoveKeybindingPrefs(extension_id, command_name);
+
+ // Set the keybinding pref.
base::DictionaryValue* keybinding = new base::DictionaryValue();
keybinding->SetString(kExtension, extension_id);
keybinding->SetString(kCommandName, command_name);
bindings->Set(key, keybinding);
+ // Set the was_assigned pref for the suggested key.
+ scoped_ptr<base::DictionaryValue> command_keys(new base::DictionaryValue);
+ command_keys->SetBoolean(kSuggestedKeyWasAssigned, true);
+ scoped_ptr<base::DictionaryValue> suggested_key_prefs(
+ new base::DictionaryValue);
+ suggested_key_prefs->Set(command_name, command_keys.release());
+ MergeSuggestedKeyPrefs(extension_id,
+ ExtensionPrefs::Get(profile_),
+ suggested_key_prefs.Pass());
+
std::pair<const std::string, const std::string> details =
std::make_pair(extension_id, command_name);
content::NotificationService::current()->Notify(
- chrome::NOTIFICATION_EXTENSION_COMMAND_ADDED,
+ extensions::NOTIFICATION_EXTENSION_COMMAND_ADDED,
content::Source<Profile>(profile_),
- content::Details<
- std::pair<const std::string, const std::string> >(&details));
+ content::Details<std::pair<const std::string, const std::string> >(
+ &details));
return true;
}
-void CommandService::Observe(
- int type,
- const content::NotificationSource& source,
- const content::NotificationDetails& details) {
- switch (type) {
- case chrome::NOTIFICATION_EXTENSION_INSTALLED:
- AssignInitialKeybindings(
- content::Details<const InstalledExtensionInfo>(details)->extension);
- break;
- case chrome::NOTIFICATION_EXTENSION_UNINSTALLED:
- RemoveKeybindingPrefs(
- content::Details<const Extension>(details)->id(),
- std::string());
- break;
- default:
- NOTREACHED();
- break;
- }
+void CommandService::OnExtensionWillBeInstalled(
+ content::BrowserContext* browser_context,
+ const Extension* extension,
+ bool is_update,
+ bool from_ephemeral,
+ const std::string& old_name) {
+ UpdateKeybindings(extension);
+}
+
+void CommandService::OnExtensionUninstalled(
+ content::BrowserContext* browser_context,
+ const Extension* extension,
+ extensions::UninstallReason reason) {
+ RemoveKeybindingPrefs(extension->id(), std::string());
}
void CommandService::UpdateKeybindingPrefs(const std::string& extension_id,
const std::string& command_name,
const std::string& keystroke) {
- extensions::Command command = FindCommandByName(extension_id, command_name);
+ Command command = FindCommandByName(extension_id, command_name);
// The extension command might be assigned another shortcut. Remove that
// shortcut before proceeding.
bool CommandService::SetScope(const std::string& extension_id,
const std::string& command_name,
bool global) {
- extensions::Command command = FindCommandByName(extension_id, command_name);
+ Command command = FindCommandByName(extension_id, command_name);
if (global == command.global())
return false;
return true;
}
-Command CommandService::FindCommandByName(
- const std::string& extension_id, const std::string& command) {
+Command CommandService::FindCommandByName(const std::string& extension_id,
+ const std::string& command) const {
const base::DictionaryValue* bindings =
profile_->GetPrefs()->GetDictionary(prefs::kExtensionCommands);
for (base::DictionaryValue::Iterator it(*bindings); !it.IsAtEnd();
if (!IsForCurrentPlatform(shortcut))
continue;
bool global = false;
- if (FeatureSwitch::global_commands()->IsEnabled())
- item->GetBoolean(kGlobal, &global);
+ item->GetBoolean(kGlobal, &global);
std::vector<std::string> tokens;
base::SplitString(shortcut, ':', &tokens);
return Command();
}
-void CommandService::AssignInitialKeybindings(const Extension* extension) {
- const extensions::CommandMap* commands =
- CommandsInfo::GetNamedCommands(extension);
+bool CommandService::GetBoundExtensionCommand(
+ const std::string& extension_id,
+ const ui::Accelerator& accelerator,
+ Command* command,
+ ExtensionCommandType* command_type) const {
+ const Extension* extension =
+ ExtensionRegistry::Get(profile_)
+ ->GetExtensionById(extension_id, ExtensionRegistry::ENABLED);
+ CHECK(extension);
+
+ Command prospective_command;
+ CommandMap command_map;
+ bool active = false;
+ if (GetBrowserActionCommand(extension_id,
+ CommandService::ACTIVE_ONLY,
+ &prospective_command,
+ &active) &&
+ active && accelerator == prospective_command.accelerator()) {
+ if (command)
+ *command = prospective_command;
+ if (command_type)
+ *command_type = BROWSER_ACTION;
+ return true;
+ } else if (GetPageActionCommand(extension_id,
+ CommandService::ACTIVE_ONLY,
+ &prospective_command,
+ &active) &&
+ active && accelerator == prospective_command.accelerator()) {
+ if (command)
+ *command = prospective_command;
+ if (command_type)
+ *command_type = PAGE_ACTION;
+ return true;
+ } else if (GetNamedCommands(extension_id,
+ CommandService::ACTIVE_ONLY,
+ CommandService::REGULAR,
+ &command_map)) {
+ for (CommandMap::const_iterator it = command_map.begin();
+ it != command_map.end();
+ ++it) {
+ if (accelerator == it->second.accelerator()) {
+ if (command)
+ *command = it->second;
+ if (command_type)
+ *command_type = NAMED;
+ return true;
+ }
+ }
+ }
+ return false;
+}
+
+bool CommandService::OverridesBookmarkShortcut(
+ const Extension* extension) const {
+ return RemovesBookmarkShortcut(extension) &&
+ GetBoundExtensionCommand(
+ extension->id(),
+ chrome::GetPrimaryChromeAcceleratorForCommandId(IDC_BOOKMARK_PAGE),
+ NULL,
+ NULL);
+}
+
+void CommandService::UpdateKeybindings(const Extension* extension) {
+ const ExtensionSet& extensions =
+ ExtensionRegistry::Get(profile_)->enabled_extensions();
+ // The extension is not added to the profile by this point on first install,
+ // so don't try to check for existing keybindings.
+ if (extensions.GetByID(extension->id()))
+ RemoveRelinquishedKeybindings(extension);
+ AssignKeybindings(extension);
+ UpdateExtensionSuggestedCommandPrefs(extension);
+ RemoveDefunctExtensionSuggestedCommandPrefs(extension);
+}
+
+void CommandService::RemoveRelinquishedKeybindings(const Extension* extension) {
+ // Remove keybindings if they have been removed by the extension and the user
+ // has not modified them.
+ CommandMap existing_command_map;
+ if (GetNamedCommands(extension->id(),
+ CommandService::ACTIVE_ONLY,
+ CommandService::REGULAR,
+ &existing_command_map)) {
+ const CommandMap* new_command_map =
+ CommandsInfo::GetNamedCommands(extension);
+ for (CommandMap::const_iterator it = existing_command_map.begin();
+ it != existing_command_map.end(); ++it) {
+ std::string command_name = it->first;
+ if (new_command_map->find(command_name) == new_command_map->end() &&
+ !IsCommandShortcutUserModified(extension, command_name)) {
+ RemoveKeybindingPrefs(extension->id(), command_name);
+ }
+ }
+ }
+
+ Command existing_browser_action_command;
+ const Command* new_browser_action_command =
+ CommandsInfo::GetBrowserActionCommand(extension);
+ if (GetBrowserActionCommand(extension->id(),
+ CommandService::ACTIVE_ONLY,
+ &existing_browser_action_command,
+ NULL) &&
+ // The browser action command may be defaulted to an unassigned
+ // accelerator if a browser action is specified by the extension but a
+ // keybinding is not declared. See
+ // CommandsHandler::MaybeSetBrowserActionDefault.
+ (!new_browser_action_command ||
+ new_browser_action_command->accelerator().key_code() ==
+ ui::VKEY_UNKNOWN) &&
+ !IsCommandShortcutUserModified(
+ extension,
+ existing_browser_action_command.command_name())) {
+ RemoveKeybindingPrefs(extension->id(),
+ existing_browser_action_command.command_name());
+ }
+
+ Command existing_page_action_command;
+ if (GetPageActionCommand(extension->id(),
+ CommandService::ACTIVE_ONLY,
+ &existing_page_action_command,
+ NULL) &&
+ !CommandsInfo::GetPageActionCommand(extension) &&
+ !IsCommandShortcutUserModified(
+ extension,
+ existing_page_action_command.command_name())) {
+ RemoveKeybindingPrefs(extension->id(),
+ existing_page_action_command.command_name());
+ }
+}
+
+void CommandService::AssignKeybindings(const Extension* extension) {
+ const CommandMap* commands = CommandsInfo::GetNamedCommands(extension);
if (!commands)
return;
- ExtensionService* extension_service =
- ExtensionSystem::Get(profile_)->extension_service();
- ExtensionPrefs* extension_prefs = extension_service->extension_prefs();
- if (InitialBindingsHaveBeenAssigned(extension_prefs, extension->id()))
- return;
- SetInitialBindingsHaveBeenAssigned(extension_prefs, extension->id());
-
- extensions::CommandMap::const_iterator iter = commands->begin();
- for (; iter != commands->end(); ++iter) {
- // Make sure registered Chrome shortcuts cannot be automatically assigned
- // (overwritten) by extension developers. Media keys are an exception here.
- if ((!chrome::IsChromeAccelerator(iter->second.accelerator(), profile_) &&
- IsWhitelistedGlobalShortcut(iter->second)) ||
- extensions::CommandService::IsMediaKey(iter->second.accelerator())) {
- AddKeybindingPref(iter->second.accelerator(),
+ ExtensionPrefs* extension_prefs = ExtensionPrefs::Get(profile_);
+ // TODO(wittman): remove use of this pref after M37 hits stable.
+ if (!InitialBindingsHaveBeenAssigned(extension_prefs, extension->id()))
+ SetInitialBindingsHaveBeenAssigned(extension_prefs, extension->id());
+
+ for (CommandMap::const_iterator iter = commands->begin();
+ iter != commands->end(); ++iter) {
+ const Command command = iter->second;
+ if (CanAutoAssign(command, extension)) {
+ AddKeybindingPref(command.accelerator(),
extension->id(),
- iter->second.command_name(),
+ command.command_name(),
false, // Overwriting not allowed.
- iter->second.global());
+ command.global());
}
}
- const extensions::Command* browser_action_command =
+ const Command* browser_action_command =
CommandsInfo::GetBrowserActionCommand(extension);
- if (browser_action_command) {
- if (!chrome::IsChromeAccelerator(
- browser_action_command->accelerator(), profile_)) {
- AddKeybindingPref(browser_action_command->accelerator(),
- extension->id(),
- browser_action_command->command_name(),
- false, // Overwriting not allowed.
- false); // Browser actions can't be global.
+ if (browser_action_command &&
+ CanAutoAssign(*browser_action_command, extension)) {
+ AddKeybindingPref(browser_action_command->accelerator(),
+ extension->id(),
+ browser_action_command->command_name(),
+ false, // Overwriting not allowed.
+ false); // Not global.
+ }
+
+ const Command* page_action_command =
+ CommandsInfo::GetPageActionCommand(extension);
+ if (page_action_command && CanAutoAssign(*page_action_command, extension)) {
+ AddKeybindingPref(page_action_command->accelerator(),
+ extension->id(),
+ page_action_command->command_name(),
+ false, // Overwriting not allowed.
+ false); // Not global.
+ }
+}
+
+bool CommandService::CanAutoAssign(const Command &command,
+ const Extension* extension) {
+ // Extensions are allowed to auto-assign updated keys if the user has not
+ // changed from the previous value.
+ if (IsCommandShortcutUserModified(extension, command.command_name()))
+ return false;
+
+ // Media Keys are non-exclusive, so allow auto-assigning them.
+ if (Command::IsMediaKey(command.accelerator()))
+ return true;
+
+ if (command.global()) {
+ using namespace extensions;
+ if (command.command_name() == manifest_values::kBrowserActionCommandEvent ||
+ command.command_name() == manifest_values::kPageActionCommandEvent)
+ return false; // Browser and page actions are not global in nature.
+
+ if (extension->permissions_data()->HasAPIPermission(
+ APIPermission::kCommandsAccessibility))
+ return true;
+
+ // Global shortcuts are restricted to (Ctrl|Command)+Shift+[0-9].
+#if defined OS_MACOSX
+ if (!command.accelerator().IsCmdDown())
+ return false;
+#else
+ if (!command.accelerator().IsCtrlDown())
+ return false;
+#endif
+ if (!command.accelerator().IsShiftDown())
+ return false;
+ return (command.accelerator().key_code() >= ui::VKEY_0 &&
+ command.accelerator().key_code() <= ui::VKEY_9);
+ } else {
+ // Not a global command, check if Chrome shortcut and whether
+ // we can override it.
+ if (command.accelerator() ==
+ chrome::GetPrimaryChromeAcceleratorForCommandId(IDC_BOOKMARK_PAGE) &&
+ CommandService::RemovesBookmarkShortcut(extension)) {
+ // If this check fails it either means we have an API to override a
+ // key that isn't a ChromeAccelerator (and the API can therefore be
+ // deprecated) or the IsChromeAccelerator isn't consistently
+ // returning true for all accelerators.
+ DCHECK(chrome::IsChromeAccelerator(command.accelerator(), profile_));
+ return true;
}
+
+ return !chrome::IsChromeAccelerator(command.accelerator(), profile_);
+ }
+}
+
+void CommandService::UpdateExtensionSuggestedCommandPrefs(
+ const Extension* extension) {
+ scoped_ptr<base::DictionaryValue> suggested_key_prefs(
+ new base::DictionaryValue);
+
+ const CommandMap* commands = CommandsInfo::GetNamedCommands(extension);
+ if (commands) {
+ for (CommandMap::const_iterator iter = commands->begin();
+ iter != commands->end(); ++iter) {
+ const Command command = iter->second;
+ scoped_ptr<base::DictionaryValue> command_keys(new base::DictionaryValue);
+ command_keys->SetString(
+ kSuggestedKey,
+ Command::AcceleratorToString(command.accelerator()));
+ suggested_key_prefs->Set(command.command_name(), command_keys.release());
+ }
+ }
+
+ const Command* browser_action_command =
+ CommandsInfo::GetBrowserActionCommand(extension);
+ // The browser action command may be defaulted to an unassigned accelerator if
+ // a browser action is specified by the extension but a keybinding is not
+ // declared. See CommandsHandler::MaybeSetBrowserActionDefault.
+ if (browser_action_command &&
+ browser_action_command->accelerator().key_code() != ui::VKEY_UNKNOWN) {
+ scoped_ptr<base::DictionaryValue> command_keys(new base::DictionaryValue);
+ command_keys->SetString(
+ kSuggestedKey,
+ Command::AcceleratorToString(browser_action_command->accelerator()));
+ suggested_key_prefs->Set(browser_action_command->command_name(),
+ command_keys.release());
}
- const extensions::Command* page_action_command =
+ const Command* page_action_command =
CommandsInfo::GetPageActionCommand(extension);
if (page_action_command) {
- if (!chrome::IsChromeAccelerator(
- page_action_command->accelerator(), profile_)) {
- AddKeybindingPref(page_action_command->accelerator(),
- extension->id(),
- page_action_command->command_name(),
- false, // Overwriting not allowed.
- false); // Page actions can't be global.
+ scoped_ptr<base::DictionaryValue> command_keys(new base::DictionaryValue);
+ command_keys->SetString(
+ kSuggestedKey,
+ Command::AcceleratorToString(page_action_command->accelerator()));
+ suggested_key_prefs->Set(page_action_command->command_name(),
+ command_keys.release());
+ }
+
+ // Merge into current prefs, if present.
+ MergeSuggestedKeyPrefs(extension->id(),
+ ExtensionPrefs::Get(profile_),
+ suggested_key_prefs.Pass());
+}
+
+void CommandService::RemoveDefunctExtensionSuggestedCommandPrefs(
+ const Extension* extension) {
+ ExtensionPrefs* extension_prefs = ExtensionPrefs::Get(profile_);
+ const base::DictionaryValue* current_prefs = NULL;
+ extension_prefs->ReadPrefAsDictionary(extension->id(),
+ kCommands,
+ ¤t_prefs);
+
+ if (current_prefs) {
+ scoped_ptr<base::DictionaryValue> suggested_key_prefs(
+ current_prefs->DeepCopy());
+ const CommandMap* named_commands =
+ CommandsInfo::GetNamedCommands(extension);
+ const Command* browser_action_command =
+ CommandsInfo::GetBrowserActionCommand(extension);
+ for (base::DictionaryValue::Iterator it(*current_prefs);
+ !it.IsAtEnd(); it.Advance()) {
+ if (it.key() == manifest_values::kBrowserActionCommandEvent) {
+ // The browser action command may be defaulted to an unassigned
+ // accelerator if a browser action is specified by the extension but a
+ // keybinding is not declared. See
+ // CommandsHandler::MaybeSetBrowserActionDefault.
+ if (!browser_action_command ||
+ browser_action_command->accelerator().key_code() ==
+ ui::VKEY_UNKNOWN) {
+ suggested_key_prefs->Remove(it.key(), NULL);
+ }
+ } else if (it.key() == manifest_values::kPageActionCommandEvent) {
+ if (!CommandsInfo::GetPageActionCommand(extension))
+ suggested_key_prefs->Remove(it.key(), NULL);
+ } else if (named_commands) {
+ if (named_commands->find(it.key()) == named_commands->end())
+ suggested_key_prefs->Remove(it.key(), NULL);
+ }
}
+
+ extension_prefs->UpdateExtensionPref(extension->id(),
+ kCommands,
+ suggested_key_prefs.release());
}
}
+bool CommandService::IsCommandShortcutUserModified(
+ const Extension* extension,
+ const std::string& command_name) {
+ // Get the previous suggested key, if any.
+ ui::Accelerator suggested_key;
+ bool suggested_key_was_assigned = false;
+ ExtensionPrefs* extension_prefs = ExtensionPrefs::Get(profile_);
+ const base::DictionaryValue* commands_prefs = NULL;
+ const base::DictionaryValue* suggested_key_prefs = NULL;
+ if (extension_prefs->ReadPrefAsDictionary(extension->id(),
+ kCommands,
+ &commands_prefs) &&
+ commands_prefs->GetDictionary(command_name, &suggested_key_prefs)) {
+ std::string suggested_key_string;
+ if (suggested_key_prefs->GetString(kSuggestedKey, &suggested_key_string)) {
+ suggested_key = Command::StringToAccelerator(suggested_key_string,
+ command_name);
+ }
+
+ suggested_key_prefs->GetBoolean(kSuggestedKeyWasAssigned,
+ &suggested_key_was_assigned);
+ }
+
+ // Get the active shortcut from the prefs, if any.
+ Command active_command = FindCommandByName(extension->id(), command_name);
+
+ return suggested_key_was_assigned ?
+ active_command.accelerator() != suggested_key :
+ active_command.accelerator().key_code() != ui::VKEY_UNKNOWN;
+}
+
+bool CommandService::IsKeybindingChanging(const Extension* extension,
+ const std::string& command_name) {
+ // Get the new assigned command, if any.
+ Command new_command;
+ if (command_name == manifest_values::kBrowserActionCommandEvent) {
+ new_command = *CommandsInfo::GetBrowserActionCommand(extension);
+ } else if (command_name == manifest_values::kPageActionCommandEvent) {
+ new_command = *CommandsInfo::GetPageActionCommand(extension);
+ } else { // This is a named command.
+ const CommandMap* named_commands =
+ CommandsInfo::GetNamedCommands(extension);
+ if (named_commands) {
+ CommandMap::const_iterator loc = named_commands->find(command_name);
+ if (loc != named_commands->end())
+ new_command = loc->second;
+ }
+ }
+
+ return Command::StringToAccelerator(
+ GetSuggestedKeyPref(extension, command_name), command_name) !=
+ new_command.accelerator();
+}
+
+std::string CommandService::GetSuggestedKeyPref(
+ const Extension* extension,
+ const std::string& command_name) {
+ // Get the previous suggested key, if any.
+ ui::Accelerator suggested_key;
+ ExtensionPrefs* extension_prefs = ExtensionPrefs::Get(profile_);
+ const base::DictionaryValue* commands_prefs = NULL;
+ if (extension_prefs->ReadPrefAsDictionary(extension->id(),
+ kCommands,
+ &commands_prefs)) {
+ const base::DictionaryValue* suggested_key_prefs = NULL;
+ std::string suggested_key;
+ if (commands_prefs->GetDictionary(command_name, &suggested_key_prefs) &&
+ suggested_key_prefs->GetString(kSuggestedKey, &suggested_key)) {
+ return suggested_key;
+ }
+ }
+
+ return std::string();
+}
+
void CommandService::RemoveKeybindingPrefs(const std::string& extension_id,
const std::string& command_name) {
DictionaryPrefUpdate updater(profile_->GetPrefs(),
std::pair<const std::string, const std::string> details =
std::make_pair(extension_id, command_name);
content::NotificationService::current()->Notify(
- chrome::NOTIFICATION_EXTENSION_COMMAND_REMOVED,
+ extensions::NOTIFICATION_EXTENSION_COMMAND_REMOVED,
content::Source<Profile>(profile_),
- content::Details<
- std::pair<const std::string, const std::string> >(&details));
+ content::Details<std::pair<const std::string, const std::string> >(
+ &details));
}
}
bool CommandService::GetExtensionActionCommand(
const std::string& extension_id,
QueryType query_type,
- extensions::Command* command,
+ Command* command,
bool* active,
- ExtensionActionType action_type) {
- ExtensionService* service =
- ExtensionSystem::Get(profile_)->extension_service();
- if (!service)
- return false; // Can happen in tests.
- const ExtensionSet* extensions = service->extensions();
- const Extension* extension = extensions->GetByID(extension_id);
+ ExtensionCommandType action_type) const {
+ const ExtensionSet& extensions =
+ ExtensionRegistry::Get(profile_)->enabled_extensions();
+ const Extension* extension = extensions.GetByID(extension_id);
CHECK(extension);
if (active)
*active = false;
- const extensions::Command* requested_command = NULL;
+ const Command* requested_command = NULL;
switch (action_type) {
case BROWSER_ACTION:
requested_command = CommandsInfo::GetBrowserActionCommand(extension);
case PAGE_ACTION:
requested_command = CommandsInfo::GetPageActionCommand(extension);
break;
+ case NAMED:
+ NOTREACHED();
+ return false;
}
if (!requested_command)
return false;
// Look up to see if the user has overridden how the command should work.
- extensions::Command saved_command =
+ Command saved_command =
FindCommandByName(extension_id, requested_command->command_name());
ui::Accelerator shortcut_assigned = saved_command.accelerator();
}
template <>
-void ProfileKeyedAPIFactory<CommandService>::DeclareFactoryDependencies() {
+void
+BrowserContextKeyedAPIFactory<CommandService>::DeclareFactoryDependencies() {
DependsOn(ExtensionCommandsGlobalRegistry::GetFactoryInstance());
}