Chromium Code Reviews| Index: content/renderer/media_capture_from_element/canvas_capture_handler.cc |
| diff --git a/content/renderer/media_capture_from_element/canvas_capture_handler.cc b/content/renderer/media_capture_from_element/canvas_capture_handler.cc |
| index b8b9d73736dbfa4f19739373b46ee2caa451b595..65935ec0381b7bc19d5409250ccd74c4e2428251 100644 |
| --- a/content/renderer/media_capture_from_element/canvas_capture_handler.cc |
| +++ b/content/renderer/media_capture_from_element/canvas_capture_handler.cc |
| @@ -141,7 +141,8 @@ CanvasCaptureHandler::~CanvasCaptureHandler() { |
| } |
| // static |
| -CanvasCaptureHandler* CanvasCaptureHandler::CreateCanvasCaptureHandler( |
| +std::unique_ptr<CanvasCaptureHandler> |
| +CanvasCaptureHandler::CreateCanvasCaptureHandler( |
| const blink::WebSize& size, |
| double frame_rate, |
| const scoped_refptr<base::SingleThreadTaskRunner>& io_task_runner, |
| @@ -150,7 +151,8 @@ CanvasCaptureHandler* CanvasCaptureHandler::CreateCanvasCaptureHandler( |
| // The histogram counts the number of calls to the JS API. |
| UpdateWebRTCMethodCount(WEBKIT_CANVAS_CAPTURE_STREAM); |
| - return new CanvasCaptureHandler(size, frame_rate, io_task_runner, track); |
| + return std::unique_ptr<CanvasCaptureHandler>( |
|
clamy
2017/04/27 12:00:18
Could we use base::MakeUnique?
altimin
2017/04/27 12:06:30
Same, constructor is private.
clamy
2017/04/27 12:12:48
Acknowledged.
|
| + new CanvasCaptureHandler(size, frame_rate, io_task_runner, track)); |
| } |
| void CanvasCaptureHandler::SendNewFrame(const SkImage* image) { |