Clean up SpdySessionDependencies to remove code duplication.

net/spdy/spdy_test_util_spdy2.h

Index: net/spdy/spdy_test_util_spdy2.h
diff --git a/net/spdy/spdy_test_util_spdy2.h b/net/spdy/spdy_test_util_spdy2.h
index 792b19c8dbd6eef2f9bb7bd48e56ef7439475f89..58b228dc21687a0fdc78c9594c6b52274ed6f1d9 100644
--- a/net/spdy/spdy_test_util_spdy2.h
+++ b/net/spdy/spdy_test_util_spdy2.h
@@ -354,6 +354,8 @@ class SpdySessionDependencies {
   static HttpNetworkSession* SpdyCreateSessionDeterministic(
       SpdySessionDependencies* session_deps);
 
+  net::HttpNetworkSession::Params CreateSessionParams();

[willchan no longer on Chromium, 2012/12/01 21:00:22]

If it would not require a lot of manual effort (your time is more valuable than
this nit), can you keep this static too, to be consistent with the others?

[Ryan Hamilton, 2012/12/01 22:26:41]

Done.  (only used in a couple places)

+
   // NOTE: host_resolver must be ordered before http_auth_handler_factory.
   scoped_ptr<MockHostResolverBase> host_resolver;
   scoped_ptr<CertVerifier> cert_verifier;
@@ -364,6 +366,7 @@ class SpdySessionDependencies {
   scoped_ptr<HttpAuthHandlerFactory> http_auth_handler_factory;
   HttpServerPropertiesImpl http_server_properties;
   std::string trusted_spdy_proxy;
+  NetLog* net_log;
 };
 
 class SpdyURLRequestContext : public URLRequestContext {