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

Unified Diff: net/spdy/spdy_session_pool_unittest.cc

Issue 1779733003: Fix bug in net::RequestPriority -> HTTP/2 dependency conversion. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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 side-by-side diff with in-line comments
Download patch
Index: net/spdy/spdy_session_pool_unittest.cc
diff --git a/net/spdy/spdy_session_pool_unittest.cc b/net/spdy/spdy_session_pool_unittest.cc
index 86a887a27a06398ae9211dde130adb88f6962137..fbc0feaf29be8c8a07f43141382474f09118b55e 100644
--- a/net/spdy/spdy_session_pool_unittest.cc
+++ b/net/spdy/spdy_session_pool_unittest.cc
@@ -521,7 +521,8 @@ TEST_P(SpdySessionPoolTest, IPAddressChanged) {
// setting them (when doing the appropriate protocol) since that's
// where we're eventually headed for all HTTP/2 connections.
SpdyTestUtil spdy_util(GetParam(), true);
- SpdySession::SetPriorityDependencyDefaultForTesting(true);
+ bool old_dependency_default =
+ Http2PriorityDependencies::SetEnabledDefaultForTesting(true);
MockRead reads[] = {
MockRead(SYNCHRONOUS, ERR_IO_PENDING) // Stall forever.
@@ -630,7 +631,8 @@ TEST_P(SpdySessionPoolTest, IPAddressChanged) {
EXPECT_TRUE(delegateB.StreamIsClosed());
EXPECT_EQ(ERR_NETWORK_CHANGED, delegateB.WaitForClose());
#endif // defined(OS_ANDROID) || defined(OS_WIN) || defined(OS_IOS)
- SpdySession::SetPriorityDependencyDefaultForTesting(false);
+ Http2PriorityDependencies::SetEnabledDefaultForTesting(
+ old_dependency_default);
}
} // namespace net

Powered by Google App Engine
This is Rietveld 408576698