Data channel implementation for SessionDependencyFactory.

components/devtools_bridge/session_dependency_factory.cc

 // Copyright 2014 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 "components/devtools_bridge/session_dependency_factory.h"
 
 #include "components/devtools_bridge/abstract_data_channel.h"
 #include "components/devtools_bridge/abstract_peer_connection.h"
 #include "components/devtools_bridge/rtc_configuration.h"
 #include "third_party/libjingle/source/talk/app/webrtc/mediaconstraintsinterface.h"
@@ skipping 50 matching lines @@
 
   void AddMandatory(const std::string& key, const std::string& value) {
     mandatory_.push_back(Constraint(key, value));
   }
 
  private:
   Constraints mandatory_;
   Constraints optional_;
 };
 
+class DataChannelObserverImpl : public webrtc::DataChannelObserver {
+ public:
+  DataChannelObserverImpl(
+      webrtc::DataChannelInterface* data_channel,
+      scoped_ptr<AbstractDataChannel::Observer> observer)
+      : data_channel_(data_channel),
+        observer_(observer.Pass()) {

[mnaganov (inactive), 2014/11/18 03:38:59]

You should also initialize 'open_' here, as otherwise it will be in a random
state until the first assignment.

[SeRya, 2014/11/18 06:57:27]

That's OK. Instance is not used until InitState called on signaling thread.

+  }
+
+  void InitState() {
+    open_ = data_channel_->state() == webrtc::DataChannelInterface::kOpen;
+  }
+
+  virtual void OnStateChange() override {
+    bool open = data_channel_->state() == webrtc::DataChannelInterface::kOpen;
+
+    if (open == open_) return;
+
+    open_ = open;
+    if (open) {
+      observer_->OnOpen();
+    } else {
+      observer_->OnClose();
+    }
+  }
+
+  virtual void OnMessage(const webrtc::DataBuffer& buffer) override {
+      observer_->OnMessage(buffer.data.data(), buffer.size());

[mnaganov (inactive), 2014/11/18 03:38:59]

nit: indentation

[SeRya, 2014/11/18 06:57:26]

Done.

+  }
+
+ private:
+  webrtc::DataChannelInterface* const data_channel_;
+  scoped_ptr<AbstractDataChannel::Observer> const observer_;
+  bool open_;
+};
+
 class DataChannelImpl : public AbstractDataChannel {
  public:
   explicit DataChannelImpl(
-      rtc::scoped_refptr<webrtc::DataChannelInterface> impl) : impl_(impl) {
+      rtc::Thread* const signaling_thread,
+      rtc::scoped_refptr<webrtc::DataChannelInterface> impl)
+      : signaling_thread_(signaling_thread),
+        impl_(impl) {
   }
 
-  // TODO(serya): Implement.
+  virtual void RegisterObserver(scoped_ptr<Observer> observer) override {
+    observer_.reset(new DataChannelObserverImpl(impl_.get(), observer.Pass()));
+    signaling_thread_->Invoke<void>(rtc::Bind(
+        &DataChannelImpl::RegisterObserverOnSignalingThread, this));
+  }
+
+  virtual void UnregisterObserver() override {
+    DCHECK(observer_.get() != NULL);
+    impl_->UnregisterObserver();
+    observer_.reset();
+  }
+
+  virtual void SendBinaryMessage(void* data, size_t length) override {
+    SendMessage(data, length, true);
+  }
+
+  virtual void SendTextMessage(void* data, size_t length) override {
+    SendMessage(data, length, false);
+  }
+
+  void SendMessage(void* data, size_t length, bool binary) {

[mnaganov (inactive), 2014/11/18 03:38:59]

is_binary

[SeRya, 2014/11/18 06:57:26]

Done.

+    impl_->Send(webrtc::DataBuffer(rtc::Buffer(data, length), binary));
+  }
+
+  void Close() override {
+    impl_->Close();
+  }
 
  private:
+  void RegisterObserverOnSignalingThread() {
+    // State initialization and observer registration happen atomically
+    // if done on the signaling thread (see rtc::Thread::Send).
+    observer_->InitState();
+    impl_->RegisterObserver(observer_.get());
+  }
+
+  rtc::Thread* const signaling_thread_;
+  scoped_ptr<DataChannelObserverImpl> observer_;
   const rtc::scoped_refptr<webrtc::DataChannelInterface> impl_;
 };
 
 class PeerConnectionObserverImpl
     : public webrtc::PeerConnectionObserver {
  public:
   PeerConnectionObserverImpl(AbstractPeerConnection::Delegate* delegate)
       : delegate_(delegate),
         connected_(false) {
   }
@@ skipping 218 matching lines @@
       return;
     }
     // Doesn't takes ownership.
     connection_->AddIceCandidate(candidate);
     delete candidate;
   }
 
   virtual scoped_ptr<AbstractDataChannel> CreateDataChannel(
       int channelId) override {
     webrtc::DataChannelInit init;
-    init.reliable = true;
     init.ordered = true;
     init.negotiated = true;
     init.id = channelId;
 
     return make_scoped_ptr(new DataChannelImpl(
+        signaling_thread_,
         connection_->CreateDataChannel("", &init)));
   }
 
  private:
   webrtc::CreateSessionDescriptionObserver* MakeCreateAndSetHandler() {
     return new rtc::RefCountedObject<CreateAndSetHandler>(holder_);
   }
 
   void DisposeOnSignalingThread() {
     DCHECK(signaling_thread_->IsCurrent());
@@ skipping 86 matching lines @@
 }
 
 // static
 scoped_ptr<SessionDependencyFactory> SessionDependencyFactory::CreateInstance(
     const base::Closure& cleanup_on_signaling_thread) {
   return make_scoped_ptr(new SessionDependencyFactoryImpl(
       cleanup_on_signaling_thread));
 }
 
 }  // namespace devtools_bridge