Server push cancellation: add PushPromiseHelper which reflects information on the push promise.

net/spdy/spdy_session.h

Index: net/spdy/spdy_session.h
diff --git a/net/spdy/spdy_session.h b/net/spdy/spdy_session.h
index fcd2badc18e92c9e04519d1a2d60b7d5f84c6e06..ab92e22ef1eb2a59f68f0541cdec40313627f047 100644
--- a/net/spdy/spdy_session.h
+++ b/net/spdy/spdy_session.h
@@ -326,6 +326,10 @@ class NET_EXPORT SpdySession : public BufferedSpdyFramerVisitorInterface,
                     base::WeakPtr<SpdyStream>* spdy_stream,
                     const NetLogWithSource& stream_net_log);
 
+  // Called when the pushed stream should be cancelled. If the pushed stream is
+  // not claimed and active, sends RST to the server to cancel the stream.
+  void CancelPush(const GURL& url);
+

[Zhongyi Shi, 2016/10/27 19:59:39]

Code not in use => code in the wrong place. This method should be public as
PushPromiseHelper need to call into it. 

   // Initialize the session with the given connection. |is_secure|
   // must indicate whether |connection| uses an SSL socket or not; it
   // is usually true, but it can be false for testing or when SPDY is
@@ -677,10 +681,6 @@ class NET_EXPORT SpdySession : public BufferedSpdyFramerVisitorInterface,
                            SpdyPriority priority,
                            SpdyHeaderBlock headers);
 
-  // Called when the pushed stream should be cancelled. If the pushed stream is
-  // not claimed and active, sends RST to the server to cancel the stream.
-  void CancelPush(const GURL& url);
-
   // Close the stream pointed to by the given iterator. Note that that
   // stream may hold the last reference to the session.
   void CloseActiveStreamIterator(ActiveStreamMap::iterator it, int status);