Change reporting of timing frames conditions in GetStats on receive side

webrtc/api/video/video_timing.cc

Index: webrtc/api/video/video_timing.cc
diff --git a/webrtc/api/video/video_timing.cc b/webrtc/api/video/video_timing.cc
index 0d22f904bf0f8cac635afbd934415d38fcfb3189..6de68c40957873d95ad7bafc1d9d3a8a3ef2ca87 100644
--- a/webrtc/api/video/video_timing.cc
+++ b/webrtc/api/video/video_timing.cc
@@ -39,6 +39,15 @@ bool TimingFrameInfo::IsLongerThan(const TimingFrameInfo& other) const {
   return other_delay == -1 || EndToEndDelay() > other_delay;
 }
 
+bool TimingFrameInfo::operator<(const TimingFrameInfo& other) {
+  return other.IsLongerThan(*this);
+}
+
+bool TimingFrameInfo::operator<=(const TimingFrameInfo& other) {
+  return !IsLongerThan(other);
+}
+

[tommi, 2017/09/04 09:09:21]

nit: fix whitespace

[ilnik, 2017/09/04 09:18:55]

Done.

+
 bool TimingFrameInfo::IsOutlier() const {
   return !IsInvalid() && (flags & TimingFrameFlags::kTriggeredBySize);
 }
@@ -54,15 +63,15 @@ bool TimingFrameInfo::IsInvalid() const {
 std::string TimingFrameInfo::ToString() const {
   std::stringstream out;
   if (IsInvalid()) {
-    out << "[Invalid]";
+    out << "";
   } else {
     out << rtp_timestamp << ',' << capture_time_ms << ',' << encode_start_ms
         << ',' << encode_finish_ms << ',' << packetization_finish_ms << ','
         << pacer_exit_ms << ',' << network_timestamp_ms << ','
         << network2_timestamp_ms << ',' << receive_start_ms << ','
         << receive_finish_ms << ',' << decode_start_ms << ','
-        << decode_finish_ms << ',' << render_time_ms << ", outlier_triggered "
-        << IsOutlier() << ", timer_triggered " << IsTimerTriggered();
+        << decode_finish_ms << ',' << render_time_ms << ','
+        << IsOutlier() << ',' << IsTimerTriggered();
   }
   return out.str();
 }