Movable support for VideoReceiveStream::Config and avoid copies

webrtc/call.h

Index: webrtc/call.h
diff --git a/webrtc/call.h b/webrtc/call.h
index 80134fa27d4f13a0f6e2514c0eecaefa97a93c83..f89af93b5152e5d81e24b0b75f37b9510dae07fd 100644
--- a/webrtc/call.h
+++ b/webrtc/call.h
@@ -114,7 +114,7 @@ class Call {
   virtual void DestroyVideoSendStream(VideoSendStream* send_stream) = 0;
 
   virtual VideoReceiveStream* CreateVideoReceiveStream(
-      const VideoReceiveStream::Config& config) = 0;
+      VideoReceiveStream::Config configuration) = 0;

[pbos-webrtc, 2016/06/06 16:15:34]

Should this take a rvalue reference? I'm not sure how these things work, but
this looks like it requires a copy.

[tommi, 2016/06/08 09:49:39]

This accepts an rvalue or a copy.  I've modified the type so that creating a
copy will always need to be an explicit function call.  If the caller uses
std::move() or passes a value that's already rvalue (e.g. the return value from
a function), it will be passed as rvalue.

   virtual void DestroyVideoReceiveStream(
       VideoReceiveStream* receive_stream) = 0;