[Chromoting] Implement DualBufferFrameConsumer

remoting/client/software_video_renderer.cc

Index: remoting/client/software_video_renderer.cc
diff --git a/remoting/client/software_video_renderer.cc b/remoting/client/software_video_renderer.cc
index 930087dcf9f985ec84d4a7f1a4da3110f5452ab3..134dc85113b796bb1c55cbfad28792185176d5e1 100644
--- a/remoting/client/software_video_renderer.cc
+++ b/remoting/client/software_video_renderer.cc
@@ -51,6 +51,14 @@ SoftwareVideoRenderer::SoftwareVideoRenderer(protocol::FrameConsumer* consumer)
 }
 
 SoftwareVideoRenderer::SoftwareVideoRenderer(
+    std::unique_ptr<protocol::FrameConsumer> consumer)
+    : owned_consumer_(std::move(consumer)),

[Sergey Ulanov, 2016/07/20 18:39:26]

nit: I think it's better to call the other constructor here and then initialize
owner_consumer_ in the body of the constructor. That would allow to avoid
duplication of the initializer list and the DetachFromThread() call.

[Yuwei, 2016/07/21 00:07:32]

Done.

+      consumer_(owned_consumer_.get()),
+      weak_factory_(this) {
+  thread_checker_.DetachFromThread();
+}
+
+SoftwareVideoRenderer::SoftwareVideoRenderer(
     scoped_refptr<base::SingleThreadTaskRunner> decode_task_runner,
     protocol::FrameConsumer* consumer,
     protocol::FrameStatsConsumer* stats_consumer)