Go away on odd push and already seen push stream ids

net/spdy/spdy_session_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/spdy/spdy_session.h"
 
 #include "base/base64.h"
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 4957 matching lines @@
 
   CreateNetworkSession();
   base::WeakPtr<SpdySession> session =
       CreateInsecureSpdySession(http_session_, key_, BoundNetLog());
 
   session->stream_hi_water_mark_ = 5;
   // Low client (odd) ids are fine.
   EXPECT_TRUE(session->OnUnknownFrame(3, 0));
   // Client id exceeding watermark.
   EXPECT_FALSE(session->OnUnknownFrame(9, 0));
-  // Currently we do not keep track of server initiated (even) ids.
+
+  session->last_accepted_push_stream_id_ = 6;
+  // Low server (even) ids are fine.
   EXPECT_TRUE(session->OnUnknownFrame(2, 0));
-  EXPECT_TRUE(session->OnUnknownFrame(8, 0));
+  // Server id exceeding last accepted id.
+  EXPECT_FALSE(session->OnUnknownFrame(8, 0));
 }
 
 TEST(MapFramerErrorToProtocolError, MapsValues) {
   CHECK_EQ(
       SPDY_ERROR_INVALID_CONTROL_FRAME,
       MapFramerErrorToProtocolError(SpdyFramer::SPDY_INVALID_CONTROL_FRAME));
   CHECK_EQ(
       SPDY_ERROR_INVALID_DATA_FRAME_FLAGS,
       MapFramerErrorToProtocolError(SpdyFramer::SPDY_INVALID_DATA_FRAME_FLAGS));
   CHECK_EQ(
@@ skipping 136 matching lines @@
   ssl_info.cert = ImportCertFromFile(GetTestCertsDirectory(),
                                      "spdy_pooling.pem");
   ssl_info.is_issued_by_known_root = true;
   ssl_info.public_key_hashes.push_back(test::GetTestHashValue(primary_pin));
 
   EXPECT_TRUE(SpdySession::CanPool(
       &tss, ssl_info, "www.example.org", "mail.example.org"));
 }
 
 }  // namespace net