Renamed fields in common_types.h/RtcpStatistics.

Renamed fields in common_types.h/RtcpStatistics.

BUG=webrtc:8033
Review-Url: https://codereview.webrtc.org/2992043002
Cr-Commit-Position: refs/heads/master@{#19247}
Committed: https://chromium.googlesource.com/external/webrtc/+/186d9c38733a5e4151be7df55a1220a8c47c8bf4

[srte, 2017-07-31 15:36:53]

Description was changed from

==========
Renamed fields in common_types.h/RtcpStatistics.

BUG=8033
==========

to

==========
Renamed fields in common_types.h/RtcpStatistics.

BUG=8033
==========

[srte, 2017-07-31 15:36:53]

srte@webrtc.org changed reviewers:
+ asapersson@webrtc.org, kwiberg@webrtc.org, sprang@webrtc.org

[kwiberg-webrtc, 2017-08-01 08:13:37]

Description was changed from

==========
Renamed fields in common_types.h/RtcpStatistics.

BUG=8033
==========

to

==========
Renamed fields in common_types.h/RtcpStatistics.

BUG=webrtc:8033
==========

[kwiberg-webrtc, 2017-08-01 08:26:51]

lgtm for the mechanics of the rename. I'm not in a position to verify whether
the new names are better or not.

https://codereview.webrtc.org/2992043002/diff/1/webrtc/common_types.h
File webrtc/common_types.h (right):

https://codereview.webrtc.org/2992043002/diff/1/webrtc/common_types.h#newcode168
webrtc/common_types.h:168: };
Hmm, interesting. This actually appears to *not* be undefined behavior, because
the union members are standard-layout types whose common subsequence covers all
of them. I'll have to remember this trick. :-)

[åsapersson, 2017-08-01 15:23:20]

lgtm

[sprang_webrtc, 2017-08-02 08:24:59]

lgtm
make sure you test with downstream projects though

[srte, 2017-08-04 09:45:07]

The CQ bit was checked by srte@webrtc.org

[srte, 2017-08-04 09:45:07]

The patchset sent to the CQ was uploaded after l-g-t-m from
asapersson@webrtc.org, sprang@webrtc.org, kwiberg@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2992043002/#ps20001 (title:
"Rebase on new master")

[commit-bot: I haz the power, 2017-08-04 09:45:19]

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-08-04 12:03:54]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1501839907393100,
"parent_rev": "463d7ccb36c130274c75b8f164615acaa3baeec7", "commit_rev":
"186d9c38733a5e4151be7df55a1220a8c47c8bf4"}

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

Description was changed from

==========
Renamed fields in common_types.h/RtcpStatistics.

BUG=webrtc:8033
==========

to

==========
Renamed fields in common_types.h/RtcpStatistics.

BUG=webrtc:8033

Review-Url: https://codereview.webrtc.org/2992043002
Cr-Commit-Position: refs/heads/master@{#19247}
Committed:
https://chromium.googlesource.com/external/webrtc/+/186d9c38733a5e4151be7df55...
==========

[commit-bot: I haz the power, 2017-08-04 12:04:03]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/webrtc/+/186d9c38733a5e4151be7df55...