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

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

Issue 1866483002: Add a new priority level, THROTTLED, below IDLE. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Adjusted predictor priorities. Created 4 years, 2 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
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/spdy/spdy_http_utils.h" 5 #include "net/spdy/spdy_http_utils.h"
6 6
7 #include <stdint.h> 7 #include <stdint.h>
8 8
9 #include <limits> 9 #include <limits>
10 10
11 #include "net/http/http_request_info.h" 11 #include "net/http/http_request_info.h"
12 #include "net/spdy/spdy_framer.h" 12 #include "net/spdy/spdy_framer.h"
13 #include "net/spdy/spdy_test_utils.h" 13 #include "net/spdy/spdy_test_utils.h"
14 #include "testing/gtest/include/gtest/gtest.h" 14 #include "testing/gtest/include/gtest/gtest.h"
15 15
16 namespace net { 16 namespace net {
17 17
18 namespace { 18 namespace {
19 19
20 bool kDirect = true; 20 bool kDirect = true;
21 21
22 } // namespace 22 } // namespace
23 23
24 TEST(SpdyHttpUtilsTest, ConvertRequestPriorityToSpdy3Priority) { 24 TEST(SpdyHttpUtilsTest, ConvertRequestPriorityToSpdy3Priority) {
25 EXPECT_EQ(0, ConvertRequestPriorityToSpdyPriority(HIGHEST)); 25 EXPECT_EQ(0, ConvertRequestPriorityToSpdyPriority(HIGHEST));
26 EXPECT_EQ(1, ConvertRequestPriorityToSpdyPriority(MEDIUM)); 26 EXPECT_EQ(1, ConvertRequestPriorityToSpdyPriority(MEDIUM));
27 EXPECT_EQ(2, ConvertRequestPriorityToSpdyPriority(LOW)); 27 EXPECT_EQ(2, ConvertRequestPriorityToSpdyPriority(LOW));
28 EXPECT_EQ(3, ConvertRequestPriorityToSpdyPriority(LOWEST)); 28 EXPECT_EQ(3, ConvertRequestPriorityToSpdyPriority(LOWEST));
29 EXPECT_EQ(4, ConvertRequestPriorityToSpdyPriority(IDLE)); 29 EXPECT_EQ(4, ConvertRequestPriorityToSpdyPriority(IDLE));
30 EXPECT_EQ(5, ConvertRequestPriorityToSpdyPriority(THROTTLED));
30 } 31 }
31 32
32 TEST(SpdyHttpUtilsTest, ConvertSpdy3PriorityToRequestPriority) { 33 TEST(SpdyHttpUtilsTest, ConvertSpdy3PriorityToRequestPriority) {
33 EXPECT_EQ(HIGHEST, ConvertSpdyPriorityToRequestPriority(0)); 34 EXPECT_EQ(HIGHEST, ConvertSpdyPriorityToRequestPriority(0));
34 EXPECT_EQ(MEDIUM, ConvertSpdyPriorityToRequestPriority(1)); 35 EXPECT_EQ(MEDIUM, ConvertSpdyPriorityToRequestPriority(1));
35 EXPECT_EQ(LOW, ConvertSpdyPriorityToRequestPriority(2)); 36 EXPECT_EQ(LOW, ConvertSpdyPriorityToRequestPriority(2));
36 EXPECT_EQ(LOWEST, ConvertSpdyPriorityToRequestPriority(3)); 37 EXPECT_EQ(LOWEST, ConvertSpdyPriorityToRequestPriority(3));
37 EXPECT_EQ(IDLE, ConvertSpdyPriorityToRequestPriority(4)); 38 EXPECT_EQ(IDLE, ConvertSpdyPriorityToRequestPriority(4));
39 EXPECT_EQ(THROTTLED, ConvertSpdyPriorityToRequestPriority(5));
38 // These are invalid values, but we should still handle them 40 // These are invalid values, but we should still handle them
39 // gracefully. 41 // gracefully.
40 for (int i = 5; i < std::numeric_limits<uint8_t>::max(); ++i) { 42 for (int i = 6; i < std::numeric_limits<uint8_t>::max(); ++i) {
41 EXPECT_EQ(IDLE, ConvertSpdyPriorityToRequestPriority(i)); 43 EXPECT_EQ(IDLE, ConvertSpdyPriorityToRequestPriority(i));
42 } 44 }
43 } 45 }
44 46
45 TEST(SpdyHttpUtilsTest, CreateSpdyHeadersFromHttpRequestHTTP2) { 47 TEST(SpdyHttpUtilsTest, CreateSpdyHeadersFromHttpRequestHTTP2) {
46 GURL url("https://www.google.com/index.html"); 48 GURL url("https://www.google.com/index.html");
47 HttpRequestInfo request; 49 HttpRequestInfo request;
48 request.method = "GET"; 50 request.method = "GET";
49 request.url = url; 51 request.url = url;
50 request.extra_headers.SetHeader(HttpRequestHeaders::kUserAgent, "Chrome/1.1"); 52 request.extra_headers.SetHeader(HttpRequestHeaders::kUserAgent, "Chrome/1.1");
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
88 EXPECT_EQ("CONNECT", headers[":method"]); 90 EXPECT_EQ("CONNECT", headers[":method"]);
89 EXPECT_TRUE(headers.end() == headers.find(":scheme")); 91 EXPECT_TRUE(headers.end() == headers.find(":scheme"));
90 EXPECT_EQ("www.google.com:443", headers[":authority"]); 92 EXPECT_EQ("www.google.com:443", headers[":authority"]);
91 EXPECT_EQ(headers.end(), headers.find(":path")); 93 EXPECT_EQ(headers.end(), headers.find(":path"));
92 EXPECT_EQ(headers.end(), headers.find(":scheme")); 94 EXPECT_EQ(headers.end(), headers.find(":scheme"));
93 EXPECT_TRUE(headers.end() == headers.find(":version")); 95 EXPECT_TRUE(headers.end() == headers.find(":version"));
94 EXPECT_EQ("Chrome/1.1", headers["user-agent"]); 96 EXPECT_EQ("Chrome/1.1", headers["user-agent"]);
95 } 97 }
96 98
97 } // namespace net 99 } // namespace net
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698