Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(416)

Side by Side Diff: net/spdy/spdy_network_transaction_spdy21_unittest.cc

Issue 9773004: Explicitly disable SPDY ping on: SpdyNetworkTransactionSpdy*Test.FourGetsWithMaxConcurrentPriority (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | net/spdy/spdy_network_transaction_spdy2_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "net/http/http_network_transaction.h" 5 #include "net/http/http_network_transaction.h"
6 6
7 #include <string> 7 #include <string>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 1037 matching lines...) Expand 10 before | Expand all | Expand 10 after
1048 } 1048 }
1049 EXPECT_EQ(OK, out.rv); 1049 EXPECT_EQ(OK, out.rv);
1050 } 1050 }
1051 1051
1052 // Similar to ThreeGetsWithMaxConcurrent above, however this test adds 1052 // Similar to ThreeGetsWithMaxConcurrent above, however this test adds
1053 // a fourth transaction. The third and fourth transactions have 1053 // a fourth transaction. The third and fourth transactions have
1054 // different data ("hello!" vs "hello!hello!") and because of the 1054 // different data ("hello!" vs "hello!hello!") and because of the
1055 // user specified priority, we expect to see them inverted in 1055 // user specified priority, we expect to see them inverted in
1056 // the response from the server. 1056 // the response from the server.
1057 TEST_P(SpdyNetworkTransactionSpdy21Test, FourGetsWithMaxConcurrentPriority) { 1057 TEST_P(SpdyNetworkTransactionSpdy21Test, FourGetsWithMaxConcurrentPriority) {
1058 SpdySession::set_enable_ping_based_connection_checking(false);
1059
1058 // Construct the request. 1060 // Construct the request.
1059 scoped_ptr<spdy::SpdyFrame> req(ConstructSpdyGet(NULL, 0, false, 1, LOWEST)); 1061 scoped_ptr<spdy::SpdyFrame> req(ConstructSpdyGet(NULL, 0, false, 1, LOWEST));
1060 scoped_ptr<spdy::SpdyFrame> resp(ConstructSpdyGetSynReply(NULL, 0, 1)); 1062 scoped_ptr<spdy::SpdyFrame> resp(ConstructSpdyGetSynReply(NULL, 0, 1));
1061 scoped_ptr<spdy::SpdyFrame> body(ConstructSpdyBodyFrame(1, false)); 1063 scoped_ptr<spdy::SpdyFrame> body(ConstructSpdyBodyFrame(1, false));
1062 scoped_ptr<spdy::SpdyFrame> fbody(ConstructSpdyBodyFrame(1, true)); 1064 scoped_ptr<spdy::SpdyFrame> fbody(ConstructSpdyBodyFrame(1, true));
1063 1065
1064 scoped_ptr<spdy::SpdyFrame> req2(ConstructSpdyGet(NULL, 0, false, 3, LOWEST)); 1066 scoped_ptr<spdy::SpdyFrame> req2(ConstructSpdyGet(NULL, 0, false, 3, LOWEST));
1065 scoped_ptr<spdy::SpdyFrame> resp2(ConstructSpdyGetSynReply(NULL, 0, 3)); 1067 scoped_ptr<spdy::SpdyFrame> resp2(ConstructSpdyGetSynReply(NULL, 0, 3));
1066 scoped_ptr<spdy::SpdyFrame> body2(ConstructSpdyBodyFrame(3, false)); 1068 scoped_ptr<spdy::SpdyFrame> body2(ConstructSpdyBodyFrame(3, false));
1067 scoped_ptr<spdy::SpdyFrame> fbody2(ConstructSpdyBodyFrame(3, true)); 1069 scoped_ptr<spdy::SpdyFrame> fbody2(ConstructSpdyBodyFrame(3, true));
(...skipping 4797 matching lines...) Expand 10 before | Expand all | Expand 10 after
5865 << " Write index: " 5867 << " Write index: "
5866 << data->write_index(); 5868 << data->write_index();
5867 5869
5868 // Verify the SYN_REPLY. 5870 // Verify the SYN_REPLY.
5869 HttpResponseInfo response = *trans->GetResponseInfo(); 5871 HttpResponseInfo response = *trans->GetResponseInfo();
5870 EXPECT_TRUE(response.headers != NULL); 5872 EXPECT_TRUE(response.headers != NULL);
5871 EXPECT_EQ("HTTP/1.1 200 OK", response.headers->GetStatusLine()); 5873 EXPECT_EQ("HTTP/1.1 200 OK", response.headers->GetStatusLine());
5872 } 5874 }
5873 5875
5874 } // namespace net 5876 } // namespace net
OLDNEW
« no previous file with comments | « no previous file | net/spdy/spdy_network_transaction_spdy2_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698