Change reporting of timing frames conditions in GetStats on receive side

webrtc/media/engine/webrtcvideoengine.cc

Index: webrtc/media/engine/webrtcvideoengine.cc
diff --git a/webrtc/media/engine/webrtcvideoengine.cc b/webrtc/media/engine/webrtcvideoengine.cc
index 274b801625d7730930aaef47bfa1154e14f04090..8366b64ed167f03e2d50e40710699f87d316f016 100644
--- a/webrtc/media/engine/webrtcvideoengine.cc
+++ b/webrtc/media/engine/webrtcvideoengine.cc
@@ -2393,7 +2393,7 @@ WebRtcVideoChannel::WebRtcVideoReceiveStream::GetVideoReceiverInfo(
   info.plis_sent = stats.rtcp_packet_type_counts.pli_packets;
   info.nacks_sent = stats.rtcp_packet_type_counts.nack_packets;
 
-  info.timing_frame_info = stream_->GetAndResetTimingFrameInfo();
+  info.timing_frame_info = stats.timing_frame_info;

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

std::move?

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

From my understanding it will do nothing.
TimingFrameInfo doesn't have any move constructors and Optional<> doesn't use
pointers, so there'll be no speed-up here too.

 
   if (log_stats)
     LOG(LS_INFO) << stats.ToString(rtc::TimeMillis());