Create QuicSession

webrtc/p2p/base/reliablequicstream.cc

+/*
+ *  Copyright 2011 The WebRTC Project Authors. All rights reserved.
+ *
+ *  Use of this source code is governed by a BSD-style license
+ *  that can be found in the LICENSE file in the root of the source
+ *  tree. An additional intellectual property rights grant can be found
+ *  in the file PATENTS.  All contributing project authors may
+ *  be found in the AUTHORS file in the root of the source tree.
+ */
+
+#include "net/base/net_errors.h"
+
+#include "webrtc/base/checks.h"
+#include "webrtc/p2p/base/reliablequicstream.h"
+
+namespace cricket {
+
+ReliableQuicStream::ReliableQuicStream(net::QuicStreamId id,
+                                       net::QuicSession* session)
+    : net::ReliableQuicStream(id, session) {
+  RTC_DCHECK_NE(net::kCryptoStreamId, id);
+}
+
+ReliableQuicStream::~ReliableQuicStream() {}
+
+void ReliableQuicStream::OnDataAvailable() {
+  struct iovec iov;
+  while (sequencer()->GetReadableRegions(&iov, 1) == 1) {
+    SignalDataReceived(id(), reinterpret_cast<const char*>(iov.iov_base),
+                       iov.iov_len);
+    sequencer()->MarkConsumed(iov.iov_len);
+  }
+}

[pthatcher1, 2016/01/30 03:01:52]

Are you going to write a simple unit test for this as well?

[mikescarlett, 2016/02/03 17:41:01]

Yes I will.

+
+void ReliableQuicStream::OnClose() {
+  net::ReliableQuicStream::OnClose();
+
+  SignalStreamClosed(id(), connection_error());
+}
+
+int ReliableQuicStream::Write(base::StringPiece data) {
+  // Writes the data, or buffers it.
+  WriteOrBufferData(data, false, nullptr);
+  if (!HasBufferedData()) {
+    return data.size();
+  }
+
+  return net::ERR_IO_PENDING;

[pthatcher1, 2016/01/30 03:01:52]

Can you convert this to an rtc::StreamResult?

[pthatcher1, 2016/01/30 03:01:52]

If prefer it in the reverse:

if (HasBufferedData()) {
  return net::ERR_IO_PENDING;
}

return data.size();

[mikescarlett, 2016/02/03 17:41:01]

Reversed if statement.

[mikescarlett, 2016/02/03 17:41:01]

Converted to rtc::StreamResult. Also changed the method signature.

+}
+
+}  // namespace cricket