| Index: chrome/browser/extensions/api/media_galleries/media_galleries_api.cc
|
| diff --git a/chrome/browser/extensions/api/media_galleries/media_galleries_api.cc b/chrome/browser/extensions/api/media_galleries/media_galleries_api.cc
|
| index 34f14cbe64f27e60d157d60a0dfe64ff7d47d75e..3b544150235ce282fec44092ef14883d40033b8b 100644
|
| --- a/chrome/browser/extensions/api/media_galleries/media_galleries_api.cc
|
| +++ b/chrome/browser/extensions/api/media_galleries/media_galleries_api.cc
|
| @@ -7,6 +7,7 @@
|
| #include "chrome/browser/extensions/api/media_galleries/media_galleries_api.h"
|
|
|
| #include <stddef.h>
|
| +
|
| #include <set>
|
| #include <string>
|
| #include <utility>
|
| @@ -15,6 +16,7 @@
|
| #include "base/callback.h"
|
| #include "base/lazy_instance.h"
|
| #include "base/macros.h"
|
| +#include "base/memory/ptr_util.h"
|
| #include "base/numerics/safe_conversions.h"
|
| #include "base/stl_util.h"
|
| #include "base/strings/string_number_conversions.h"
|
| @@ -157,9 +159,9 @@ base::ListValue* ConstructFileSystemList(
|
| APIPermission::kMediaGalleries, &delete_param);
|
|
|
| const int child_id = rfh->GetProcess()->GetID();
|
| - scoped_ptr<base::ListValue> list(new base::ListValue());
|
| + std::unique_ptr<base::ListValue> list(new base::ListValue());
|
| for (size_t i = 0; i < filesystems.size(); ++i) {
|
| - scoped_ptr<base::DictionaryValue> file_system_dict_value(
|
| + std::unique_ptr<base::DictionaryValue> file_system_dict_value(
|
| new base::DictionaryValue());
|
|
|
| // Send the file system id so the renderer can create a valid FileSystem
|
| @@ -314,14 +316,14 @@ void MediaGalleriesEventRouter::DispatchEventToExtension(
|
| const std::string& extension_id,
|
| events::HistogramValue histogram_value,
|
| const std::string& event_name,
|
| - scoped_ptr<base::ListValue> event_args) {
|
| + std::unique_ptr<base::ListValue> event_args) {
|
| DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
|
|
|
| EventRouter* router = EventRouter::Get(profile_);
|
| if (!router->ExtensionHasEventListener(extension_id, event_name))
|
| return;
|
|
|
| - scoped_ptr<extensions::Event> event(new extensions::Event(
|
| + std::unique_ptr<extensions::Event> event(new extensions::Event(
|
| histogram_value, event_name, std::move(event_args)));
|
| router->DispatchEventToExtension(extension_id, std::move(event));
|
| }
|
| @@ -364,7 +366,7 @@ MediaGalleriesGetMediaFileSystemsFunction::
|
|
|
| bool MediaGalleriesGetMediaFileSystemsFunction::RunAsync() {
|
| media_galleries::UsageCount(media_galleries::GET_MEDIA_FILE_SYSTEMS);
|
| - scoped_ptr<GetMediaFileSystems::Params> params(
|
| + std::unique_ptr<GetMediaFileSystems::Params> params(
|
| GetMediaFileSystems::Params::Create(*args_));
|
| EXTENSION_FUNCTION_VALIDATE(params.get());
|
| MediaGalleries::GetMediaFileSystemsInteractivity interactive =
|
| @@ -427,7 +429,7 @@ void MediaGalleriesGetMediaFileSystemsFunction::GetAndReturnGalleries() {
|
|
|
| void MediaGalleriesGetMediaFileSystemsFunction::ReturnGalleries(
|
| const std::vector<MediaFileSystemInfo>& filesystems) {
|
| - scoped_ptr<base::ListValue> list(
|
| + std::unique_ptr<base::ListValue> list(
|
| ConstructFileSystemList(render_frame_host(), extension(), filesystems));
|
| if (!list.get()) {
|
| SendResponse(false);
|
| @@ -535,7 +537,7 @@ void MediaGalleriesAddUserSelectedFolderFunction::OnDirectorySelected(
|
| void MediaGalleriesAddUserSelectedFolderFunction::ReturnGalleriesAndId(
|
| MediaGalleryPrefId pref_id,
|
| const std::vector<MediaFileSystemInfo>& filesystems) {
|
| - scoped_ptr<base::ListValue> list(
|
| + std::unique_ptr<base::ListValue> list(
|
| ConstructFileSystemList(render_frame_host(), extension(), filesystems));
|
| if (!list.get()) {
|
| SendResponse(false);
|
| @@ -584,7 +586,7 @@ bool MediaGalleriesGetMetadataFunction::RunAsync() {
|
| const base::Value* options_value = NULL;
|
| if (!args_->Get(1, &options_value))
|
| return false;
|
| - scoped_ptr<MediaGalleries::MediaMetadataOptions> options =
|
| + std::unique_ptr<MediaGalleries::MediaMetadataOptions> options =
|
| MediaGalleries::MediaMetadataOptions::FromValue(*options_value);
|
| if (!options)
|
| return false;
|
| @@ -612,7 +614,7 @@ void MediaGalleriesGetMetadataFunction::OnPreferencesInit(
|
| void MediaGalleriesGetMetadataFunction::GetMetadata(
|
| MediaGalleries::GetMetadataType metadata_type,
|
| const std::string& blob_uuid,
|
| - scoped_ptr<std::string> blob_header,
|
| + std::unique_ptr<std::string> blob_header,
|
| int64_t total_blob_length) {
|
| DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
|
|
|
| @@ -651,8 +653,9 @@ void MediaGalleriesGetMetadataFunction::GetMetadata(
|
| }
|
|
|
| void MediaGalleriesGetMetadataFunction::OnSafeMediaMetadataParserDone(
|
| - bool parse_success, scoped_ptr<base::DictionaryValue> metadata_dictionary,
|
| - scoped_ptr<std::vector<metadata::AttachedImage> > attached_images) {
|
| + bool parse_success,
|
| + std::unique_ptr<base::DictionaryValue> metadata_dictionary,
|
| + std::unique_ptr<std::vector<metadata::AttachedImage>> attached_images) {
|
| DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
|
|
|
| if (!parse_success) {
|
| @@ -663,7 +666,7 @@ void MediaGalleriesGetMetadataFunction::OnSafeMediaMetadataParserDone(
|
| DCHECK(metadata_dictionary.get());
|
| DCHECK(attached_images.get());
|
|
|
| - scoped_ptr<base::DictionaryValue> result_dictionary(
|
| + std::unique_ptr<base::DictionaryValue> result_dictionary(
|
| new base::DictionaryValue);
|
| result_dictionary->Set(kMetadataKey, metadata_dictionary.release());
|
|
|
| @@ -676,20 +679,18 @@ void MediaGalleriesGetMetadataFunction::OnSafeMediaMetadataParserDone(
|
| result_dictionary->Set(kAttachedImagesBlobInfoKey, new base::ListValue);
|
| metadata::AttachedImage* first_image = &attached_images->front();
|
| content::BrowserContext::CreateMemoryBackedBlob(
|
| - GetProfile(),
|
| - first_image->data.c_str(),
|
| - first_image->data.size(),
|
| - base::Bind(&MediaGalleriesGetMetadataFunction::ConstructNextBlob,
|
| - this, base::Passed(&result_dictionary),
|
| + GetProfile(), first_image->data.c_str(), first_image->data.size(),
|
| + base::Bind(&MediaGalleriesGetMetadataFunction::ConstructNextBlob, this,
|
| + base::Passed(&result_dictionary),
|
| base::Passed(&attached_images),
|
| - base::Passed(make_scoped_ptr(new std::vector<std::string>))));
|
| + base::Passed(base::WrapUnique(new std::vector<std::string>))));
|
| }
|
|
|
| void MediaGalleriesGetMetadataFunction::ConstructNextBlob(
|
| - scoped_ptr<base::DictionaryValue> result_dictionary,
|
| - scoped_ptr<std::vector<metadata::AttachedImage> > attached_images,
|
| - scoped_ptr<std::vector<std::string> > blob_uuids,
|
| - scoped_ptr<content::BlobHandle> current_blob) {
|
| + std::unique_ptr<base::DictionaryValue> result_dictionary,
|
| + std::unique_ptr<std::vector<metadata::AttachedImage>> attached_images,
|
| + std::unique_ptr<std::vector<std::string>> blob_uuids,
|
| + std::unique_ptr<content::BlobHandle> current_blob) {
|
| DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
|
|
|
| DCHECK(result_dictionary.get());
|
| @@ -756,7 +757,7 @@ bool MediaGalleriesAddGalleryWatchFunction::RunAsync() {
|
| if (!render_frame_host() || !render_frame_host()->GetProcess())
|
| return false;
|
|
|
| - scoped_ptr<AddGalleryWatch::Params> params(
|
| + std::unique_ptr<AddGalleryWatch::Params> params(
|
| AddGalleryWatch::Params::Create(*args_));
|
| EXTENSION_FUNCTION_VALIDATE(params.get());
|
|
|
| @@ -840,7 +841,7 @@ bool MediaGalleriesRemoveGalleryWatchFunction::RunAsync() {
|
| if (!render_frame_host() || !render_frame_host()->GetProcess())
|
| return false;
|
|
|
| - scoped_ptr<RemoveGalleryWatch::Params> params(
|
| + std::unique_ptr<RemoveGalleryWatch::Params> params(
|
| RemoveGalleryWatch::Params::Create(*args_));
|
| EXTENSION_FUNCTION_VALIDATE(params.get());
|
|
|
|
|