zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add ',' between elements in RTCStatsReport::ToJson

Browse Source 
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}
This commit is contained in:
Dirk-Jan C. Binnema
2019-01-07 11:41:05 +00:00
committed by Commit Bot
parent 03fbf1eb4b
commit 8905d04109
2 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
AUTHORS
View File

@ -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>

1
stats/rtcstatsreport.cc
View File

@ -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();