Interface with webrtc through encoded frames

remoting/protocol/webrtc_connection_to_client.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef REMOTING_PROTOCOL_WEBRTC_CONNECTION_TO_CLIENT_H_
 #define REMOTING_PROTOCOL_WEBRTC_CONNECTION_TO_CLIENT_H_
 
 #include <stdint.h>
 
 #include <string>
@@ skipping 12 matching lines @@
 class HostControlDispatcher;
 class HostEventDispatcher;
 
 class WebrtcConnectionToClient : public ConnectionToClient,
                                  public Session::EventHandler,
                                  public WebrtcTransport::EventHandler,
                                  public ChannelDispatcherBase::EventHandler {
  public:
   WebrtcConnectionToClient(
       scoped_ptr<Session> session,
-      scoped_refptr<protocol::TransportContext> transport_context);
+      scoped_refptr<protocol::TransportContext> transport_context,
+      scoped_refptr<base::SingleThreadTaskRunner> video_encode_task_runner);
   ~WebrtcConnectionToClient() override;
 
   // ConnectionToClient interface.
   void SetEventHandler(
       ConnectionToClient::EventHandler* event_handler) override;
   Session* session() override;
   void Disconnect(ErrorCode error) override;
   void OnInputEventReceived(int64_t timestamp) override;
   scoped_ptr<VideoStream> StartVideoStream(
       scoped_ptr<webrtc::DesktopCapturer> desktop_capturer) override;
@@ skipping 17 matching lines @@
 
   // ChannelDispatcherBase::EventHandler interface.
   void OnChannelInitialized(ChannelDispatcherBase* channel_dispatcher) override;
 
  private:
   base::ThreadChecker thread_checker_;
 
   // Event handler for handling events sent from this object.
   ConnectionToClient::EventHandler* event_handler_ = nullptr;
 
-  WebrtcTransport transport_;
+  scoped_ptr<WebrtcTransport> transport_;

[Sergey Ulanov, 2016/04/07 21:05:39]

What's the reason you need this change? transport_ has the same lifetime as
WebrtcConnectionToClient, so it can be a member.
If there is something else I'm missing then please change this to
std::unique_ptr<>

 
   scoped_ptr<Session> session_;
 
+  scoped_refptr<base::SingleThreadTaskRunner> video_encode_task_runner_;
+
   scoped_ptr<HostControlDispatcher> control_dispatcher_;
   scoped_ptr<HostEventDispatcher> event_dispatcher_;
 
   DISALLOW_COPY_AND_ASSIGN(WebrtcConnectionToClient);
 };
 
 }  // namespace protocol
 }  // namespace remoting
 
 #endif  // REMOTING_PROTOCOL_WEBRTC_CONNECTION_TO_CLIENT_H_