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

Unified Diff: content/renderer/pepper/content_renderer_pepper_host_factory.cc

Issue 2253053003: Re-write many calls to WrapUnique() with MakeUnique() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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: content/renderer/pepper/content_renderer_pepper_host_factory.cc
diff --git a/content/renderer/pepper/content_renderer_pepper_host_factory.cc b/content/renderer/pepper/content_renderer_pepper_host_factory.cc
index 6614c9c923a2c99ffc537d7d59201766e4d4f1e5..08ae05c5bc1811825246572194ab0320e88cb9ac 100644
--- a/content/renderer/pepper/content_renderer_pepper_host_factory.cc
+++ b/content/renderer/pepper/content_renderer_pepper_host_factory.cc
@@ -123,8 +123,7 @@ ContentRendererPepperHostFactory::CreateResourceHost(
case PpapiHostMsg_Compositor_Create::ID: {
if (!CanUseCompositorAPI(host_, instance))
return nullptr;
- return base::WrapUnique(
- new PepperCompositorHost(host_, instance, resource));
+ return base::MakeUnique<PepperCompositorHost>(host_, instance, resource);
}
case PpapiHostMsg_FileRef_CreateForFileAPI::ID: {
PP_Resource file_system;
@@ -134,8 +133,8 @@ ContentRendererPepperHostFactory::CreateResourceHost(
NOTREACHED();
return nullptr;
}
- return base::WrapUnique(new PepperFileRefRendererHost(
- host_, instance, resource, file_system, internal_path));
+ return base::MakeUnique<PepperFileRefRendererHost>(
+ host_, instance, resource, file_system, internal_path);
}
case PpapiHostMsg_FileSystem_Create::ID: {
PP_FileSystemType file_system_type;
@@ -144,8 +143,8 @@ ContentRendererPepperHostFactory::CreateResourceHost(
NOTREACHED();
return nullptr;
}
- return base::WrapUnique(new PepperFileSystemHost(
- host_, instance, resource, file_system_type));
+ return base::MakeUnique<PepperFileSystemHost>(host_, instance, resource,
+ file_system_type);
}
case PpapiHostMsg_Graphics2D_Create::ID: {
PP_Size size;
@@ -174,32 +173,31 @@ ContentRendererPepperHostFactory::CreateResourceHost(
host_, instance, resource, size, is_always_opaque, image_data));
}
case PpapiHostMsg_URLLoader_Create::ID:
- return base::WrapUnique(
- new PepperURLLoaderHost(host_, false, instance, resource));
+ return base::MakeUnique<PepperURLLoaderHost>(host_, false, instance,
+ resource);
case PpapiHostMsg_VideoDecoder_Create::ID:
- return base::WrapUnique(
- new PepperVideoDecoderHost(host_, instance, resource));
+ return base::MakeUnique<PepperVideoDecoderHost>(host_, instance,
+ resource);
case PpapiHostMsg_VideoEncoder_Create::ID:
- return base::WrapUnique(
- new PepperVideoEncoderHost(host_, instance, resource));
+ return base::MakeUnique<PepperVideoEncoderHost>(host_, instance,
+ resource);
case PpapiHostMsg_WebSocket_Create::ID:
- return base::WrapUnique(
- new PepperWebSocketHost(host_, instance, resource));
+ return base::MakeUnique<PepperWebSocketHost>(host_, instance, resource);
#if defined(ENABLE_WEBRTC)
case PpapiHostMsg_MediaStreamVideoTrack_Create::ID:
- return base::WrapUnique(
- new PepperMediaStreamVideoTrackHost(host_, instance, resource));
+ return base::MakeUnique<PepperMediaStreamVideoTrackHost>(host_, instance,
+ resource);
// These private MediaStream interfaces are exposed as if they were public
// so they can be used by NaCl plugins. However, they are available only
// for whitelisted apps.
case PpapiHostMsg_VideoDestination_Create::ID:
if (CanUseMediaStreamAPI(host_, instance))
- return base::WrapUnique(
- new PepperVideoDestinationHost(host_, instance, resource));
+ return base::MakeUnique<PepperVideoDestinationHost>(host_, instance,
+ resource);
case PpapiHostMsg_VideoSource_Create::ID:
if (CanUseMediaStreamAPI(host_, instance))
- return base::WrapUnique(
- new PepperVideoSourceHost(host_, instance, resource));
+ return base::MakeUnique<PepperVideoSourceHost>(host_, instance,
+ resource);
#endif // defined(ENABLE_WEBRTC)
}
@@ -207,14 +205,14 @@ ContentRendererPepperHostFactory::CreateResourceHost(
if (GetPermissions().HasPermission(ppapi::PERMISSION_DEV)) {
switch (message.type()) {
case PpapiHostMsg_AudioEncoder_Create::ID:
- return base::WrapUnique(
- new PepperAudioEncoderHost(host_, instance, resource));
+ return base::MakeUnique<PepperAudioEncoderHost>(host_, instance,
+ resource);
case PpapiHostMsg_AudioInput_Create::ID:
- return base::WrapUnique(
- new PepperAudioInputHost(host_, instance, resource));
+ return base::MakeUnique<PepperAudioInputHost>(host_, instance,
+ resource);
case PpapiHostMsg_FileChooser_Create::ID:
- return base::WrapUnique(
- new PepperFileChooserHost(host_, instance, resource));
+ return base::MakeUnique<PepperFileChooserHost>(host_, instance,
+ resource);
case PpapiHostMsg_VideoCapture_Create::ID: {
std::unique_ptr<PepperVideoCaptureHost> host(
new PepperVideoCaptureHost(host_, instance, resource));
« no previous file with comments | « content/renderer/media/rtc_certificate_generator.cc ('k') | content/renderer/presentation/presentation_dispatcher.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698