diff options
author | vmpstr <vmpstr@chromium.org> | 2016-03-16 13:44:02 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-03-16 20:46:15 +0000 |
commit | 4065174ac28746b10a832411426bd756d9c5979c (patch) | |
tree | 9ea7bf51f94a7b35b1921187fc8b93c48df9077e /cc/test | |
parent | e886edbb2d3d7eade99d9542b56dfc9a7701248f (diff) | |
download | chromium_src-4065174ac28746b10a832411426bd756d9c5979c.zip chromium_src-4065174ac28746b10a832411426bd756d9c5979c.tar.gz chromium_src-4065174ac28746b10a832411426bd756d9c5979c.tar.bz2 |
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}
Diffstat (limited to 'cc/test')
-rw-r--r-- | cc/test/fake_video_frame_provider.h | 4 |
1 files changed, 2 insertions, 2 deletions
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<media::VideoFrame>& frame) { - frame_ = frame; + void set_frame(scoped_refptr<media::VideoFrame> frame) { + frame_ = std::move(frame); } int put_current_frame_count() const { return put_current_frame_count_; } |