| Index: trunk/src/net/http/http_stream_factory_impl_request.cc
|
| ===================================================================
|
| --- trunk/src/net/http/http_stream_factory_impl_request.cc (revision 274100)
|
| +++ trunk/src/net/http/http_stream_factory_impl_request.cc (working copy)
|
| @@ -250,13 +250,6 @@
|
| }
|
| }
|
|
|
| -// TODO(jgraettinger): Currently, HttpStreamFactoryImpl::Job notifies a
|
| -// Request that the session is ready, which in turn notifies it's delegate,
|
| -// and then it notifies HttpStreamFactoryImpl so that /other/ requests may
|
| -// be woken, but only if the spdy_session is still okay. This is tough to grok.
|
| -// Instead, see if Job can notify HttpStreamFactoryImpl only, and have one
|
| -// path for notifying any requests waiting for the session (including the
|
| -// request which spawned it).
|
| void HttpStreamFactoryImpl::Request::OnNewSpdySessionReady(
|
| Job* job,
|
| scoped_ptr<HttpStream> stream,
|
| @@ -299,7 +292,7 @@
|
| stream.release());
|
| }
|
| // |this| may be deleted after this point.
|
| - if (spdy_session && spdy_session->IsAvailable()) {
|
| + if (spdy_session) {
|
| factory->OnNewSpdySessionReady(spdy_session,
|
| direct,
|
| used_ssl_config,
|
|
|