commit 3dbf3a53c60589678bacb5bc4b0d12ec1fdee306
author danzh <danzh@google.com> Thu Apr 29 12:26:31 2021 -0700
committer Copybara-Service <copybara-worker@google.com> Thu Apr 29 17:04:09 2021 -0700
tree ef7910e5cf30138752b451da2c8004916ebae7a2
parent ffae80c46d83ad299b72dc9badc95bff5d777b3a

add code count to verify reverse path validation successful rate

CODE_COUNT only, no behavior change

PiperOrigin-RevId: 371177427
Change-Id: I574f0ce817dfb4bb5c2ea526062dea9457f482f7

quic/core/quic_connection.cc

diff --git a/quic/core/quic_connection.cc b/quic/core/quic_connection.cc
index 38117a4..500ec61 100644
--- a/quic/core/quic_connection.cc
+++ b/quic/core/quic_connection.cc
@@ -1806,6 +1806,7 @@
     // higher prority.
     QUIC_DVLOG(1) << "Proactively validate the effective peer address "
                   << current_effective_peer_address;
+    QUIC_CODE_COUNT_N(quic_kick_off_client_address_validation, 2, 6);
     ValidatePath(std::make_unique<ReversePathValidationContext>(
                      default_path_.self_address, current_effective_peer_address,
                      current_effective_peer_address, this),
@@ -5609,6 +5610,7 @@
           // higher prority.
           QUIC_DVLOG(1) << "Proactively validate the effective peer address "
                         << current_effective_peer_address;
+          QUIC_CODE_COUNT_N(quic_kick_off_client_address_validation, 1, 6);
           ValidatePath(
               std::make_unique<ReversePathValidationContext>(
                   default_path_.self_address, last_packet_source_address_,
@@ -7000,10 +7002,12 @@
   QUIC_DLOG(INFO) << "Successfully validated new path " << *context;
   if (connection_->IsDefaultPath(context->self_address(),
                                  context->peer_address())) {
+    QUIC_CODE_COUNT_N(quic_kick_off_client_address_validation, 3, 6);
     connection_->OnEffectivePeerMigrationValidated();
   } else {
     QUICHE_DCHECK(connection_->IsAlternativePath(
         context->self_address(), context->effective_peer_address()));
+    QUIC_CODE_COUNT_N(quic_kick_off_client_address_validation, 4, 6);
     QUIC_DVLOG(1) << "Mark alternative peer address "
                   << context->effective_peer_address() << " validated.";
     connection_->alternative_path_.validated = true;
@@ -7020,9 +7024,11 @@
   if (connection_->IsDefaultPath(context->self_address(),
                                  context->peer_address())) {
     // Only act upon validation failure on the default path.
+    QUIC_CODE_COUNT_N(quic_kick_off_client_address_validation, 5, 6);
     connection_->RestoreToLastValidatedPath(original_direct_peer_address_);
   } else if (connection_->IsAlternativePath(
                  context->self_address(), context->effective_peer_address())) {
+    QUIC_CODE_COUNT_N(quic_kick_off_client_address_validation, 6, 6);
     connection_->alternative_path_.Clear();
   }
 }