Revert "Revert of Wire up BWE stats through WebrtcSession so that they are filled in both for audio…

Revert "Revert of Wire up BWE stats through WebrtcSession so that they are filled in both for audio and video calls. (patchset #8 id:140001 of https://codereview.webrtc.org/2863123002/ )"

This reverts commit d72098a41971833e210bfdcffaab7a18ced4775f.

BUG=webrtc:5079
Review-Url: https://codereview.webrtc.org/2915263002
Cr-Commit-Position: refs/heads/master@{#18411}
Committed: https://chromium.googlesource.com/external/webrtc/+/f79ade1320d1f27dc85a1bf7da5ca430ca80ab85

[stefan-webrtc, 2017-06-02 08:02:07]

Re-add bw_estimations.

[stefan-webrtc, 2017-06-02 08:22:37]

stefan@webrtc.org changed reviewers:
+ hbos@webrtc.org

[stefan-webrtc, 2017-06-02 08:22:37]

Had to revert and reland with a small change due to an internal project. Could
you take a quick look?

[stefan-webrtc, 2017-06-02 08:41:54]

.

[hbos, 2017-06-02 10:37:51]

https://codereview.webrtc.org/2915263002/diff/60001/webrtc/media/base/mediach...
File webrtc/media/base/mediachannel.h (right):

https://codereview.webrtc.org/2915263002/diff/60001/webrtc/media/base/mediach...
webrtc/media/base/mediachannel.h:873: std::vector<BandwidthEstimationInfo>
bw_estimations;
You've still removed filling in the bw_estimations from
WebRtcVideoChannel2::GetStats, are upstream projects okay with that?

[holmer, 2017-06-02 11:34:00]

holmer@google.com changed reviewers:
+ holmer@google.com

[holmer, 2017-06-02 11:34:26]

https://codereview.webrtc.org/2915263002/diff/60001/webrtc/media/base/mediach...
File webrtc/media/base/mediachannel.h (right):

https://codereview.webrtc.org/2915263002/diff/60001/webrtc/media/base/mediach...
webrtc/media/base/mediachannel.h:873: std::vector<BandwidthEstimationInfo>
bw_estimations;
On 2017/06/02 10:37:51, hbos wrote:
> You've still removed filling in the bw_estimations from
> WebRtcVideoChannel2::GetStats, are upstream projects okay with that?

Yes, that should be fine according to Noah.

[hbos, 2017-06-02 11:35:21]

lgtm

[stefan-webrtc, 2017-06-02 13:21:39]

The CQ bit was checked by stefan@webrtc.org

[commit-bot: I haz the power, 2017-06-02 13:21:44]

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-06-02 13:44:04]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1496409699868690,
"parent_rev": "2047db5f0d864d0e82b69ac325562b9d99bec3cf", "commit_rev":
"f79ade1320d1f27dc85a1bf7da5ca430ca80ab85"}

[commit-bot: I haz the power, 2017-06-02 13:44:07]

Description was changed from

==========
Revert "Revert of Wire up BWE stats through WebrtcSession so that they are
filled in both for audio and video calls. (patchset #8 id:140001 of
https://codereview.webrtc.org/2863123002/ )"

This reverts commit d72098a41971833e210bfdcffaab7a18ced4775f.

BUG=webrtc:5079
==========

to

==========
Revert "Revert of Wire up BWE stats through WebrtcSession so that they are
filled in both for audio and video calls. (patchset #8 id:140001 of
https://codereview.webrtc.org/2863123002/ )"

This reverts commit d72098a41971833e210bfdcffaab7a18ced4775f.

BUG=webrtc:5079

Review-Url: https://codereview.webrtc.org/2915263002
Cr-Commit-Position: refs/heads/master@{#18411}
Committed:
https://chromium.googlesource.com/external/webrtc/+/f79ade1320d1f27dc85a1bf7d...
==========

[commit-bot: I haz the power, 2017-06-02 13:44:08]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/external/webrtc/+/f79ade1320d1f27dc85a1bf7d...