Add support for dynamic channels in WebrtcTransport.

remoting/protocol/webrtc_data_stream_adapter.cc

 // 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.
 
 #include "remoting/protocol/webrtc_data_stream_adapter.h"
 
 #include <stdint.h>
 
 #include "base/bind.h"
 #include "base/callback.h"
@@ skipping 14 matching lines @@
                                          public webrtc::DataChannelObserver {
  public:
   explicit Channel(base::WeakPtr<WebrtcDataStreamAdapter> adapter);
   ~Channel() override;
 
   void Start(rtc::scoped_refptr<webrtc::DataChannelInterface> channel);
 
   std::string name() { return channel_->label(); }
 
   // MessagePipe interface.
-  void StartReceiving(const MessageReceivedCallback& callback) override;
+  void Start(EventHandler* event_handler) override;
   void Send(google::protobuf::MessageLite* message,
             const base::Closure& done) override;
 
  private:
   enum class State { CONNECTING, OPEN, CLOSED };
 
   // webrtc::DataChannelObserver interface.
   void OnStateChange() override;
   void OnMessage(const webrtc::DataBuffer& buffer) override;
 
   void OnConnected();
 
-  void OnError();
+  void OnClosed();
+  void NotifyClosed();
 
   base::WeakPtr<WebrtcDataStreamAdapter> adapter_;
 
   rtc::scoped_refptr<webrtc::DataChannelInterface> channel_;
 
-  MessageReceivedCallback message_received_callback_;
+  EventHandler* event_handler_ = nullptr;
 
   State state_ = State::CONNECTING;
 
+  base::WeakPtrFactory<Channel> weak_factory_;
+
   DISALLOW_COPY_AND_ASSIGN(Channel);
 };
 
 WebrtcDataStreamAdapter::Channel::Channel(
     base::WeakPtr<WebrtcDataStreamAdapter> adapter)
-    : adapter_(adapter) {}
+    : adapter_(adapter), weak_factory_(this) {}
 
 WebrtcDataStreamAdapter::Channel::~Channel() {
   if (channel_) {
     channel_->UnregisterObserver();
     channel_->Close();
   }
 }
 
 void WebrtcDataStreamAdapter::Channel::Start(
     rtc::scoped_refptr<webrtc::DataChannelInterface> channel) {
   DCHECK(!channel_);
 
   channel_ = channel;
   channel_->RegisterObserver(this);
 
    if (channel_->state() == webrtc::DataChannelInterface::kOpen) {
     OnConnected();
   } else {
     DCHECK_EQ(channel_->state(), webrtc::DataChannelInterface::kConnecting);
   }
 }
 
-void WebrtcDataStreamAdapter::Channel::StartReceiving(
-    const MessageReceivedCallback& callback) {
-  DCHECK(message_received_callback_.is_null());
-  DCHECK(!callback.is_null());
+void WebrtcDataStreamAdapter::Channel::Start(EventHandler* event_handler) {
+  DCHECK(!event_handler_);
+  DCHECK(event_handler);
 
-  message_received_callback_ = callback;
+  event_handler_ = event_handler;
 }
 
 void WebrtcDataStreamAdapter::Channel::Send(
     google::protobuf::MessageLite* message,
     const base::Closure& done) {
+  if(state_ != State::OPEN)
+    return;

[Jamie, 2016/07/19 18:24:47]

Should we log an error here? Silently dropping the packet doesn't seem right.

[Sergey Ulanov, 2016/07/19 23:38:44]

Replaced this with a DCHECK().
Also changed OnMessagePipeClosed() notification to synchronous. This means that
EventHandler is notified as soon as state changes and so it knows it cannot call
Send() anymore.

+
   rtc::CopyOnWriteBuffer buffer;
   buffer.SetSize(message->ByteSize());
   message->SerializeWithCachedSizesToArray(
       reinterpret_cast<uint8_t*>(buffer.data()));
   webrtc::DataBuffer data_buffer(std::move(buffer), true /* binary */);
   if (!channel_->Send(data_buffer)) {
-    OnError();
+    LOG(ERROR) << "Send failed on data channel " << channel_->label();
+    channel_->Close();
     return;
   }
 
   if (!done.is_null())
     done.Run();
 }
 
 void WebrtcDataStreamAdapter::Channel::OnStateChange() {
   switch (channel_->state()) {
     case webrtc::DataChannelInterface::kOpen:
       OnConnected();
       break;
 
     case webrtc::DataChannelInterface::kClosing:
       // Currently channels are not expected to be closed.

[Jamie, 2016/07/19 18:24:47]

Log an error in this case?

[Sergey Ulanov, 2016/07/19 23:38:43]

Removed the comment. This class now does support closing channels dynamically.

-      OnError();
+      OnClosed();
       break;
 
     case webrtc::DataChannelInterface::kConnecting:
     case webrtc::DataChannelInterface::kClosed:
       break;
   }
 }
 
-void WebrtcDataStreamAdapter::Channel::OnConnected() {
-  CHECK(state_ == State::CONNECTING);
-  state_ = State::OPEN;
-  adapter_->OnChannelConnected(this);
-}
-
-void WebrtcDataStreamAdapter::Channel::OnError() {
-  if (state_ == State::CLOSED)
-    return;
-
-  state_ = State::CLOSED;
-
-  // Notify the adapter about the error asychronously.
-  base::ThreadTaskRunnerHandle::Get()->PostTask(
-      FROM_HERE,
-      base::Bind(&WebrtcDataStreamAdapter::OnChannelError, adapter_));
-}
-
 void WebrtcDataStreamAdapter::Channel::OnMessage(
     const webrtc::DataBuffer& rtc_buffer) {
   std::unique_ptr<CompoundBuffer> buffer(new CompoundBuffer());
   buffer->AppendCopyOf(reinterpret_cast<const char*>(rtc_buffer.data.data()),
                        rtc_buffer.data.size());
   buffer->Lock();
-  message_received_callback_.Run(std::move(buffer));
+  event_handler_->OnMessageReceived(std::move(buffer));
+}
+
+void WebrtcDataStreamAdapter::Channel::OnConnected() {
+  DCHECK(state_ == State::CONNECTING);
+  state_ = State::OPEN;
+  adapter_->OnChannelConnected(this);
+}
+
+void WebrtcDataStreamAdapter::Channel::OnClosed() {
+  switch (state_) {
+    case State::CONNECTING:

[Jamie, 2016/07/19 18:24:47]

You're not setting the state to CLOSED in this branch; is that intentional?

[Sergey Ulanov, 2016/07/19 23:38:44]

No. Fixed now.

+      LOG(WARNING) << "Channel " << channel_->label()
+                   << " was closed before it's connected.";
+      // Notify the adapter about the error asynchronously.
+      base::ThreadTaskRunnerHandle::Get()->PostTask(
+          FROM_HERE,
+          base::Bind(&WebrtcDataStreamAdapter::OnChannelError, adapter_));
+      return;
+
+    case State::OPEN:
+      state_ = State::CLOSED;
+      base::ThreadTaskRunnerHandle::Get()->PostTask(
+          FROM_HERE,
+          base::Bind(&Channel::NotifyClosed, weak_factory_.GetWeakPtr()));
+      return;
+
+    case State::CLOSED:
+      break;
+  }
+}
+
+void WebrtcDataStreamAdapter::Channel::NotifyClosed() {
+  DCHECK(state_ == State::CLOSED);
+  event_handler_->OnMessagePipeClosed();
 }
 
 struct WebrtcDataStreamAdapter::PendingChannel {
-  PendingChannel() {}
   PendingChannel(std::unique_ptr<Channel> channel,
                  const ChannelCreatedCallback& connected_callback)
       : channel(std::move(channel)), connected_callback(connected_callback) {}
   PendingChannel(PendingChannel&& other)
       : channel(std::move(other.channel)),
         connected_callback(std::move(other.connected_callback)) {}
   PendingChannel& operator=(PendingChannel&& other) {
     channel = std::move(other.channel);
     connected_callback = std::move(other.connected_callback);
     return *this;
   }
 
   std::unique_ptr<Channel> channel;
   ChannelCreatedCallback connected_callback;
 };
 
 WebrtcDataStreamAdapter::WebrtcDataStreamAdapter(
-    bool outgoing,
     const ErrorCallback& error_callback)
-    : outgoing_(outgoing),
-      error_callback_(error_callback),
-      weak_factory_(this) {}
+    : error_callback_(error_callback), weak_factory_(this) {}
 
 WebrtcDataStreamAdapter::~WebrtcDataStreamAdapter() {
   DCHECK(pending_channels_.empty());
 }
 
 void WebrtcDataStreamAdapter::Initialize(
     rtc::scoped_refptr<webrtc::PeerConnectionInterface> peer_connection) {
   peer_connection_ = peer_connection;
 }
 
-void WebrtcDataStreamAdapter::OnIncomingDataChannel(
-    webrtc::DataChannelInterface* data_channel) {
-  DCHECK(!outgoing_);
-
-  auto it = pending_channels_.find(data_channel->label());
-  if (it == pending_channels_.end()) {
-    LOG(ERROR) << "Received unexpected data channel " << data_channel->label();
-    return;
-  }
-  it->second.channel->Start(data_channel);
+void WebrtcDataStreamAdapter::WrapIncomingDataChannel(
+    rtc::scoped_refptr<webrtc::DataChannelInterface> data_channel,
+    const ChannelCreatedCallback& callback) {
+  AddPendingChannel(data_channel, callback);
 }
 
 void WebrtcDataStreamAdapter::CreateChannel(
     const std::string& name,
     const ChannelCreatedCallback& callback) {
-  DCHECK(peer_connection_);
-  DCHECK(pending_channels_.find(name) == pending_channels_.end());
-
-  Channel* channel = new Channel(weak_factory_.GetWeakPtr());
-  pending_channels_[name] = PendingChannel(base::WrapUnique(channel), callback);
-
-  if (outgoing_) {
-    webrtc::DataChannelInit config;
-    config.reliable = true;
-    channel->Start(peer_connection_->CreateDataChannel(name, &config));
-  }
+  webrtc::DataChannelInit config;
+  config.reliable = true;
+  AddPendingChannel(peer_connection_->CreateDataChannel(name, &config),
+                    callback);
 }
 
 void WebrtcDataStreamAdapter::CancelChannelCreation(const std::string& name) {
   auto it = pending_channels_.find(name);
   DCHECK(it != pending_channels_.end());
   pending_channels_.erase(it);
 }
 
+void WebrtcDataStreamAdapter::AddPendingChannel(
+    rtc::scoped_refptr<webrtc::DataChannelInterface> data_channel,
+    const ChannelCreatedCallback& callback) {
+  DCHECK(peer_connection_);
+  DCHECK(pending_channels_.find(data_channel->label()) ==
+         pending_channels_.end());
+
+  Channel* channel = new Channel(weak_factory_.GetWeakPtr());
+  pending_channels_.insert(
+      std::make_pair(data_channel->label(),
+                     PendingChannel(base::WrapUnique(channel), callback)));
+
+  channel->Start(data_channel);
+}
+
 void WebrtcDataStreamAdapter::OnChannelConnected(Channel* channel) {
   auto it = pending_channels_.find(channel->name());
   DCHECK(it != pending_channels_.end());
+
   PendingChannel pending_channel = std::move(it->second);
   pending_channels_.erase(it);
 
   // Once the channel is connected its ownership  is passed to the
   // |connected_callback|.
   pending_channel.connected_callback.Run(std::move(pending_channel.channel));
 }
 
 void WebrtcDataStreamAdapter::OnChannelError() {
   error_callback_.Run(CHANNEL_CONNECTION_ERROR);
 }
 
 }  // namespace protocol
 }  // namespace remoting