Sent QUIC "PING" frames when a stream is open and the connection

net/quic/quic_session.cc

Index: net/quic/quic_session.cc
diff --git a/net/quic/quic_session.cc b/net/quic/quic_session.cc
index da1c863fcc64ae46be0806eade8c8046efd5f043..01a47cd6ab96121b8ab15b60ffd211a879263a97 100644
--- a/net/quic/quic_session.cc
+++ b/net/quic/quic_session.cc
@@ -86,6 +86,10 @@ class VisitorShim : public QuicConnectionVisitorInterface {
     return session_->HasPendingHandshake();
   }
 
+  virtual bool HasOpenStreams() const OVERRIDE {
+    return session_->HasOpenStreams();
+  }
+
  private:
   QuicSession* session_;
 };
@@ -301,6 +305,10 @@ bool QuicSession::HasPendingHandshake() const {
   return has_pending_handshake_;
 }
 
+bool QuicSession::HasOpenStreams() const {
+  return GetNumOpenStreams() > 0;

[jar (doing other things), 2014/04/19 16:35:43]

How does the crypto stream fit into the mix?  It is probably open... but we
shouldn't consider it for this purpose.  Is it a "special" stream and excluded
from that count?  Similarly for the header stream.

Maybe the name should be different.  Maybe:
HasClosableStreams() 

[Ryan Hamilton, 2014/04/19 19:18:48]

Both the crypto and headers streams are not considered "open" in this context.
That being said, I've renamed the method to HasOpenDataStreams to be consistent
with the QuicDataStream class (which exclude crypto and headers streams.

+}
+
 QuicConsumedData QuicSession::WritevData(
     QuicStreamId id,
     const IOVector& data,