Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(142)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: webrtc/api/rtcstatscollector.cc

Issue 2607933002: RTCMediaStreamTrackStats.framesDropped collected by RTCStatsCollector. (Closed)
Patch Set: Added unittests for webrtcvideoengine2 and receive_statistics_proxy Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webrtc/api/rtcstats_integrationtest.cc ('k') | webrtc/api/rtcstatscollector_unittest.cc » ('j') | webrtc/media/engine/webrtcvideoengine2_unittest.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/api/rtcstatscollector.cc
diff --git a/webrtc/api/rtcstatscollector.cc b/webrtc/api/rtcstatscollector.cc
index 5d782736a37f62604f9013569ed68e4a447fa984..2020c014828c2a059628caca9a393613d440b3fa 100644
--- a/webrtc/api/rtcstatscollector.cc
+++ b/webrtc/api/rtcstatscollector.cc
@@ -437,6 +437,10 @@ ProduceMediaStreamTrackStatsFromVideoReceiverInfo(
video_receiver_info.frame_height);
}
video_track_stats->frames_received = video_receiver_info.frames_received;
+ RTC_DCHECK_GE(video_receiver_info.frames_received,
+ video_receiver_info.frames_rendered);
+ video_track_stats->frames_dropped = video_receiver_info.frames_received -
+ video_receiver_info.frames_rendered;
return video_track_stats;
}
« no previous file with comments | « webrtc/api/rtcstats_integrationtest.cc ('k') | webrtc/api/rtcstatscollector_unittest.cc » ('j') | webrtc/media/engine/webrtcvideoengine2_unittest.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698