From 8905d04109d9844a70da8ba478edebc35d686d67 Mon Sep 17 00:00:00 2001
From: "Dirk-Jan C. Binnema" <djcb@djcbsoftware.nl>
Date: Mon, 7 Jan 2019 11:41:05 +0000
Subject: [PATCH] Add ',' between elements in RTCStatsReport::ToJson

There was no ',' between array-elements the stats-report, yielding
invalid JSON. Let's bring back the ",", which was lost during recent
refactoring.

Bug: webrtc:10173
Change-Id: Ib58025d56c4895c6af33b9777cb2ebdb94a678ea
Reviewed-on: https://webrtc-review.googlesource.com/c/116483
Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Commit-Queue: Oleh Prypin <oprypin@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#26161}
---
 AUTHORS                 | 1 +
 stats/rtcstatsreport.cc | 1 +
 2 files changed, 2 insertions(+)

diff --git a/AUTHORS b/AUTHORS
index 1216b48d95..dc7932661d 100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -17,6 +17,7 @@ Colin Plumb
 David Porter <david@porter.me>
 Dax Booysen <dax@younow.com>
 Danail Kirov <dkirovbroadsoft@gmail.com>
+Dirk-Jan C. Binnema <djcb@djcbsoftware.nl>
 Dmitry Lizin <sdkdimon@gmail.com>
 Eric Rescorla, RTFM Inc. <ekr@rtfm.com>
 Frederik Riedel, Frogg GmbH <frederik.riedel@frogg.io>
diff --git a/stats/rtcstatsreport.cc b/stats/rtcstatsreport.cc
index a62a97a36d..82db8f60d0 100644
--- a/stats/rtcstatsreport.cc
+++ b/stats/rtcstatsreport.cc
@@ -127,6 +127,7 @@ std::string RTCStatsReport::ToJson() const {
   const char* separator = "";
   for (ConstIterator it = begin(); it != end(); ++it) {
     sb << separator << it->ToJson();
+    separator = ",";
   }
   sb << "]";
   return sb.Release();