From 4065174ac28746b10a832411426bd756d9c5979c Mon Sep 17 00:00:00 2001 From: vmpstr Date: Wed, 16 Mar 2016 13:44:02 -0700 Subject: cc: Remove some unnecessary const scoped_refptr&. This patch makes the API a bit clearer by taking a copy of scoped_refptr in some situations (where ownership is shared) and using a raw pointer in situations where ownership is not needed. The only references that remain are the ones captured in lambdas or Callbacks. R=danakj BUG=589044 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1803863003 Cr-Commit-Position: refs/heads/master@{#381524} --- cc/test/fake_video_frame_provider.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'cc/test') diff --git a/cc/test/fake_video_frame_provider.h b/cc/test/fake_video_frame_provider.h index 4cbd80a..0b77f5d 100644 --- a/cc/test/fake_video_frame_provider.h +++ b/cc/test/fake_video_frame_provider.h @@ -25,8 +25,8 @@ class FakeVideoFrameProvider : public VideoFrameProvider { Client* client() { return client_; } - void set_frame(const scoped_refptr& frame) { - frame_ = frame; + void set_frame(scoped_refptr frame) { + frame_ = std::move(frame); } int put_current_frame_count() const { return put_current_frame_count_; } -- cgit v1.1