Add unit tests to ReceiveStatisticsProxy class.

Add unit tests to ReceiveStatisticsProxy class.

BUG=none

Committed: https://crrev.com/46c4e3caa32f1d46f681455ae6e401a96d33b529
Cr-Commit-Position: refs/heads/master@{#14911}

[åsapersson, 2016-10-27 13:47:42]

Description was changed from

==========
Add unit tests to ReceiveStatisticsProxy class.

BUG=none
==========

to

==========
Add unit tests to ReceiveStatisticsProxy class.

BUG=none
==========

[åsapersson, 2016-10-27 13:47:42]

asapersson@webrtc.org changed reviewers:
+ sprang@webrtc.org, stefan@webrtc.org

[stefan-webrtc, 2016-10-31 09:45:42]

https://codereview.webrtc.org/2450363003/diff/20001/webrtc/video/receive_stat...
File webrtc/video/receive_statistics_proxy_unittest.cc (right):

https://codereview.webrtc.org/2450363003/diff/20001/webrtc/video/receive_stat...
webrtc/video/receive_statistics_proxy_unittest.cc:106:
statistics_proxy_.reset();
Why reset here? Maybe comment on that.

[åsapersson, 2016-10-31 10:20:25]

https://codereview.webrtc.org/2450363003/diff/20001/webrtc/video/receive_stat...
File webrtc/video/receive_statistics_proxy_unittest.cc (right):

https://codereview.webrtc.org/2450363003/diff/20001/webrtc/video/receive_stat...
webrtc/video/receive_statistics_proxy_unittest.cc:106:
statistics_proxy_.reset();
On 2016/10/31 09:45:42, stefan-webrtc (holmer) wrote:
> Why reset here? Maybe comment on that.

Done.

[sprang_webrtc, 2016-10-31 15:34:38]

lgtm

[åsapersson, 2016-11-02 15:14:42]

Patchset #4 (id:60001) has been deleted

[stefan-webrtc, 2016-11-03 13:43:18]

lgtm

[åsapersson, 2016-11-03 13:46:57]

The CQ bit was checked by asapersson@webrtc.org

[åsapersson, 2016-11-03 13:46:57]

The patchset sent to the CQ was uploaded after l-g-t-m from sprang@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2450363003/#ps80001 (title:
"rebase")

[commit-bot: I haz the power, 2016-11-03 13:47:05]

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, 2016-11-03 13:48:21]

Description was changed from

==========
Add unit tests to ReceiveStatisticsProxy class.

BUG=none
==========

to

==========
Add unit tests to ReceiveStatisticsProxy class.

BUG=none
==========

[commit-bot: I haz the power, 2016-11-03 13:48:21]

Committed patchset #4 (id:80001)

[commit-bot: I haz the power, 2016-11-03 13:48:42]

Description was changed from

==========
Add unit tests to ReceiveStatisticsProxy class.

BUG=none
==========

to

==========
Add unit tests to ReceiveStatisticsProxy class.

BUG=none

Committed: https://crrev.com/46c4e3caa32f1d46f681455ae6e401a96d33b529
Cr-Commit-Position: refs/heads/master@{#14911}
==========

[commit-bot: I haz the power, 2016-11-03 13:48:43]

Patchset 4 (id:??) landed as
https://crrev.com/46c4e3caa32f1d46f681455ae6e401a96d33b529
Cr-Commit-Position: refs/heads/master@{#14911}