Get VideoCapturer stats via VideoTrackSourceInterface in StatsCollector,

webrtc/api/rtpsender.cc

 /*
  *  Copyright 2015 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 328 matching lines @@
 }
 
 RtpParameters VideoRtpSender::GetParameters() const {
   return provider_->GetVideoRtpParameters(ssrc_);
 }
 
 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