Fix QuicConnectionTest failure in Chromium on 32-bit Android platforms. PiperOrigin-RevId: 502872432
diff --git a/quiche/quic/core/quic_connection_test.cc b/quiche/quic/core/quic_connection_test.cc index e0c92b1..c4fe88f 100644 --- a/quiche/quic/core/quic_connection_test.cc +++ b/quiche/quic/core/quic_connection_test.cc
@@ -16938,8 +16938,7 @@ connection_.SupportsMultiplePacketNumberSpaces() ? 1 : 2); } -// TODO(b/265478749): re-enable this test. -TEST_P(QuicConnectionTest, DISABLED_ReceivedPacketInfoDefaults) { +TEST_P(QuicConnectionTest, ReceivedPacketInfoDefaults) { EXPECT_TRUE(QuicConnectionPeer::TestLastReceivedPacketInfoDefaults()); }
diff --git a/quiche/quic/test_tools/quic_connection_peer.cc b/quiche/quic/test_tools/quic_connection_peer.cc index 3d63682..9506f37 100644 --- a/quiche/quic/test_tools/quic_connection_peer.cc +++ b/quiche/quic/test_tools/quic_connection_peer.cc
@@ -585,6 +585,23 @@ // static bool QuicConnectionPeer::TestLastReceivedPacketInfoDefaults() { QuicConnection::ReceivedPacketInfo info{QuicTime::Zero()}; + QUIC_DVLOG(2) + << "QuicConnectionPeer::TestLastReceivedPacketInfoDefaults" + << " dest_addr passed: " + << (info.destination_address == QuicSocketAddress()) + << " source_addr passed: " << (info.source_address == QuicSocketAddress()) + << " receipt_time passed: " << (info.receipt_time == QuicTime::Zero()) + << " received_bytes_counted passed: " << !info.received_bytes_counted + << " destination_connection_id passed: " + << (info.destination_connection_id == QuicConnectionId()) + << " length passed: " << (info.length == 0) + << " decrypted passed: " << !info.decrypted << " decrypted_level passed: " + << (info.decrypted_level == ENCRYPTION_INITIAL) + << " frames.empty passed: " << info.frames.empty() + << " ecn_codepoint passed: " << (info.ecn_codepoint == ECN_NOT_ECT) + << " sizeof(ReceivedPacketInfo) passed: " + << (sizeof(size_t) != 8 || + sizeof(QuicConnection::ReceivedPacketInfo) == 256); return info.destination_address == QuicSocketAddress() && info.source_address == QuicSocketAddress() && info.receipt_time == QuicTime::Zero() && @@ -596,7 +613,8 @@ // If the condition below fails, the contents of ReceivedPacketInfo // have changed. Please add the relevant conditions and update the // length below. - sizeof(QuicConnection::ReceivedPacketInfo) == 256; + (sizeof(size_t) != 8 || + sizeof(QuicConnection::ReceivedPacketInfo) == 256); } } // namespace test