commit ea74008ed61de8d07bb0a3db6efd2e8b840fde5a
author QUICHE team <quiche-dev@google.com> Mon Mar 11 17:58:43 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Mon Mar 11 17:59:13 2019 -0700
tree 6cfb1d2bff18aba7681bd58d2bde2f32d4d31fea
parent 4d9d629f241c23699eaea4d2d41e98a06359cbe6

Use QuicVersionUsesCryptoFrames instead of checking against QUIC_VERSION_47

gfe-relnote: refactor QUIC_VERSION_47 checks. No behavior change
PiperOrigin-RevId: 237924704
Change-Id: Iaf70e71f64fbcc51f500e3919c2f026b146d7ab6

quic/core/chlo_extractor.cc

diff --git a/quic/core/chlo_extractor.cc b/quic/core/chlo_extractor.cc
index 15a118b..355ecd7 100644
--- a/quic/core/chlo_extractor.cc
+++ b/quic/core/chlo_extractor.cc
@@ -125,7 +125,7 @@
 }
 void ChloFramerVisitor::OnCoalescedPacket(const QuicEncryptedPacket& packet) {}
 bool ChloFramerVisitor::OnStreamFrame(const QuicStreamFrame& frame) {
-  if (framer_->transport_version() >= QUIC_VERSION_47) {
+  if (QuicVersionUsesCryptoFrames(framer_->transport_version())) {
     // CHLO will be sent in CRYPTO frames in v47 and above.
     return false;
   }
@@ -139,7 +139,7 @@
 }
 
 bool ChloFramerVisitor::OnCryptoFrame(const QuicCryptoFrame& frame) {
-  if (framer_->transport_version() < QUIC_VERSION_47) {
+  if (!QuicVersionUsesCryptoFrames(framer_->transport_version())) {
     // CHLO will be in stream frames before v47.
     return false;
   }