message: DCHECK time stamp comparison in BBR bandwidth sampler calculation
  Summary:
  all these conditions should be satisified since we use a steady clock.
  Change the if to DCHECK so it's no longer a const in non-debug mode.
diff: @@ -46,9 +46,9 @@ void BbrBandwidthSampler::onPacketAcked(
  }
  Bandwidth sendRate, ackRate;
  if (outstandingPacket.lastAckedPacketInfo) {
  - // TODO: I think I can DCHECK this condition:
  - if (outstandingPacket.time >
  - outstandingPacket.lastAckedPacketInfo->sentTime) {
  + DCHECK(
  + outstandingPacket.time >
  + outstandingPacket.lastAckedPacketInfo->sentTime);
  DCHECK_GE(
  outstandingPacket.totalBytesSent,
  outstandingPacket.lastAckedPacketInfo->totalBytesSent);
  @@ -58,9 +58,8 @@ void BbrBandwidthSampler::onPacketAcked(
  std::chrono::duration_cast<std::chrono::microseconds>(
  outstandingPacket.time -
  outstandingPacket.lastAckedPacketInfo->sentTime));
  - }
  - if (ackEvent.ackTime > outstandingPacket.lastAckedPacketInfo->ackTime) {
  + DCHECK(ackEvent.ackTime > outstandingPacket.lastAckedPacketInfo->ackTime);
  DCHECK_GE(
  conn_.lossState.totalBytesAcked,
  outstandingPacket.lastAckedPacketInfo->totalBytesAcked);
  @@ -70,7 +69,6 @@ void BbrBandwidthSampler::onPacketAcked(
  std::chrono::duration_cast<std::chrono::microseconds>(
  ackEvent.ackTime -
  outstandingPacket.lastAckedPacketInfo->ackTime));
  - }
  } else if (ackEvent.ackTime > outstandingPacket.time) {
  // No previous ack info from outstanding packet, fallback to units/lrtt.
  // This is a per packet delivery rate. Given there can be multiple packets
  