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

Unified Diff: chrome/common/extensions/api/extension_action/action_info.cc

Issue 11588004: Move ScriptBadge, ActionInfo out of Extension; preparation for BrowserAction (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Latest master for CQ Created 7 years, 11 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/common/extensions/api/extension_action/action_info.cc
diff --git a/chrome/common/extensions/api/extension_action/action_info.cc b/chrome/common/extensions/api/extension_action/action_info.cc
new file mode 100644
index 0000000000000000000000000000000000000000..7d7ac1030313954e349b2a6378df3de760be470e
--- /dev/null
+++ b/chrome/common/extensions/api/extension_action/action_info.cc
@@ -0,0 +1,53 @@
+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/common/extensions/api/extension_action/action_info.h"
+
+#include "base/memory/scoped_ptr.h"
+#include "chrome/common/extensions/extension.h"
+#include "chrome/common/extensions/extension_manifest_constants.h"
+
+namespace extensions {
+
+namespace {
+
+// The manifest data container for the ActionInfos for BrowserActions and
+// ScriptBadges.
+struct ActionInfoData : public Extension::ManifestData {
+ explicit ActionInfoData(ActionInfo* action_info);
+ virtual ~ActionInfoData();
+
+ // The action associated with the BrowserAction or ScriptBadge.
+ // This is never NULL for ScriptBadge.
+ scoped_ptr<ActionInfo> action_info;
+};
+
+ActionInfoData::ActionInfoData(ActionInfo* info) : action_info(info) {
+}
+
+ActionInfoData::~ActionInfoData() {
+}
+
+} // namespace
+
+ActionInfo::ActionInfo() {
+}
+
+ActionInfo::~ActionInfo() {
+}
+
+// static
+const ActionInfo* ActionInfo::GetScriptBadgeInfo(const Extension* extension) {
+ ActionInfoData* data = static_cast<ActionInfoData*>(
+ extension->GetManifestData(extension_manifest_keys::kScriptBadge));
+ return data ? data->action_info.get() : NULL;
+}
+
+// static
+void ActionInfo::SetScriptBadgeInfo(Extension* extension, ActionInfo* info) {
+ extension->SetManifestData(extension_manifest_keys::kScriptBadge,
+ new ActionInfoData(info));
+}
+
+} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698