Clean up quic_network_transaction_unittests for v99.
Change-Id: I03ee9470e8e5cdae4221902c2a0da1251148cf5a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1828283
Commit-Queue: Ryan Hamilton <[email protected]>
Reviewed-by: Ryan Hamilton <[email protected]>
Cr-Commit-Position: refs/heads/master@{#700868}
diff --git a/net/quic/quic_network_transaction_unittest.cc b/net/quic/quic_network_transaction_unittest.cc
index 69e3099e1b..8a999db 100644
--- a/net/quic/quic_network_transaction_unittest.cc
+++ b/net/quic/quic_network_transaction_unittest.cc
@@ -1283,10 +1283,6 @@
}
TEST_P(QuicNetworkTransactionTest, LargeResponseHeaders) {
- // TODO(rch): honor the max header list size. b/136108828
- if (quic::VersionUsesQpack(version_.transport_version))
- return;
-
session_params_.quic_params.origins_to_force_quic_on.insert(
HostPortPair::FromString("mail.example.org:443"));
@@ -2469,7 +2465,7 @@
TEST_P(QuicNetworkTransactionTest, GoAwayWithConnectionMigrationOnPortsOnly) {
if (version_.transport_version == quic::QUIC_VERSION_99) {
- // Not available under version 99
+ // GoAway is not available under version 99
return;
}
MockQuicData mock_quic_data(version_);
@@ -8886,8 +8882,7 @@
}
if (quic::VersionUsesQpack(version_.transport_version)) {
- // TODO(rch): both stream_dependencies and priority frames need to be
- // supported in IETF QUIC.
+ // HTTP/3 currently doesn't support PRIORITY.
return;
}
