Convert a QUIC LOG(WARNING) into a LOG(INFO), because a buggy client or a client with corrupted mem…

net/quic/quic_connection.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/quic/quic_connection.h"
 
 #include <string.h>
 #include <sys/types.h>
 
 #include <algorithm>
@@ skipping 828 matching lines @@
   if (incoming_ack.largest_observed > packet_generator_.packet_number()) {
     DLOG(WARNING) << ENDPOINT << "Peer's observed unsent packet:"
                   << incoming_ack.largest_observed << " vs "
                   << packet_generator_.packet_number();
     // We got an error for data we have not sent.  Error out.
     return "Largest observed too high.";
   }
 
   if (incoming_ack.largest_observed <
       sent_packet_manager_->GetLargestObserved(incoming_ack.path_id)) {
-    LOG(WARNING) << ENDPOINT << "Peer's largest_observed packet decreased:"
-                 << incoming_ack.largest_observed << " vs "
-                 << sent_packet_manager_->GetLargestObserved(
-                        incoming_ack.path_id)
-                 << " packet_number:" << last_header_.packet_number
-                 << " largest seen with ack:" << largest_seen_packet_with_ack_
-                 << " connection_id: " << connection_id_;
+    VLOG(1) << ENDPOINT << "Peer's largest_observed packet decreased:"
+            << incoming_ack.largest_observed << " vs "
+            << sent_packet_manager_->GetLargestObserved(
+                incoming_ack.path_id)
+            << " packet_number:" << last_header_.packet_number
+            << " largest seen with ack:" << largest_seen_packet_with_ack_
+            << " connection_id: " << connection_id_;
     // A new ack has a diminished largest_observed value.  Error out.
     // If this was an old packet, we wouldn't even have checked.
     return "Largest observed too low.";
   }
 
   if (version() <= QUIC_VERSION_33) {
     if (!incoming_ack.packets.Empty() &&
         incoming_ack.packets.Max() > incoming_ack.largest_observed) {
       LOG(WARNING) << ENDPOINT
                    << "Peer sent missing packet: " << incoming_ack.packets.Max()
@@ skipping 1628 matching lines @@
 // the sender and a signaling mechanism -- if the sender uses a
 // different MinRTO, we may get spurious retransmissions. May not have
 // any benefits, but if the delayed ack becomes a significant source
 // of (likely, tail) latency, then consider such a mechanism.
 const QuicTime::Delta QuicConnection::DelayedAckTime() {
   return QuicTime::Delta::FromMilliseconds(
       min(kMaxDelayedAckTimeMs, kMinRetransmissionTimeMs / 2));
 }
 
 }  // namespace net