Avoid precision loss in MedianSlopeEstimator from int64_t -> double conversion

Pass arrival time as an int64_t rather than a double to the MedianSlopeEstimator to avoid precision loss.

Also clean up the unit test.

BUG=webrtc:6892
Committed: https://crrev.com/ebcbcc3b2451f5c4fb07f7b37815bd54f364d057
Cr-Original-Commit-Position: refs/heads/master@{#15634}
Review-Url: https://codereview.webrtc.org/2578543002
Cr-Commit-Position: refs/heads/master@{#15650}
Committed: https://chromium.googlesource.com/external/webrtc/+/eb538fdcc6f92deabf5399b1eb24912cec4752a5

[terelius, 2016-12-14 13:30:25]

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

[commit-bot: I haz the power, 2016-12-14 13:30:31]

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, 2016-12-14 13:56:53]

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

[commit-bot: I haz the power, 2016-12-14 13:56:54]

Dry run: This issue passed the CQ dry run.

[terelius, 2016-12-14 14:23:55]

Description was changed from

==========
Pass arrival time as an int64_t rather than a double to the MedianSlopeFilter.

Also clean up the unit test.

BUG=webrtc:6892
==========

to

==========
Pass arrival time as an int64_t rather than a double to the
MedianSlopeEstimator.

Also clean up the unit test.

BUG=webrtc:6892
==========

[terelius, 2016-12-14 14:28:58]

Description was changed from

==========
Pass arrival time as an int64_t rather than a double to the
MedianSlopeEstimator.

Also clean up the unit test.

BUG=webrtc:6892
==========

to

==========
Pass arrival time as an int64_t rather than a double to the MedianSlopeEstimator
to avoid precision loss.

Also clean up the unit test.

BUG=webrtc:6892
==========

[terelius, 2016-12-14 14:29:34]

terelius@webrtc.org changed reviewers:
+ brandtr@webrtc.org, stefan@webrtc.org

[brandtr, 2016-12-15 12:25:35]

lgtm

https://codereview.webrtc.org/2578543002/diff/20001/webrtc/modules/congestion...
File webrtc/modules/congestion_controller/median_slope_estimator_unittest.cc
(right):

https://codereview.webrtc.org/2578543002/diff/20001/webrtc/modules/congestion...
webrtc/modules/congestion_controller/median_slope_estimator_unittest.cc:31: for
(size_t i = 0; i < kPacketCount; i++) {
++i

https://codereview.webrtc.org/2578543002/diff/20001/webrtc/modules/congestion...
webrtc/modules/congestion_controller/median_slope_estimator_unittest.cc:37: for
(size_t i = 1; i < kPacketCount; i++) {
..

[terelius, 2016-12-15 13:42:17]

https://codereview.webrtc.org/2578543002/diff/20001/webrtc/modules/congestion...
File webrtc/modules/congestion_controller/median_slope_estimator_unittest.cc
(right):

https://codereview.webrtc.org/2578543002/diff/20001/webrtc/modules/congestion...
webrtc/modules/congestion_controller/median_slope_estimator_unittest.cc:31: for
(size_t i = 0; i < kPacketCount; i++) {
On 2016/12/15 12:25:35, brandtr wrote:
> ++i

Done.

https://codereview.webrtc.org/2578543002/diff/20001/webrtc/modules/congestion...
webrtc/modules/congestion_controller/median_slope_estimator_unittest.cc:37: for
(size_t i = 1; i < kPacketCount; i++) {
On 2016/12/15 12:25:35, brandtr wrote:
> ..

Done.

[stefan-webrtc, 2016-12-15 13:44:54]

lgtm

[terelius, 2016-12-15 14:03:16]

The CQ bit was checked by terelius@webrtc.org

[terelius, 2016-12-15 14:03:16]

The patchset sent to the CQ was uploaded after l-g-t-m from brandtr@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2578543002/#ps40001 (title:
"Nits")

[commit-bot: I haz the power, 2016-12-15 14:03:23]

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-12-15 14:31:19]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1481810596424040,
"parent_rev": "058fb7eca0d8128cdd9913b06ffe5bb36a719447", "commit_rev":
"6863ea7ee8f125baf280da04d8c14990c958652d"}

[commit-bot: I haz the power, 2016-12-15 14:31:21]

Description was changed from

==========
Pass arrival time as an int64_t rather than a double to the MedianSlopeEstimator
to avoid precision loss.

Also clean up the unit test.

BUG=webrtc:6892
==========

to

==========
Pass arrival time as an int64_t rather than a double to the MedianSlopeEstimator
to avoid precision loss.

Also clean up the unit test.

BUG=webrtc:6892

Review-Url: https://codereview.webrtc.org/2578543002
==========

[commit-bot: I haz the power, 2016-12-15 14:31:21]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-12-15 14:31:30]

Description was changed from

==========
Pass arrival time as an int64_t rather than a double to the MedianSlopeEstimator
to avoid precision loss.

Also clean up the unit test.

BUG=webrtc:6892

Review-Url: https://codereview.webrtc.org/2578543002
==========

to

==========
Pass arrival time as an int64_t rather than a double to the MedianSlopeEstimator
to avoid precision loss.

Also clean up the unit test.

BUG=webrtc:6892

Committed: https://crrev.com/ebcbcc3b2451f5c4fb07f7b37815bd54f364d057
Cr-Commit-Position: refs/heads/master@{#15634}
==========

[commit-bot: I haz the power, 2016-12-15 14:31:31]

Patchset 3 (id:??) landed as
https://crrev.com/ebcbcc3b2451f5c4fb07f7b37815bd54f364d057
Cr-Commit-Position: refs/heads/master@{#15634}

[terelius, 2016-12-15 14:41:28]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.webrtc.org/2572353003/ by terelius@webrtc.org.

The reason for reverting is: Multiple definitions of TestEstimator.

[terelius, 2016-12-15 14:51:07]

Description was changed from

==========
Pass arrival time as an int64_t rather than a double to the MedianSlopeEstimator
to avoid precision loss.

Also clean up the unit test.

BUG=webrtc:6892

Committed: https://crrev.com/ebcbcc3b2451f5c4fb07f7b37815bd54f364d057
Cr-Commit-Position: refs/heads/master@{#15634}
==========

to

==========
Pass arrival time as an int64_t rather than a double to the MedianSlopeEstimator
to avoid precision loss.

Also clean up the unit test.

BUG=webrtc:6892

Committed: https://crrev.com/ebcbcc3b2451f5c4fb07f7b37815bd54f364d057
Cr-Commit-Position: refs/heads/master@{#15634}
==========

[terelius, 2016-12-16 10:16:51]

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

[commit-bot: I haz the power, 2016-12-16 10:16:55]

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, 2016-12-16 10:29:45]

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

[commit-bot: I haz the power, 2016-12-16 10:29:46]

Dry run: This issue passed the CQ dry run.

[terelius, 2016-12-16 10:35:32]

The CQ bit was checked by terelius@webrtc.org

[terelius, 2016-12-16 10:35:32]

The patchset sent to the CQ was uploaded after l-g-t-m from brandtr@webrtc.org,
stefan@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2578543002/#ps60001 (title:
"Move test function to anonymous namespace")

[commit-bot: I haz the power, 2016-12-16 10:35:35]

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-12-16 10:37:11]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1481884532136610,
"parent_rev": "beafee3009ac5f17e9e18d9452c3ae8d29ca92b0", "commit_rev":
"eb538fdcc6f92deabf5399b1eb24912cec4752a5"}

[commit-bot: I haz the power, 2016-12-16 10:37:14]

Description was changed from

==========
Pass arrival time as an int64_t rather than a double to the MedianSlopeEstimator
to avoid precision loss.

Also clean up the unit test.

BUG=webrtc:6892

Committed: https://crrev.com/ebcbcc3b2451f5c4fb07f7b37815bd54f364d057
Cr-Commit-Position: refs/heads/master@{#15634}
==========

to

==========
Pass arrival time as an int64_t rather than a double to the MedianSlopeEstimator
to avoid precision loss.

Also clean up the unit test.

BUG=webrtc:6892

Committed: https://crrev.com/ebcbcc3b2451f5c4fb07f7b37815bd54f364d057
Cr-Original-Commit-Position: refs/heads/master@{#15634}
Review-Url: https://codereview.webrtc.org/2578543002
Cr-Commit-Position: refs/heads/master@{#15650}
Committed:
https://chromium.googlesource.com/external/webrtc/+/eb538fdcc6f92deabf5399b1e...
==========

[commit-bot: I haz the power, 2016-12-16 10:37:15]

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