Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(40)

Side by Side Diff: chrome/common/extensions/api/commands/commands_handler.cc

Issue 23744004: Move the rest of extension_manifest_constants to top-level extensions/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Latest master Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/common/extensions/api/commands/commands_handler.h" 5 #include "chrome/common/extensions/api/commands/commands_handler.h"
6 6
7 #include "base/strings/string_number_conversions.h" 7 #include "base/strings/string_number_conversions.h"
8 #include "base/strings/utf_string_conversions.h" 8 #include "base/strings/utf_string_conversions.h"
9 #include "base/values.h" 9 #include "base/values.h"
10 #include "chrome/common/extensions/extension_manifest_constants.h"
11 #include "extensions/common/error_utils.h" 10 #include "extensions/common/error_utils.h"
11 #include "extensions/common/manifest_constants.h"
12 12
13 namespace extensions { 13 namespace extensions {
14 14
15 namespace keys = manifest_keys; 15 namespace keys = manifest_keys;
16 16
17 namespace { 17 namespace {
18 // The maximum number of commands (including page action/browser actions) with a 18 // The maximum number of commands (including page action/browser actions) with a
19 // keybinding an extension can have. 19 // keybinding an extension can have.
20 const int kMaxCommandsWithKeybindingPerExtension = 4; 20 const int kMaxCommandsWithKeybindingPerExtension = 4;
21 } // namespace 21 } // namespace
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
65 if (!extension->manifest()->HasKey(keys::kCommands)) { 65 if (!extension->manifest()->HasKey(keys::kCommands)) {
66 scoped_ptr<CommandsInfo> commands_info(new CommandsInfo); 66 scoped_ptr<CommandsInfo> commands_info(new CommandsInfo);
67 MaybeSetBrowserActionDefault(extension, commands_info.get()); 67 MaybeSetBrowserActionDefault(extension, commands_info.get());
68 extension->SetManifestData(keys::kCommands, 68 extension->SetManifestData(keys::kCommands,
69 commands_info.release()); 69 commands_info.release());
70 return true; 70 return true;
71 } 71 }
72 72
73 const base::DictionaryValue* dict = NULL; 73 const base::DictionaryValue* dict = NULL;
74 if (!extension->manifest()->GetDictionary(keys::kCommands, &dict)) { 74 if (!extension->manifest()->GetDictionary(keys::kCommands, &dict)) {
75 *error = ASCIIToUTF16(extension_manifest_errors::kInvalidCommandsKey); 75 *error = ASCIIToUTF16(manifest_errors::kInvalidCommandsKey);
76 return false; 76 return false;
77 } 77 }
78 78
79 scoped_ptr<CommandsInfo> commands_info(new CommandsInfo); 79 scoped_ptr<CommandsInfo> commands_info(new CommandsInfo);
80 80
81 int command_index = 0; 81 int command_index = 0;
82 int keybindings_found = 0; 82 int keybindings_found = 0;
83 for (DictionaryValue::Iterator iter(*dict); !iter.IsAtEnd(); 83 for (DictionaryValue::Iterator iter(*dict); !iter.IsAtEnd();
84 iter.Advance()) { 84 iter.Advance()) {
85 ++command_index; 85 ++command_index;
86 86
87 const DictionaryValue* command = NULL; 87 const DictionaryValue* command = NULL;
88 if (!iter.value().GetAsDictionary(&command)) { 88 if (!iter.value().GetAsDictionary(&command)) {
89 *error = ErrorUtils::FormatErrorMessageUTF16( 89 *error = ErrorUtils::FormatErrorMessageUTF16(
90 extension_manifest_errors::kInvalidKeyBindingDictionary, 90 manifest_errors::kInvalidKeyBindingDictionary,
91 base::IntToString(command_index)); 91 base::IntToString(command_index));
92 return false; 92 return false;
93 } 93 }
94 94
95 scoped_ptr<extensions::Command> binding(new Command()); 95 scoped_ptr<extensions::Command> binding(new Command());
96 if (!binding->Parse(command, iter.key(), command_index, error)) 96 if (!binding->Parse(command, iter.key(), command_index, error))
97 return false; // |error| already set. 97 return false; // |error| already set.
98 98
99 if (binding->accelerator().key_code() != ui::VKEY_UNKNOWN) { 99 if (binding->accelerator().key_code() != ui::VKEY_UNKNOWN) {
100 if (++keybindings_found > kMaxCommandsWithKeybindingPerExtension) { 100 if (++keybindings_found > kMaxCommandsWithKeybindingPerExtension) {
101 *error = ErrorUtils::FormatErrorMessageUTF16( 101 *error = ErrorUtils::FormatErrorMessageUTF16(
102 extension_manifest_errors::kInvalidKeyBindingTooMany, 102 manifest_errors::kInvalidKeyBindingTooMany,
103 base::IntToString(kMaxCommandsWithKeybindingPerExtension)); 103 base::IntToString(kMaxCommandsWithKeybindingPerExtension));
104 return false; 104 return false;
105 } 105 }
106 } 106 }
107 107
108 std::string command_name = binding->command_name(); 108 std::string command_name = binding->command_name();
109 if (command_name == extension_manifest_values::kBrowserActionCommandEvent) { 109 if (command_name == manifest_values::kBrowserActionCommandEvent) {
110 commands_info->browser_action_command.reset(binding.release()); 110 commands_info->browser_action_command.reset(binding.release());
111 } else if (command_name == 111 } else if (command_name ==
112 extension_manifest_values::kPageActionCommandEvent) { 112 manifest_values::kPageActionCommandEvent) {
113 commands_info->page_action_command.reset(binding.release()); 113 commands_info->page_action_command.reset(binding.release());
114 } else if (command_name == 114 } else if (command_name ==
115 extension_manifest_values::kScriptBadgeCommandEvent) { 115 manifest_values::kScriptBadgeCommandEvent) {
116 commands_info->script_badge_command.reset(binding.release()); 116 commands_info->script_badge_command.reset(binding.release());
117 } else { 117 } else {
118 if (command_name[0] != '_') // All commands w/underscore are reserved. 118 if (command_name[0] != '_') // All commands w/underscore are reserved.
119 commands_info->named_commands[command_name] = *binding.get(); 119 commands_info->named_commands[command_name] = *binding.get();
120 } 120 }
121 } 121 }
122 122
123 MaybeSetBrowserActionDefault(extension, commands_info.get()); 123 MaybeSetBrowserActionDefault(extension, commands_info.get());
124 124
125 extension->SetManifestData(keys::kCommands, 125 extension->SetManifestData(keys::kCommands,
126 commands_info.release()); 126 commands_info.release());
127 return true; 127 return true;
128 } 128 }
129 129
130 bool CommandsHandler::AlwaysParseForType(Manifest::Type type) const { 130 bool CommandsHandler::AlwaysParseForType(Manifest::Type type) const {
131 return type == Manifest::TYPE_EXTENSION || 131 return type == Manifest::TYPE_EXTENSION ||
132 type == Manifest::TYPE_LEGACY_PACKAGED_APP || 132 type == Manifest::TYPE_LEGACY_PACKAGED_APP ||
133 type == Manifest::TYPE_PLATFORM_APP; 133 type == Manifest::TYPE_PLATFORM_APP;
134 } 134 }
135 135
136 void CommandsHandler::MaybeSetBrowserActionDefault(const Extension* extension, 136 void CommandsHandler::MaybeSetBrowserActionDefault(const Extension* extension,
137 CommandsInfo* info) { 137 CommandsInfo* info) {
138 if (extension->manifest()->HasKey(keys::kBrowserAction) && 138 if (extension->manifest()->HasKey(keys::kBrowserAction) &&
139 !info->browser_action_command.get()) { 139 !info->browser_action_command.get()) {
140 info->browser_action_command.reset( 140 info->browser_action_command.reset(
141 new Command(extension_manifest_values::kBrowserActionCommandEvent, 141 new Command(manifest_values::kBrowserActionCommandEvent,
142 string16(), 142 string16(),
143 std::string())); 143 std::string()));
144 } 144 }
145 } 145 }
146 146
147 const std::vector<std::string> CommandsHandler::Keys() const { 147 const std::vector<std::string> CommandsHandler::Keys() const {
148 return SingleKey(keys::kCommands); 148 return SingleKey(keys::kCommands);
149 } 149 }
150 150
151 } // namespace extensions 151 } // namespace extensions
OLDNEW
« no previous file with comments | « chrome/chrome_common.gypi ('k') | chrome/common/extensions/api/commands/commands_manifest_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698