tree 8d8665a9c711b93407d38d81dd6dcea611a310d4
parent 1cf7402137a103fe97446bdb5bffa2b8c5dc2535
author dmcardle <dmcardle@google.com> 1763492659 -0800
committer Copybara-Service <copybara-worker@google.com> 1763492711 -0800

Weaken DCHECK in `operator*(QuicBandwidth, QuicTime::Delta)`

cl/829138971 added a DCHECK to operator*(QuicBandwidth, QuicTime::Delta) to assert that multiplications do not commit UB in the form of signed integer overflow. This apparently caused an Envoy integration test to flakily fail when multiplying a positive bandwidth against a negative RTT.

I can reproduce the failure on the linked bug by running the Envoy integration test like so:

```
blaze test --runs_per_test=150 \
  --test_filter='*QuicHttpIntegrationTests/QuicHttpIntegrationTest.MultipleQuicConnectionsNoBPF/IPv6' \
  -- //third_party/envoy/src/test/integration:quic_http_integration_test
```

This CL weakens the DCHECK's condition to permit negative time deltas while we debug the root cause.

PiperOrigin-RevId: 833888758