Skip to content

Commit

Permalink
Remove unnecessary base::Passed() with base::BindOnce()
Browse files Browse the repository at this point in the history
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 <tommi@chromium.org>
Reviewed-by: Tommi <tommi@chromium.org>
Auto-Submit: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#659582}
  • Loading branch information
sdefresne authored and Commit Bot committed May 14, 2019
1 parent 54ec7ed commit 9156f6d
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions media/capture/video/video_frame_receiver_on_task_runner.cc
Original file line number Diff line number Diff line change
Expand Up @@ -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(
Expand Down

0 comments on commit 9156f6d

Please sign in to comment.