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

Unified Diff: chrome/browser/extensions/api/extension_action/extension_actions_api.cc

Issue 10914244: Remove support for page_action.icons, and the legacy code surrounding it. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: docs Created 8 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/api/extension_action/extension_actions_api.cc
diff --git a/chrome/browser/extensions/api/extension_action/extension_actions_api.cc b/chrome/browser/extensions/api/extension_action/extension_actions_api.cc
index 78a1815af961a2a4634ca560f2fe863649158e59..1a69d0d84b45e9c3b04862122736bd56c9ae3b38 100644
--- a/chrome/browser/extensions/api/extension_action/extension_actions_api.cc
+++ b/chrome/browser/extensions/api/extension_action/extension_actions_api.cc
@@ -440,7 +440,6 @@ bool ExtensionActionHideFunction::RunExtensionAction() {
bool ExtensionActionSetIconFunction::RunExtensionAction() {
// setIcon can take a variant argument: either a dictionary of canvas
// ImageData, or an icon index.
- int icon_index;
base::DictionaryValue* canvas_set = NULL;
if (details_->GetDictionary("imageData", &canvas_set)) {
gfx::ImageSkia icon;
@@ -458,19 +457,6 @@ bool ExtensionActionSetIconFunction::RunExtensionAction() {
}
extension_action_->SetIcon(tab_id_, gfx::Image(icon));
- } else if (details_->GetInteger("iconIndex", &icon_index)) {
- // If --enable-script-badges is on there might legitimately be an iconIndex
- // set. Until we decide what to do with that, ignore.
- if (!GetExtension()->page_action())
- return true;
- if (icon_index < 0 ||
- static_cast<size_t>(icon_index) >=
- extension_action_->icon_paths()->size()) {
- error_ = kIconIndexOutOfBounds;
- return false;
- }
- extension_action_->SetIcon(tab_id_, gfx::Image());
- extension_action_->SetIconIndex(tab_id_, icon_index);
} else {
EXTENSION_FUNCTION_VALIDATE(false);
}

Powered by Google App Engine
This is Rietveld 408576698