When an idle socket is added back to a socket pool, check for stalled jobs in lower pools

net/spdy/spdy_session.cc

Index: net/spdy/spdy_session.cc
===================================================================
--- net/spdy/spdy_session.cc	(revision 211652)
+++ net/spdy/spdy_session.cc	(working copy)
@@ -504,7 +504,7 @@
   if (error == ERR_IO_PENDING)
     error = OK;
   if (error == OK) {
-    connection_->AddLayeredPool(this);
+    connection_->AddHigherLayeredPool(this);
     SendInitialSettings();
     // Write out any data that we might have to send, such as the
     // settings frame.
@@ -998,6 +998,11 @@
   active_streams_.erase(it);
 
   DeleteStream(owned_stream.Pass(), status);
+
+  // If there are no active streams, and the socket pool is stalled, try to free
+  // up a socket slot.
+  if (active_streams_.empty() && connection_->IsPoolStalled())
+    RemoveFromPool();

[Ryan Hamilton, 2013/07/21 14:53:54]

akalin has been refactoring SpdySession quite a bit recently.  You might need to
rebase, I'm not sure.

[mmenke, 2013/07/22 20:13:33]

Yea, this completely doesn't work now.  Removing from the pool now deletes the
session, but a lot of methods run code after CloseActiveStreamIterator.

I'll have to get a better understanding of SpdySessions to figure out how to
update the CL.

 }
 
 void SpdySession::CloseCreatedStreamIterator(CreatedStreamSet::iterator it,