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

Unified Diff: services/video_capture/receiver_mojo_to_media_adapter.cc

Issue 2490153003: [Mojo Video Capture] Replace const scoped_refptr<T>& with scoped_refptr<T> and use std::move (Closed)
Patch Set: fix bots Created 4 years, 1 month 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: services/video_capture/receiver_mojo_to_media_adapter.cc
diff --git a/services/video_capture/receiver_mojo_to_media_adapter.cc b/services/video_capture/receiver_mojo_to_media_adapter.cc
index cc06ab53d4f733d5f9bb1c1a35b0581bbf058363..9200df0ecf09913bf37f652419ce54ac6ea9c5d2 100644
--- a/services/video_capture/receiver_mojo_to_media_adapter.cc
+++ b/services/video_capture/receiver_mojo_to_media_adapter.cc
@@ -17,10 +17,10 @@ ReceiverMojoToMediaAdapter::~ReceiverMojoToMediaAdapter() = default;
void ReceiverMojoToMediaAdapter::OnIncomingCapturedVideoFrame(
std::unique_ptr<media::VideoCaptureDevice::Client::Buffer> buffer,
- const scoped_refptr<media::VideoFrame>& frame) {
+ scoped_refptr<media::VideoFrame> frame) {
// O: |frame| should already be backed by a MojoSharedBufferVideoFrame
// assuming we have used the correct buffer factory with the pool.
- auto video_frame_ptr = media::mojom::VideoFrame::From(frame);
+ auto video_frame_ptr = media::mojom::VideoFrame::From(std::move(frame));
receiver_->OnIncomingCapturedVideoFrame(std::move(video_frame_ptr));
}

Powered by Google App Engine
This is Rietveld 408576698