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

Unified Diff: content/browser/renderer_host/media/video_capture_controller.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: content/browser/renderer_host/media/video_capture_controller.cc
diff --git a/content/browser/renderer_host/media/video_capture_controller.cc b/content/browser/renderer_host/media/video_capture_controller.cc
index f9c91568fcf6eec2202d3d09e964f68c09409ade..bf0d9d9c0a955761f59085374597e3bf0c035f9e 100644
--- a/content/browser/renderer_host/media/video_capture_controller.cc
+++ b/content/browser/renderer_host/media/video_capture_controller.cc
@@ -95,11 +95,11 @@ class VideoFrameReceiverOnIOThread : public media::VideoFrameReceiver {
void OnIncomingCapturedVideoFrame(
std::unique_ptr<media::VideoCaptureDevice::Client::Buffer> buffer,
- const scoped_refptr<media::VideoFrame>& frame) override {
+ scoped_refptr<media::VideoFrame> frame) override {
BrowserThread::PostTask(
BrowserThread::IO, FROM_HERE,
base::Bind(&VideoFrameReceiver::OnIncomingCapturedVideoFrame, receiver_,
- base::Passed(&buffer), frame));
+ base::Passed(&buffer), std::move(frame)));
}
void OnError() override {
@@ -402,7 +402,7 @@ VideoCaptureController::~VideoCaptureController() {
void VideoCaptureController::OnIncomingCapturedVideoFrame(
std::unique_ptr<media::VideoCaptureDevice::Client::Buffer> buffer,
- const scoped_refptr<VideoFrame>& frame) {
+ scoped_refptr<VideoFrame> frame) {
DCHECK_CURRENTLY_ON(BrowserThread::IO);
const int buffer_id = buffer->id();
DCHECK_NE(buffer_id, media::VideoCaptureBufferPool::kInvalidId);

Powered by Google App Engine
This is Rietveld 408576698