Provide default impl of TimeUntilNextProcess in MockRtpRtcp instead of mocking.

Provide default impl of TimeUntilNextProcess in MockRtpRtcp instead of mocking.

This method isn't currently mocked or required by any test, so the safe thing
is to return a reasonably large value from the implementation to avoid busy loops.

BUG=webrtc:7187
TBR=mflodman@webrtc.org

Review-Url: https://codereview.webrtc.org/2744233002
Cr-Commit-Position: refs/heads/master@{#17284}
Committed: https://chromium.googlesource.com/external/webrtc/+/994ebb47024e7bf7e65039384a48a9854011d283

[tommi, 2017-03-13 10:59:47]

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

[tommi, 2017-03-13 11:03:09]

Description was changed from

==========
Provide default impl of TimeUntilNextProcess in MockRtpRtcp instead of mocking.

This method isn't currently mocked or required by any test, so the safe thing
is to return a reasonably large value from the implementation to avoid busy
loops.

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

to

==========
Provide default impl of TimeUntilNextProcess in MockRtpRtcp instead of mocking.

This method isn't currently mocked or required by any test, so the safe thing
is to return a reasonably large value from the implementation to avoid busy
loops.

BUG=webrtc:7187
==========

[tommi, 2017-03-13 20:59:11]

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

[commit-bot: I haz the power, 2017-03-13 20:59:34]

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, 2017-03-13 21:35:15]

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

[commit-bot: I haz the power, 2017-03-13 21:35:16]

Dry run: This issue passed the CQ dry run.

[tommi, 2017-03-14 18:45:16]

tommi@webrtc.org changed reviewers:
+ mflodman@webrtc.org
- stefan@webrtc.org

[tommi, 2017-03-14 18:45:27]

magnus - can you take a look?

[tommi, 2017-03-17 10:31:41]

Description was changed from

==========
Provide default impl of TimeUntilNextProcess in MockRtpRtcp instead of mocking.

This method isn't currently mocked or required by any test, so the safe thing
is to return a reasonably large value from the implementation to avoid busy
loops.

BUG=webrtc:7187
==========

to

==========
Provide default impl of TimeUntilNextProcess in MockRtpRtcp instead of mocking.

This method isn't currently mocked or required by any test, so the safe thing
is to return a reasonably large value from the implementation to avoid busy
loops.

BUG=webrtc:7187
TBR=mflodman@webrtc.org
==========

[tommi, 2017-03-17 10:32:06]

TBR-ing as there is no test handling this right now.

[tommi, 2017-03-17 10:32:12]

The CQ bit was checked by tommi@webrtc.org

[commit-bot: I haz the power, 2017-03-17 10:32:18]

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-03-17 11:01:31]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489746732162500, "parent_rev":
"90474ccd1255c6b44e7a7938fa4298795306c9b1", "commit_rev":
"994ebb47024e7bf7e65039384a48a9854011d283"}

[commit-bot: I haz the power, 2017-03-17 11:01:35]

Description was changed from

==========
Provide default impl of TimeUntilNextProcess in MockRtpRtcp instead of mocking.

This method isn't currently mocked or required by any test, so the safe thing
is to return a reasonably large value from the implementation to avoid busy
loops.

BUG=webrtc:7187
TBR=mflodman@webrtc.org
==========

to

==========
Provide default impl of TimeUntilNextProcess in MockRtpRtcp instead of mocking.

This method isn't currently mocked or required by any test, so the safe thing
is to return a reasonably large value from the implementation to avoid busy
loops.

BUG=webrtc:7187
TBR=mflodman@webrtc.org

Review-Url: https://codereview.webrtc.org/2744233002
Cr-Commit-Position: refs/heads/master@{#17284}
Committed:
https://chromium.googlesource.com/external/webrtc/+/994ebb47024e7bf7e65039384...
==========

[commit-bot: I haz the power, 2017-03-17 11:01:37]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/994ebb47024e7bf7e65039384...