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

Unified Diff: chrome/renderer/pepper/chrome_renderer_pepper_host_factory.cc

Issue 2341693002: Re-write many calls to WrapUnique() with MakeUnique() (Closed)
Patch Set: Change std::string() back to "" to fix compile Created 4 years, 3 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/renderer/pepper/chrome_renderer_pepper_host_factory.cc
diff --git a/chrome/renderer/pepper/chrome_renderer_pepper_host_factory.cc b/chrome/renderer/pepper/chrome_renderer_pepper_host_factory.cc
index c1e2d1a09751ff3f01cc44171a4156560367e57e..12d432198186daf817c0b903af985b05ce17cda2 100644
--- a/chrome/renderer/pepper/chrome_renderer_pepper_host_factory.cc
+++ b/chrome/renderer/pepper/chrome_renderer_pepper_host_factory.cc
@@ -44,19 +44,19 @@ ChromeRendererPepperHostFactory::CreateResourceHost(
ppapi::PERMISSION_FLASH)) {
switch (message.type()) {
case PpapiHostMsg_Flash_Create::ID: {
- return base::WrapUnique(
- new PepperFlashRendererHost(host_, instance, resource));
+ return base::MakeUnique<PepperFlashRendererHost>(host_, instance,
+ resource);
}
case PpapiHostMsg_FlashFullscreen_Create::ID: {
- return base::WrapUnique(
- new PepperFlashFullscreenHost(host_, instance, resource));
+ return base::MakeUnique<PepperFlashFullscreenHost>(host_, instance,
+ resource);
}
case PpapiHostMsg_FlashMenu_Create::ID: {
ppapi::proxy::SerializedFlashMenu serialized_menu;
if (ppapi::UnpackMessage<PpapiHostMsg_FlashMenu_Create>(
message, &serialized_menu)) {
- return base::WrapUnique(new PepperFlashMenuHost(
- host_, instance, resource, serialized_menu));
+ return base::MakeUnique<PepperFlashMenuHost>(
+ host_, instance, resource, serialized_menu);
}
break;
}
@@ -76,14 +76,14 @@ ChromeRendererPepperHostFactory::CreateResourceHost(
PP_PrivateFontCharset charset;
if (ppapi::UnpackMessage<PpapiHostMsg_FlashFontFile_Create>(
message, &description, &charset)) {
- return base::WrapUnique(new PepperFlashFontFileHost(
- host_, instance, resource, description, charset));
+ return base::MakeUnique<PepperFlashFontFileHost>(
+ host_, instance, resource, description, charset);
}
break;
}
case PpapiHostMsg_FlashDRM_Create::ID:
- return base::WrapUnique(
- new PepperFlashDRMRendererHost(host_, instance, resource));
+ return base::MakeUnique<PepperFlashDRMRendererHost>(host_, instance,
+ resource);
}
}
@@ -91,8 +91,7 @@ ChromeRendererPepperHostFactory::CreateResourceHost(
ppapi::PERMISSION_PRIVATE)) {
switch (message.type()) {
case PpapiHostMsg_PDF_Create::ID: {
- return base::WrapUnique(
- new pdf::PepperPDFHost(host_, instance, resource));
+ return base::MakeUnique<pdf::PepperPDFHost>(host_, instance, resource);
}
}
}
@@ -103,7 +102,7 @@ ChromeRendererPepperHostFactory::CreateResourceHost(
// access to the other private interfaces.
switch (message.type()) {
case PpapiHostMsg_UMA_Create::ID: {
- return base::WrapUnique(new PepperUMAHost(host_, instance, resource));
+ return base::MakeUnique<PepperUMAHost>(host_, instance, resource);
}
}
« no previous file with comments | « chrome/renderer/extensions/extension_localization_peer_unittest.cc ('k') | chrome/renderer/pepper/pepper_helper.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698