Get VideoCapturer stats via VideoTrackSourceInterface in StatsCollector,

webrtc/api/rtpsender.cc

Index: webrtc/api/rtpsender.cc
diff --git a/webrtc/api/rtpsender.cc b/webrtc/api/rtpsender.cc
index a7537867e26f2bb4104e64f4350d52eb7b6bca3f..f8ba8b09d13f3cece6f0e73828aa0bc65fbe3c92 100644
--- a/webrtc/api/rtpsender.cc
+++ b/webrtc/api/rtpsender.cc
@@ -346,4 +346,22 @@ bool VideoRtpSender::SetParameters(const RtpParameters& parameters) {
   return provider_->SetVideoRtpParameters(ssrc_, parameters);
 }
 
+StatsReport::Id VideoRtpSender::stats_id() {
+  return StatsReport::NewIdWithDirection(StatsReport::kStatsReportTypeSsrc,
+                                         rtc::ToString<uint32_t>(ssrc_),
+                                         StatsReport::kSend);
+}
+
+void VideoRtpSender::ReportStats(StatsReport* report) {
+  if (track_) {

[tommi, 2016/03/30 11:46:44]

early returns are preferred.  i.e.:

if (!track_)
  return;

...

[nisse-webrtc, 2016/03/30 12:09:31]

Done.

+    VideoTrackSourceInterface::Stats stats;

[perkj_webrtc, 2016/03/29 11:03:39]

Why not skip  VideoTrackSourceInterface::Stats all together and call
GetSource->GetStats(report);

[nisse-webrtc, 2016/03/29 11:37:31]

I don't quite like adding dependencies to StatsReport all over the code, it
seems nicer to fill out a plain struct with fixed fields. But if you think
that's the right direction, I can do it.

+    if (track_->GetSource()->GetStats(&stats)) {
+      report->AddInt(StatsReport::kStatsValueNameFrameWidthInput,
+                     stats.input_width);
+      report->AddInt(StatsReport::kStatsValueNameFrameHeightInput,
+                     stats.input_height);
+    }
+  }
+}
+
 }  // namespace webrtc