From 9156f6dcce664f49504c1a68e0a6760a0a0ccd95 Mon Sep 17 00:00:00 2001 From: Sylvain Defresne Date: Tue, 14 May 2019 18:16:20 +0000 Subject: [PATCH] Remove unnecessary base::Passed() with base::BindOnce() The documentation on base::Callback<...> (src/docs/callback.md) recommends against using base::Passed(...) with base::BindOnce. > Avoid using `base::Passed()` with `base::BindOnce()`, as `std::move()` > does the same thing and is more familiar. This CL was uploaded by git cl split. Bug: none Change-Id: I3d67bad8fd17130ffa750475a65a8cb25afdaa68 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1611884 Commit-Queue: Tommi Reviewed-by: Tommi Auto-Submit: Sylvain Defresne Cr-Commit-Position: refs/heads/master@{#659582} --- media/capture/video/video_frame_receiver_on_task_runner.cc | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/media/capture/video/video_frame_receiver_on_task_runner.cc b/media/capture/video/video_frame_receiver_on_task_runner.cc index 7f6bb4be2f5274..fc761bcb3a0bcb 100644 --- a/media/capture/video/video_frame_receiver_on_task_runner.cc +++ b/media/capture/video/video_frame_receiver_on_task_runner.cc @@ -20,9 +20,8 @@ void VideoFrameReceiverOnTaskRunner::OnNewBuffer( int buffer_id, media::mojom::VideoBufferHandlePtr buffer_handle) { task_runner_->PostTask( - FROM_HERE, - base::BindOnce(&VideoFrameReceiver::OnNewBuffer, receiver_, buffer_id, - base::Passed(std::move(buffer_handle)))); + FROM_HERE, base::BindOnce(&VideoFrameReceiver::OnNewBuffer, receiver_, + buffer_id, std::move(buffer_handle))); } void VideoFrameReceiverOnTaskRunner::OnFrameReadyInBuffer(