add mediaType field to ssrc stat

talk/app/webrtc/statscollector.cc

 /*
  * libjingle
  * Copyright 2012 Google Inc.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are met:
  *
  *  1. Redistributions of source code must retain the above copyright notice,
  *     this list of conditions and the following disclaimer.
  *  2. Redistributions in binary form must reproduce the above copyright notice,
@@ skipping 157 matching lines @@
   for (const auto& f : floats)
     report->AddFloat(f.name, f.value);
 
   for (const auto& i : ints)
     report->AddInt(i.name, i.value);
 
   report->AddInt64(StatsReport::kStatsValueNameBytesReceived,
                    info.bytes_rcvd);
   report->AddInt64(StatsReport::kStatsValueNameCaptureStartNtpTimeMs,
                    info.capture_start_ntp_time_ms);
+  report->AddString(StatsReport::kStatsValueNameMediaType, "audio");

[tommi, 2016/01/11 12:22:50]

Can you move this to the top of the function?

If you could also add a TODO for me:
// TODO(tommi): Look into setting the media type of a report when the report is
first created.

Same below.  The reason is that now we're setting this in several places while
it should be in only one place. Secondly we're setting it each time the report
is updated but this value should never change.

 }
 
 void ExtractStats(const cricket::VoiceSenderInfo& info, StatsReport* report) {
   ExtractCommonSendProperties(info, report);
 
   SetAudioProcessingStats(report, info.audio_level, info.typing_noise_detected,
       info.echo_return_loss, info.echo_return_loss_enhancement,
       info.echo_delay_median_ms, info.aec_quality_min, info.echo_delay_std_ms);
 
   const IntForAdd ints[] = {
     { StatsReport::kStatsValueNameJitterReceived, info.jitter_ms },
     { StatsReport::kStatsValueNamePacketsLost, info.packets_lost },
     { StatsReport::kStatsValueNamePacketsSent, info.packets_sent },
   };
 
   for (const auto& i : ints)
     report->AddInt(i.name, i.value);
+  report->AddString(StatsReport::kStatsValueNameMediaType, "audio");
 }
 
 void ExtractStats(const cricket::VideoReceiverInfo& info, StatsReport* report) {
   ExtractCommonReceiveProperties(info, report);
   report->AddInt64(StatsReport::kStatsValueNameBytesReceived,
                    info.bytes_rcvd);
   report->AddInt64(StatsReport::kStatsValueNameCaptureStartNtpTimeMs,
                    info.capture_start_ntp_time_ms);
   const IntForAdd ints[] = {
     { StatsReport::kStatsValueNameCurrentDelayMs, info.current_delay_ms },
@@ skipping 11 matching lines @@
     { StatsReport::kStatsValueNameNacksSent, info.nacks_sent },
     { StatsReport::kStatsValueNamePacketsLost, info.packets_lost },
     { StatsReport::kStatsValueNamePacketsReceived, info.packets_rcvd },
     { StatsReport::kStatsValueNamePlisSent, info.plis_sent },
     { StatsReport::kStatsValueNameRenderDelayMs, info.render_delay_ms },
     { StatsReport::kStatsValueNameTargetDelayMs, info.target_delay_ms },
   };
 
   for (const auto& i : ints)
     report->AddInt(i.name, i.value);
+  report->AddString(StatsReport::kStatsValueNameMediaType, "video");
 }
 
 void ExtractStats(const cricket::VideoSenderInfo& info, StatsReport* report) {
   ExtractCommonSendProperties(info, report);
 
   report->AddBoolean(StatsReport::kStatsValueNameBandwidthLimitedResolution,
                      (info.adapt_reason & 0x2) > 0);
   report->AddBoolean(StatsReport::kStatsValueNameCpuLimitedResolution,
                      (info.adapt_reason & 0x1) > 0);
   report->AddBoolean(StatsReport::kStatsValueNameViewLimitedResolution,
@@ skipping 12 matching lines @@
     { StatsReport::kStatsValueNameFrameWidthInput, info.input_frame_width },
     { StatsReport::kStatsValueNameFrameWidthSent, info.send_frame_width },
     { StatsReport::kStatsValueNameNacksReceived, info.nacks_rcvd },
     { StatsReport::kStatsValueNamePacketsLost, info.packets_lost },
     { StatsReport::kStatsValueNamePacketsSent, info.packets_sent },
     { StatsReport::kStatsValueNamePlisReceived, info.plis_rcvd },
   };
 
   for (const auto& i : ints)
     report->AddInt(i.name, i.value);
+  report->AddString(StatsReport::kStatsValueNameMediaType, "video");
 }
 
 void ExtractStats(const cricket::BandwidthEstimationInfo& info,
                   double stats_gathering_started,
                   PeerConnectionInterface::StatsOutputLevel level,
                   StatsReport* report) {
   DCHECK(report->type() == StatsReport::kStatsReportTypeBwe);
 
   report->set_timestamp(stats_gathering_started);
   const IntForAdd ints[] = {
@@ skipping 672 matching lines @@
     report->set_timestamp(stats_gathering_started_);
   }
 
 }
 
 void StatsCollector::ClearUpdateStatsCacheForTest() {
   stats_gathering_started_ = 0;
 }
 
 }  // namespace webrtc