Reland of Trace the stats report as JSON instead of each stat separately.

Reland of Trace the stats report as JSON instead of each stat separately. (patchset #1 id:1 of https://codereview.webrtc.org/3001683002/ )

Reason for revert:
Fixed

Original issue's description:
> Revert of Trace the stats report as JSON instead of each stat separately. (patchset #3 id:100001 of https://codereview.webrtc.org/2986453002/ )
> 
> Reason for revert:
> It breaks a downstream project.
> 
> Original issue's description:
> > Trace the stats report as JSON instead of each stat separately.
> > 
> > Trace the whole report as a string instead of each field on it's own. And test that the traces collected are valid.
> > 
> > R=tommi@webrtc.org, hbos@webrtc.org
> > BUG=chromium:653087
> > 
> > Review-Url: https://codereview.webrtc.org/2986453002
> > Cr-Commit-Position: refs/heads/master@{#19341}
> > Committed: https://chromium.googlesource.com/external/webrtc/+/80c65780e618f90369295fe5daf8075e9b374c26
> 
> TBR=hbos@webrtc.org,tommi@webrtc.org,ehmaldonado@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=chromium:653087
> 
> Review-Url: https://codereview.webrtc.org/3001683002
> Cr-Commit-Position: refs/heads/master@{#19344}
> Committed: https://chromium.googlesource.com/external/webrtc/+/3439c893585e0958502ef397056a372be1777688

BUG=chromium:653087
Review-Url: https://codereview.webrtc.org/3000943002
Cr-Commit-Position: refs/heads/master@{#19673}
Committed: https://chromium.googlesource.com/external/webrtc/+/8ab0fd80ad4af302d8f72e94e577458084772288

[ehmaldonado_webrtc, 2017-08-15 10:41:22]

Created Reland of Trace the stats report as JSON instead of each stat
separately.

[ehmaldonado_webrtc, 2017-08-15 10:49:14]

Patchset #2 (id:70001) has been deleted

[tommi, 2017-08-15 13:47:06]

lgtm

[ehmaldonado_webrtc, 2017-08-15 13:51:20]

On 2017/08/15 13:47:06, tommi (ooo) wrote:
> lgtm

(Haven't changed anything yet, sorry :)

[tommi, 2017-08-15 13:54:15]

On 2017/08/15 13:51:20, ehmaldonado_webrtc wrote:
> On 2017/08/15 13:47:06, tommi (ooo) wrote:
> > lgtm
> 
> (Haven't changed anything yet, sorry :)

ah, in that case, not lgtm :)

I thought you were just going to reland. anyway, just ping back when I should
take a look

[ehmaldonado_webrtc, 2017-08-15 19:47:18]

Patchset #2 (id:90001) has been deleted

[ehmaldonado_webrtc, 2017-09-04 15:25:40]

The CQ bit was checked by ehmaldonado@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-09-04 15:25:50]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-09-04 15:25:51]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-09-04 15:25:52]

Dry run: This issue passed the CQ dry run.

[ehmaldonado_webrtc, 2017-09-04 15:34:01]

Description was changed from

==========
Reland of Trace the stats report as JSON instead of each stat separately.
(patchset #1 id:1 of https://codereview.webrtc.org/3001683002/ )

Reason for revert:
Fixed

Original issue's description:
> Revert of Trace the stats report as JSON instead of each stat separately.
(patchset #3 id:100001 of https://codereview.webrtc.org/2986453002/ )
> 
> Reason for revert:
> It breaks a downstream project.
> 
> Original issue's description:
> > Trace the stats report as JSON instead of each stat separately.
> > 
> > Trace the whole report as a string instead of each field on it's own. And
test that the traces collected are valid.
> > 
> > R=tommi@webrtc.org, hbos@webrtc.org
> > BUG=chromium:653087
> > 
> > Review-Url: https://codereview.webrtc.org/2986453002
> > Cr-Commit-Position: refs/heads/master@{#19341}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/80c65780e618f90369295fe5d...
> 
> TBR=hbos@webrtc.org,tommi@webrtc.org,ehmaldonado@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=chromium:653087
> 
> Review-Url: https://codereview.webrtc.org/3001683002
> Cr-Commit-Position: refs/heads/master@{#19344}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/3439c893585e0958502ef3970...

TBR=hbos@webrtc.org,tommi@webrtc.org,mbonadei@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:653087
==========

to

==========
Reland of Trace the stats report as JSON instead of each stat separately.
(patchset #1 id:1 of https://codereview.webrtc.org/3001683002/ )

Reason for revert:
Fixed

Original issue's description:
> Revert of Trace the stats report as JSON instead of each stat separately.
(patchset #3 id:100001 of https://codereview.webrtc.org/2986453002/ )
> 
> Reason for revert:
> It breaks a downstream project.
> 
> Original issue's description:
> > Trace the stats report as JSON instead of each stat separately.
> > 
> > Trace the whole report as a string instead of each field on it's own. And
test that the traces collected are valid.
> > 
> > R=tommi@webrtc.org, hbos@webrtc.org
> > BUG=chromium:653087
> > 
> > Review-Url: https://codereview.webrtc.org/2986453002
> > Cr-Commit-Position: refs/heads/master@{#19341}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/80c65780e618f90369295fe5d...
> 
> TBR=hbos@webrtc.org,tommi@webrtc.org,ehmaldonado@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=chromium:653087
> 
> Review-Url: https://codereview.webrtc.org/3001683002
> Cr-Commit-Position: refs/heads/master@{#19344}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/3439c893585e0958502ef3970...

BUG=chromium:653087
==========

[ehmaldonado_webrtc, 2017-09-04 15:34:08]

The CQ bit was checked by ehmaldonado@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-09-04 15:34:11]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-09-04 16:10:20]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-09-04 16:10:21]

Dry run: This issue passed the CQ dry run.

[ehmaldonado_webrtc, 2017-09-04 18:00:57]

On 2017/08/15 13:54:15, tommi wrote:
> On 2017/08/15 13:51:20, ehmaldonado_webrtc wrote:
> > On 2017/08/15 13:47:06, tommi (ooo) wrote:
> > > lgtm
> > 
> > (Haven't changed anything yet, sorry :)
> 
> ah, in that case, not lgtm :)
> 
> I thought you were just going to reland. anyway, just ping back when I should
> take a look

PTAL. The linux_internal failure seems to be unrelated.

[ehmaldonado_webrtc, 2017-09-04 20:19:16]

On 2017/09/04 18:00:57, ehmaldonado_webrtc wrote:
> On 2017/08/15 13:54:15, tommi wrote:
> > On 2017/08/15 13:51:20, ehmaldonado_webrtc wrote:
> > > On 2017/08/15 13:47:06, tommi (ooo) wrote:
> > > > lgtm
> > > 
> > > (Haven't changed anything yet, sorry :)
> > 
> > ah, in that case, not lgtm :)
> > 
> > I thought you were just going to reland. anyway, just ping back when I
should
> > take a look
> 
> PTAL. The linux_internal failure seems to be unrelated.

Re-triggered linux_internal and it works now :)

[tommi, 2017-09-04 21:04:32]

lgtm

[ehmaldonado_webrtc, 2017-09-04 21:32:52]

The CQ bit was checked by ehmaldonado@webrtc.org

[commit-bot: I haz the power, 2017-09-04 21:33:01]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-09-04 21:35:05]

CQ is committing da patch.

Bot data: {"patchset_id": 110001, "attempt_start_ts": 1504560772652050,
"parent_rev": "3eac8002db8fdf5124026d5377a1f3ec1d16bb0d", "commit_rev":
"8ab0fd80ad4af302d8f72e94e577458084772288"}

[commit-bot: I haz the power, 2017-09-04 21:35:10]

Description was changed from

==========
Reland of Trace the stats report as JSON instead of each stat separately.
(patchset #1 id:1 of https://codereview.webrtc.org/3001683002/ )

Reason for revert:
Fixed

Original issue's description:
> Revert of Trace the stats report as JSON instead of each stat separately.
(patchset #3 id:100001 of https://codereview.webrtc.org/2986453002/ )
> 
> Reason for revert:
> It breaks a downstream project.
> 
> Original issue's description:
> > Trace the stats report as JSON instead of each stat separately.
> > 
> > Trace the whole report as a string instead of each field on it's own. And
test that the traces collected are valid.
> > 
> > R=tommi@webrtc.org, hbos@webrtc.org
> > BUG=chromium:653087
> > 
> > Review-Url: https://codereview.webrtc.org/2986453002
> > Cr-Commit-Position: refs/heads/master@{#19341}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/80c65780e618f90369295fe5d...
> 
> TBR=hbos@webrtc.org,tommi@webrtc.org,ehmaldonado@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=chromium:653087
> 
> Review-Url: https://codereview.webrtc.org/3001683002
> Cr-Commit-Position: refs/heads/master@{#19344}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/3439c893585e0958502ef3970...

BUG=chromium:653087
==========

to

==========
Reland of Trace the stats report as JSON instead of each stat separately.
(patchset #1 id:1 of https://codereview.webrtc.org/3001683002/ )

Reason for revert:
Fixed

Original issue's description:
> Revert of Trace the stats report as JSON instead of each stat separately.
(patchset #3 id:100001 of https://codereview.webrtc.org/2986453002/ )
> 
> Reason for revert:
> It breaks a downstream project.
> 
> Original issue's description:
> > Trace the stats report as JSON instead of each stat separately.
> > 
> > Trace the whole report as a string instead of each field on it's own. And
test that the traces collected are valid.
> > 
> > R=tommi@webrtc.org, hbos@webrtc.org
> > BUG=chromium:653087
> > 
> > Review-Url: https://codereview.webrtc.org/2986453002
> > Cr-Commit-Position: refs/heads/master@{#19341}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/80c65780e618f90369295fe5d...
> 
> TBR=hbos@webrtc.org,tommi@webrtc.org,ehmaldonado@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=chromium:653087
> 
> Review-Url: https://codereview.webrtc.org/3001683002
> Cr-Commit-Position: refs/heads/master@{#19344}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/3439c893585e0958502ef3970...

BUG=chromium:653087

Review-Url: https://codereview.webrtc.org/3000943002
Cr-Commit-Position: refs/heads/master@{#19673}
Committed:
https://chromium.googlesource.com/external/webrtc/+/8ab0fd80ad4af302d8f72e94e...
==========

[commit-bot: I haz the power, 2017-09-04 21:35:11]

Committed patchset #2 (id:110001) as
https://chromium.googlesource.com/external/webrtc/+/8ab0fd80ad4af302d8f72e94e...