webrtc-internals: use spec event values

webrtc-internals: use spec event values

This maps the internal names of the values in the
iceConnectionStateChange, iceGatheringStateChange and signalingStateChange
to their equivalents defined in the specification.

This is only done before displaying the values in order not to change the
format of the JSON dump.

BUG=678292
Committed: https://crrev.com/5934544dd649e157b0991d4ae6c295faf9c5d99d
Cr-Commit-Position: refs/heads/master@{#441345}

[fippo, 2016-12-23 20:28:54]

Description was changed from

==========
webrtc-internals: use spec event values

This maps the internal names of the values in the
iceConnectionStateChange, iceGatheringStateChange and signalingStateChange
to their equivalents defined in the specification.

This is only done before displaying the values in order not to change the
format of the JSON dump.

BUG=
==========

to

==========
webrtc-internals: use spec event values

This maps the internal names of the values in the
iceConnectionStateChange, iceGatheringStateChange and signalingStateChange
to their equivalents defined in the specification.

This is only done before displaying the values in order not to change the
format of the JSON dump.

BUG=
==========

[fippo, 2016-12-23 20:28:54]

philipp.hancke@googlemail.com changed reviewers:
+ hta@chromium.org, tommi@chromium.org

[hta - Chromium, 2017-01-03 10:42:32]

lgtm

Is this for what's displayed in webrtc-internals?

If I read it correctly, it will pass through any values not listed in the tables
unchanged, so when the JSON-generating functions switch to generating
spec-compliant strings, this code will Just Work.

[fippo, 2017-01-03 13:40:51]

On 2017/01/03 10:42:32, hta - Chromium wrote:
> lgtm
> 
> Is this for what's displayed in webrtc-internals?

Yes, it touches just the "api trace" output that is on the left.
I didn't want to change the output format of the dump since there might be
analysis code relying on that.

> If I read it correctly, it will pass through any values not listed in the
tables
> unchanged, so when the JSON-generating functions switch to generating
> spec-compliant strings, this code will Just Work.

That is the plan. The left side of the mappings comes from
https://cs.chromium.org/chromium/src/third_party/WebKit/public/platform/WebRT...
btw.

[tommi (sloooow) - chröme, 2017-01-04 08:59:54]

The CQ bit was checked by tommi@chromium.org

[tommi (sloooow) - chröme, 2017-01-04 08:59:55]

lgtm

[commit-bot: I haz the power, 2017-01-04 09:00:07]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-04 09:08:48]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1483520394715570,
"parent_rev": "c506ec4be09b2bf115066921cd66b97a06c641b6", "commit_rev":
"78a0b1d240306d966030eb8fbc582d91334383de"}

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

Description was changed from

==========
webrtc-internals: use spec event values

This maps the internal names of the values in the
iceConnectionStateChange, iceGatheringStateChange and signalingStateChange
to their equivalents defined in the specification.

This is only done before displaying the values in order not to change the
format of the JSON dump.

BUG=
==========

to

==========
webrtc-internals: use spec event values

This maps the internal names of the values in the
iceConnectionStateChange, iceGatheringStateChange and signalingStateChange
to their equivalents defined in the specification.

This is only done before displaying the values in order not to change the
format of the JSON dump.

BUG=

Review-Url: https://codereview.chromium.org/2602603002
==========

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

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2017-01-04 09:11:42]

Description was changed from

==========
webrtc-internals: use spec event values

This maps the internal names of the values in the
iceConnectionStateChange, iceGatheringStateChange and signalingStateChange
to their equivalents defined in the specification.

This is only done before displaying the values in order not to change the
format of the JSON dump.

BUG=

Review-Url: https://codereview.chromium.org/2602603002
==========

to

==========
webrtc-internals: use spec event values

This maps the internal names of the values in the
iceConnectionStateChange, iceGatheringStateChange and signalingStateChange
to their equivalents defined in the specification.

This is only done before displaying the values in order not to change the
format of the JSON dump.

BUG=

Committed: https://crrev.com/5934544dd649e157b0991d4ae6c295faf9c5d99d
Cr-Commit-Position: refs/heads/master@{#441345}
==========

[commit-bot: I haz the power, 2017-01-04 09:11:43]

Patchset 2 (id:??) landed as
https://crrev.com/5934544dd649e157b0991d4ae6c295faf9c5d99d
Cr-Commit-Position: refs/heads/master@{#441345}

[fippo, 2017-01-04 17:27:16]

Description was changed from

==========
webrtc-internals: use spec event values

This maps the internal names of the values in the
iceConnectionStateChange, iceGatheringStateChange and signalingStateChange
to their equivalents defined in the specification.

This is only done before displaying the values in order not to change the
format of the JSON dump.

BUG=

Committed: https://crrev.com/5934544dd649e157b0991d4ae6c295faf9c5d99d
Cr-Commit-Position: refs/heads/master@{#441345}
==========

to

==========
webrtc-internals: use spec event values

This maps the internal names of the values in the
iceConnectionStateChange, iceGatheringStateChange and signalingStateChange
to their equivalents defined in the specification.

This is only done before displaying the values in order not to change the
format of the JSON dump.

BUG=678292

Committed: https://crrev.com/5934544dd649e157b0991d4ae6c295faf9c5d99d
Cr-Commit-Position: refs/heads/master@{#441345}
==========