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

Unified Diff: chrome/common/extensions/api/media_galleries_private/media_galleries_handler.cc

Issue 338473003: Remove media_galleries_handlers - not used. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 6 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/media_galleries_private/media_galleries_handler.cc
diff --git a/chrome/common/extensions/api/media_galleries_private/media_galleries_handler.cc b/chrome/common/extensions/api/media_galleries_private/media_galleries_handler.cc
deleted file mode 100644
index a973036440e71fb54407b4367a33314b735abf62..0000000000000000000000000000000000000000
--- a/chrome/common/extensions/api/media_galleries_private/media_galleries_handler.cc
+++ /dev/null
@@ -1,148 +0,0 @@
-// Copyright 2013 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/media_galleries_private/media_galleries_handler.h"
-
-#include "base/logging.h"
-#include "base/strings/string_number_conversions.h"
-#include "base/strings/string_util.h"
-#include "base/strings/utf_string_conversions.h"
-#include "base/values.h"
-#include "chrome/common/extensions/extension_constants.h"
-#include "extensions/common/error_utils.h"
-#include "extensions/common/manifest.h"
-#include "extensions/common/manifest_constants.h"
-
-namespace keys = extensions::manifest_keys;
-namespace errors = extensions::manifest_errors;
-
-namespace {
-
-// Stored on the Extension.
-struct MediaGalleriesHandlerInfo : public extensions::Extension::ManifestData {
- MediaGalleriesHandler::List media_galleries_handlers;
-
- MediaGalleriesHandlerInfo();
- virtual ~MediaGalleriesHandlerInfo();
-};
-
-MediaGalleriesHandlerInfo::MediaGalleriesHandlerInfo() {
-}
-
-MediaGalleriesHandlerInfo::~MediaGalleriesHandlerInfo() {
-}
-
-MediaGalleriesHandler* LoadMediaGalleriesHandler(
- const std::string& extension_id,
- const base::DictionaryValue* media_galleries_handler,
- base::string16* error) {
- scoped_ptr<MediaGalleriesHandler> result(new MediaGalleriesHandler());
- result->set_extension_id(extension_id);
-
- std::string handler_id;
- // Read the file action |id| (mandatory).
- if (!media_galleries_handler->HasKey(keys::kPageActionId) ||
- !media_galleries_handler->GetString(keys::kPageActionId, &handler_id)) {
- *error = base::ASCIIToUTF16(errors::kInvalidPageActionId);
- return NULL;
- }
- result->set_id(handler_id);
-
- // Read the page action title from |default_title| (mandatory).
- std::string title;
- if (!media_galleries_handler->HasKey(keys::kPageActionDefaultTitle) ||
- !media_galleries_handler->GetString(keys::kPageActionDefaultTitle,
- &title)) {
- *error = base::ASCIIToUTF16(errors::kInvalidPageActionDefaultTitle);
- return NULL;
- }
- result->set_title(title);
-
- std::string default_icon;
- // Read the media galleries action |default_icon| (optional).
- if (media_galleries_handler->HasKey(keys::kPageActionDefaultIcon)) {
- if (!media_galleries_handler->GetString(
- keys::kPageActionDefaultIcon, &default_icon) ||
- default_icon.empty()) {
- *error = base::ASCIIToUTF16(errors::kInvalidPageActionIconPath);
- return NULL;
- }
- result->set_icon_path(default_icon);
- }
-
- return result.release();
-}
-
-// Loads MediaGalleriesHandlers from |extension_actions| into a list in
-// |result|.
-bool LoadMediaGalleriesHandlers(
- const std::string& extension_id,
- const base::ListValue* extension_actions,
- MediaGalleriesHandler::List* result,
- base::string16* error) {
- for (base::ListValue::const_iterator iter = extension_actions->begin();
- iter != extension_actions->end();
- ++iter) {
- if (!(*iter)->IsType(base::Value::TYPE_DICTIONARY)) {
- *error = base::ASCIIToUTF16(errors::kInvalidMediaGalleriesHandler);
- return false;
- }
- scoped_ptr<MediaGalleriesHandler> action(
- LoadMediaGalleriesHandler(
- extension_id,
- reinterpret_cast<base::DictionaryValue*>(*iter), error));
- if (!action)
- return false; // Failed to parse media galleries action definition.
- result->push_back(linked_ptr<MediaGalleriesHandler>(action.release()));
- }
- return true;
-}
-
-} // namespace
-
-MediaGalleriesHandler::MediaGalleriesHandler() {
-}
-
-MediaGalleriesHandler::~MediaGalleriesHandler() {
-}
-
-// static
-MediaGalleriesHandler::List*
-MediaGalleriesHandler::GetHandlers(const extensions::Extension* extension) {
- MediaGalleriesHandlerInfo* info = static_cast<MediaGalleriesHandlerInfo*>(
- extension->GetManifestData(keys::kMediaGalleriesHandlers));
- if (info)
- return &info->media_galleries_handlers;
- return NULL;
-}
-
-MediaGalleriesHandlerParser::MediaGalleriesHandlerParser() {
-}
-
-MediaGalleriesHandlerParser::~MediaGalleriesHandlerParser() {
-}
-
-bool MediaGalleriesHandlerParser::Parse(extensions::Extension* extension,
- base::string16* error) {
- const base::ListValue* media_galleries_handlers_value = NULL;
- if (!extension->manifest()->GetList(keys::kMediaGalleriesHandlers,
- &media_galleries_handlers_value)) {
- *error = base::ASCIIToUTF16(errors::kInvalidMediaGalleriesHandler);
- return false;
- }
- scoped_ptr<MediaGalleriesHandlerInfo> info(new MediaGalleriesHandlerInfo);
- if (!LoadMediaGalleriesHandlers(extension->id(),
- media_galleries_handlers_value,
- &info->media_galleries_handlers,
- error)) {
- return false; // Failed to parse media galleries actions definition.
- }
-
- extension->SetManifestData(keys::kMediaGalleriesHandlers, info.release());
- return true;
-}
-
-const std::vector<std::string> MediaGalleriesHandlerParser::Keys() const {
- return SingleKey(keys::kMediaGalleriesHandlers);
-}

Powered by Google App Engine
This is Rietveld 408576698