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

Unified Diff: chrome/browser/extensions/api/autotest_private/autotest_private_api.cc

Issue 2806283002: Revert of Stop passing raw pointers to base::Value API in c/b/chromeos and c/b/extensions (Closed)
Patch Set: Created 3 years, 8 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/autotest_private/autotest_private_api.cc
diff --git a/chrome/browser/extensions/api/autotest_private/autotest_private_api.cc b/chrome/browser/extensions/api/autotest_private/autotest_private_api.cc
index e50590e4ca262775a3b410c6363cb665ce1e3afe..8005c8ead7fddb932e7c91f8f587ded746060f81 100644
--- a/chrome/browser/extensions/api/autotest_private/autotest_private_api.cc
+++ b/chrome/browser/extensions/api/autotest_private/autotest_private_api.cc
@@ -10,7 +10,6 @@
#include "base/lazy_instance.h"
#include "base/memory/ptr_util.h"
#include "base/strings/string_number_conversions.h"
-#include "base/values.h"
#include "build/build_config.h"
#include "chrome/browser/extensions/extension_action_manager.h"
#include "chrome/browser/extensions/extension_service.h"
@@ -41,14 +40,13 @@
namespace extensions {
namespace {
-std::unique_ptr<base::ListValue> GetHostPermissions(const Extension* ext,
- bool effective_perm) {
+base::ListValue* GetHostPermissions(const Extension* ext, bool effective_perm) {
const PermissionsData* permissions_data = ext->permissions_data();
const URLPatternSet& pattern_set =
effective_perm ? permissions_data->GetEffectiveHostPermissions()
: permissions_data->active_permissions().explicit_hosts();
- auto permissions = base::MakeUnique<base::ListValue>();
+ base::ListValue* permissions = new base::ListValue;
for (URLPatternSet::const_iterator perm = pattern_set.begin();
perm != pattern_set.end();
++perm) {
@@ -58,8 +56,8 @@
return permissions;
}
-std::unique_ptr<base::ListValue> GetAPIPermissions(const Extension* ext) {
- auto permissions = base::MakeUnique<base::ListValue>();
+base::ListValue* GetAPIPermissions(const Extension* ext) {
+ base::ListValue* permissions = new base::ListValue;
std::set<std::string> perm_list =
ext->permissions_data()->active_permissions().GetAPIsAsStrings();
for (std::set<std::string>::const_iterator perm = perm_list.begin();
@@ -168,7 +166,7 @@
ExtensionActionManager* extension_action_manager =
ExtensionActionManager::Get(browser_context());
- auto extensions_values = base::MakeUnique<base::ListValue>();
+ base::ListValue* extensions_values = new base::ListValue;
ExtensionList all;
all.insert(all.end(), extensions.begin(), extensions.end());
all.insert(all.end(), disabled_extensions.begin(), disabled_extensions.end());
@@ -214,7 +212,7 @@
std::unique_ptr<base::DictionaryValue> return_value(
new base::DictionaryValue);
- return_value->Set("extensions", std::move(extensions_values));
+ return_value->Set("extensions", extensions_values);
return RespondNow(OneArgument(std::move(return_value)));
}

Powered by Google App Engine
This is Rietveld 408576698