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

Unified Diff: webrtc/api/rtpsender.h

Issue 1827023002: Get VideoCapturer stats via VideoTrackSourceInterface in StatsCollector, (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Move StatsReport logic to VideoRtpSender. Created 4 years, 9 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
Index: webrtc/api/rtpsender.h
diff --git a/webrtc/api/rtpsender.h b/webrtc/api/rtpsender.h
index 879a332209b3bdef484da7094178005b36286107..dd8cbac67272aa3abe87a140bfcda895ea4fd372 100644
--- a/webrtc/api/rtpsender.h
+++ b/webrtc/api/rtpsender.h
@@ -100,6 +100,12 @@ class AudioRtpSender : public ObserverInterface,
RtpParameters GetParameters() const;
bool SetParameters(const RtpParameters& parameters);
+ StatsReport::Id stats_id() override {
+ // Effectively nullptr.
+ return StatsReport::Id();
+ }
+ void ReportStats(StatsReport* report) override{};
perkj_webrtc 2016/03/29 11:03:39 GetStats ?
tommi 2016/03/30 11:46:44 void ReportStats(StatsReport* report) override {}
nisse-webrtc 2016/03/30 12:09:31 Fixed. For some reason, git cl format wants to re
+
private:
bool can_send_track() const { return track_ && ssrc_; }
// Helper function to construct options for
@@ -164,6 +170,9 @@ class VideoRtpSender : public ObserverInterface,
RtpParameters GetParameters() const;
bool SetParameters(const RtpParameters& parameters);
+ StatsReport::Id stats_id() override;
+ void ReportStats(StatsReport* report) override;
+
private:
bool can_send_track() const { return track_ && ssrc_; }
// Helper function to construct options for

Powered by Google App Engine
This is Rietveld 408576698