RTCIceCandidateStats added.

webrtc/api/rtcstatscollector.h

Index: webrtc/api/rtcstatscollector.h
diff --git a/webrtc/api/rtcstatscollector.h b/webrtc/api/rtcstatscollector.h
index 2f5ed8610a6399fc57ea59bd085bcb9beadd08ac..654bca96b7addcd3ed86172e2e6b293951ea2404 100644
--- a/webrtc/api/rtcstatscollector.h
+++ b/webrtc/api/rtcstatscollector.h
@@ -21,10 +21,12 @@
 #include "webrtc/base/scoped_ref_ptr.h"
 #include "webrtc/base/timeutils.h"
 
-namespace rtc {
+namespace cricket {
+class Candidate;
+}  // namespace cricket
 
+namespace rtc {
 class SSLCertificate;
-
 }  // namespace rtc
 
 namespace webrtc {
@@ -76,12 +78,22 @@ class RTCStatsCollector : public virtual rtc::RefCountInterface {
   void AddPartialResults_s(rtc::scoped_refptr<RTCStatsReport> partial_report);
   void DeliverCachedReport();
 
+  // RTCCertificateStats
   void ProduceCertificateStats_s(
       int64_t timestamp_us, const SessionStats& session_stats,
       RTCStatsReport* report) const;
   void ProduceCertificateStatsFromSSLCertificateAndChain_s(
       int64_t timestamp_us, const rtc::SSLCertificate& certificate,
       RTCStatsReport* report) const;
+  // RTCIceCandidateStats

[hta-webrtc, 2016/10/04 14:12:45]

Try to make comments full sentences. If the comment says no more than the
function name, consider omitting it.

"This function produces both RTCIceCandidateStats and RTCIceCandidatePairStats."
is a fine summary.

[hbos, 2016/10/05 10:16:30]

Done.

+  // RTCIceCandidatePairStats (todo)
+  void ProduceIceCandidateAndPairStats_s(
+      int64_t timestamp_us, const SessionStats& session_stats,
+      RTCStatsReport* report) const;
+  const std::string& ProduceIceCandidateStats_s(
+      int64_t timestamp_us, const cricket::Candidate& candidate, bool is_local,
+      RTCStatsReport* report) const;
+  // RTCPeerConnectionStats
   void ProducePeerConnectionStats_s(
       int64_t timestamp_us, RTCStatsReport* report) const;
 
@@ -105,6 +117,8 @@ class RTCStatsCollector : public virtual rtc::RefCountInterface {
   rtc::scoped_refptr<const RTCStatsReport> cached_report_;
 };
 
+const char* CandidateTypeToRTCIceCandidateType(const std::string& type);

[hta-webrtc, 2016/10/04 14:12:45]

If this function is only exposed for testing, add a comment saying so.

[hbos, 2016/10/05 10:16:30]

Done.

+
 }  // namespace webrtc
 
 #endif  // WEBRTC_API_RTCSTATSCOLLECTOR_H_