message: Remove Quic clone packet event when the packet is acked
  Summary:
  somehow the code has been this: removing it when it's acked and it needs
  process. That doesn't sound right. The clone packets counter is updated no
  matter if it's processed
diff: @@ -175,16 +175,15 @@ void processAckFrame(
  }
  // Only invoke AckVisitor if the packet doesn't have an associated
  // PacketEvent; or the PacketEvent is in conn.outstandings.packetEvents
  - if (needsProcess /*!rPacketIt->associatedEvent ||
  - conn.outstandings.packetEvents.count(*rPacketIt->associatedEvent)*/) {
  + if (needsProcess) {
  for (auto& packetFrame : rPacketIt->packet.frames) {
  ackVisitor(*rPacketIt, packetFrame, frame);
  }
  + }
  // Remove this PacketEvent from the outstandings.packetEvents set
  if (rPacketIt->associatedEvent) {
  conn.outstandings.packetEvents.erase(*rPacketIt->associatedEvent);
  }
  - }
  if (!ack.largestAckedPacket ||
  *ack.largestAckedPacket < currentPacketNum) {
  ack.largestAckedPacket = currentPacketNum;
  