Improve testing for SDCH.

net/http/http_transaction_test_util.h

Index: net/http/http_transaction_test_util.h
diff --git a/net/http/http_transaction_test_util.h b/net/http/http_transaction_test_util.h
index d340fc9c05023732c0ddc4309fbae6cbbcab84b7..a56e6a7f684c8cfae2b9d4788b29e038e79e2586 100644
--- a/net/http/http_transaction_test_util.h
+++ b/net/http/http_transaction_test_util.h
@@ -28,6 +28,7 @@
 namespace net {
 class HttpRequestHeaders;
 class IOBuffer;
+struct HttpRequestInfo;
 }
 
 //-----------------------------------------------------------------------------
@@ -227,6 +228,9 @@ class MockNetworkTransaction
     return websocket_handshake_stream_create_helper_;
   }
   net::RequestPriority priority() const { return priority_; }
+  const net::HttpRequestInfo* http_request_info() const {

[mef, 2014/07/23 18:34:30]

nit: nl above, maybe name it |request()| to match variable name.

[Randy Smith (Not in Mondays), 2014/07/29 23:44:22]

The nl doesn't fit to my eye with the relationship of the two methods above--I
take this as "simple accessors are clustered together without newlines
separating them", but if you feel strongly I'll change it.  I've changed the
name.

[mef, 2014/07/30 17:14:58]

Acknowledged.

+    return request_;
+  }
 
  private:
   int StartInternal(const net::HttpRequestInfo* request,