Remove rtcp_utility as mostly unused.

webrtc/modules/rtp_rtcp/source/nack_stats.h

+/*
+*  Copyright (c) 2017 The WebRTC project authors. All Rights Reserved.
+*
+*  Use of this source code is governed by a BSD-style license
+*  that can be found in the LICENSE file in the root of the source
+*  tree. An additional intellectual property rights grant can be found
+*  in the file PATENTS.  All contributing project authors may
+*  be found in the AUTHORS file in the root of the source tree.
+*/
+
+#ifndef WEBRTC_MODULES_RTP_RTCP_SOURCE_NACK_STATS_H_
+#define WEBRTC_MODULES_RTP_RTCP_SOURCE_NACK_STATS_H_
+
+#include <stdint.h>
+
+namespace webrtc {
+
+class NackStats {

[nisse-webrtc, 2017/02/09 12:16:52]

Rename to RtpNackStats or RtcpNackStats (and rename the file too), now when it's
no longer under the RTPUtility namespace.

[danilchap, 2017/02/09 12:28:16]

Done.

+ public:
+  NackStats();
+
+  // Updates stats with requested sequence number.
+  // This function should be called for each NACK request to calculate the
+  // number of unique NACKed RTP packets.
+  void ReportRequest(uint16_t sequence_number);
+
+  // Gets the number of NACKed RTP packets.
+  uint32_t requests() const { return requests_; }
+
+  // Gets the number of unique NACKed RTP packets.
+  uint32_t unique_requests() const { return unique_requests_; }
+
+ private:
+  uint16_t max_sequence_number_;
+  uint32_t requests_;
+  uint32_t unique_requests_;

[nisse-webrtc, 2017/02/09 12:16:52]

You could consider using a wider type than uint32_t. But uint32_t is consistent
with use in RtcpPacketTypeCounter (in common_types.h).

[danilchap, 2017/02/09 12:28:16]

I consciously left the implementation of this class unchanged since it is not
subject of this CL. (otherwise would change this counters to size_t to follow
chromium guidelines)

[nisse-webrtc, 2017/02/09 12:34:05]

Acknowledged.

+};
+
+}  // namespace webrtc
+#endif  // WEBRTC_MODULES_RTP_RTCP_SOURCE_NACK_STATS_H_