Return false if PeerConnection::GetStats() is called on invalid tracks

webrtc/api/statscollector.cc

Index: webrtc/api/statscollector.cc
diff --git a/webrtc/api/statscollector.cc b/webrtc/api/statscollector.cc
index a40835c77e257cac26fe7a0c5b8132c561b00972..27544afbc8c1f9c6148f75688f23818400ad38cf 100644
--- a/webrtc/api/statscollector.cc
+++ b/webrtc/api/statscollector.cc
@@ -549,6 +549,11 @@ StatsReport* StatsCollector::PrepareReport(
   return report;
 }
 
+bool StatsCollector::IsValidTrack(MediaStreamTrackInterface* track) {
+  return reports_.Find(StatsReport::NewTypedId(
+             StatsReport::kStatsReportTypeTrack, track->id())) != nullptr;

[Taylor Brandstetter, 2016/11/04 00:48:48]

nit: Since this only needs the track ID, the argument could just be "const
std::string& track_id". Then you wouldn't need to worry about checking if it's
null here.

+}
+
 StatsReport* StatsCollector::AddCertificateReports(
     const rtc::SSLCertificate* cert) {
   RTC_DCHECK(pc_->session()->signaling_thread()->IsCurrent());