Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(19)

Unified Diff: webrtc/p2p/quic/quicsession.cc

Issue 1844803002: Modify PeerConnection for end-to-end QuicDataChannel usage (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Remove webrtcsdp.cc from this CL Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/p2p/quic/quicsession.cc
diff --git a/webrtc/p2p/quic/quicsession.cc b/webrtc/p2p/quic/quicsession.cc
index 281af5e3a745a254475f5c85b8d63949c8452384..37dfc7ddc84b1396f361b2ad87d9327692ccca28 100644
--- a/webrtc/p2p/quic/quicsession.cc
+++ b/webrtc/p2p/quic/quicsession.cc
@@ -75,15 +75,14 @@ ReliableQuicStream* QuicSession::CreateOutgoingDynamicStream(
net::SpdyPriority priority) {
ReliableQuicStream* stream = CreateDataStream(GetNextOutgoingStreamId());
if (stream) {
- ActivateStream(stream);
+ ActivateStream(stream); // QuicSession owns the stream.
}
return stream;
}
ReliableQuicStream* QuicSession::CreateDataStream(net::QuicStreamId id) {
if (crypto_stream_ == nullptr || !crypto_stream_->encryption_established()) {
- // Encryption not active so no stream created
- return nullptr;
+ return nullptr; // Encryption not active so no stream created
}
return new ReliableQuicStream(id, this);
}

Powered by Google App Engine
This is Rietveld 408576698