Send PING to check the status of the SPDY connection.

net/http/http_network_layer.cc

Index: net/http/http_network_layer.cc
===================================================================
--- net/http/http_network_layer.cc	(revision 105385)
+++ net/http/http_network_layer.cc	(working copy)
@@ -42,6 +42,7 @@
   static const char kOff[] = "off";
   static const char kSSL[] = "ssl";
   static const char kDisableSSL[] = "no-ssl";
+  static const char kDisablePing[] = "no-ping";
   static const char kExclude[] = "exclude";  // Hosts to exclude
   static const char kDisableCompression[] = "no-compress";
   static const char kDisableAltProtocols[] = "no-alt-protocols";
@@ -84,6 +85,9 @@
 
   bool use_alt_protocols = true;
 
+  // Enable sending PING unless it is disabled by the options.
+  SpdySession::set_send_ping_for_every_request(true);

[jar (doing other things), 2011/10/14 19:59:07]

As per discussion... I think you should disable this in the few tests that are
currently failing. Then you can (in the default case) have true without this
call.

[ramant (doing other things), 2011/10/14 23:42:45]

Done.

+
   for (std::vector<std::string>::iterator it = spdy_options.begin();
        it != spdy_options.end(); ++it) {
     const std::string& element = *it;
@@ -101,6 +105,8 @@
     } else if (option == kSSL) {
       HttpStreamFactory::set_force_spdy_over_ssl(true);
       HttpStreamFactory::set_force_spdy_always(true);
+    } else if (option == kDisablePing) {
+      SpdySession::set_send_ping_for_every_request(false);
     } else if (option == kExclude) {
       HttpStreamFactory::add_forced_spdy_exclusion(value);
     } else if (option == kDisableCompression) {