[blink] Unique pointers in Platform.h

content/renderer/media/renderer_webaudiodevice_impl.cc

 // Copyright (c) 2012 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 "content/renderer/media/renderer_webaudiodevice_impl.h"
 
 #include <stddef.h>
 
 #include <string>
 
@@ skipping 86 matching lines @@
     int session_id,
     const std::string& device_id,
     const url::Origin& security_origin) {
   return AudioDeviceFactory::GetOutputDeviceInfo(frame_id, session_id,
                                                  device_id, security_origin)
       .output_params();
 }
 
 }  // namespace
 
-RendererWebAudioDeviceImpl* RendererWebAudioDeviceImpl::Create(
+std::unique_ptr<RendererWebAudioDeviceImpl> RendererWebAudioDeviceImpl::Create(
     media::ChannelLayout layout,
     int channels,
     const blink::WebAudioLatencyHint& latency_hint,
     WebAudioDevice::RenderCallback* callback,
     int session_id,
     const url::Origin& security_origin) {
-  return new RendererWebAudioDeviceImpl(layout, channels, latency_hint,
-                                        callback, session_id, security_origin,
-                                        base::Bind(&GetOutputDeviceParameters),
-                                        base::Bind(&FrameIdFromCurrentContext));
+  return std::unique_ptr<RendererWebAudioDeviceImpl>(

[clamy, 2017/04/27 12:00:18]

Could we use base::MakeUnique?

[altimin, 2017/04/27 12:06:30]

RendererWebAudioDeviceImpl's constructor is private. Not sure if we want to
change this.

[clamy, 2017/04/27 12:12:48]

Acknowledged.

+      new RendererWebAudioDeviceImpl(layout, channels, latency_hint, callback,
+                                     session_id, security_origin,
+                                     base::Bind(&GetOutputDeviceParameters),
+                                     base::Bind(&FrameIdFromCurrentContext)));
 }
 
 RendererWebAudioDeviceImpl::RendererWebAudioDeviceImpl(
     media::ChannelLayout layout,
     int channels,
     const blink::WebAudioLatencyHint& latency_hint,
     WebAudioDevice::RenderCallback* callback,
     int session_id,
     const url::Origin& security_origin,
     const OutputDeviceParamsCallback& device_params_cb,
@@ skipping 112 matching lines @@
 const scoped_refptr<base::SingleThreadTaskRunner>&
 RendererWebAudioDeviceImpl::GetMediaTaskRunner() {
   if (!media_task_runner_) {
     media_task_runner_ =
         RenderThreadImpl::current()->GetMediaThreadTaskRunner();
   }
   return media_task_runner_;
 }
 
 }  // namespace content