Move RTP keep-alive config from VideoSendStream::Config to Call::Config

Move RTP keep-alive config from VideoSendStream::Config to Call::Config

This makes more sense since logically it's a transport level feature,
not a media stream feature. Even if the implementation details forces it
to be an rtp stream detail, for the moment.

BUG=webrtc:7907
Review-Url: https://codereview.webrtc.org/2978503002
Cr-Commit-Position: refs/heads/master@{#18963}
Committed: https://chromium.googlesource.com/external/webrtc/+/e5c4a810e21e74acec09c987f20fb0e6788062fd

[sprang_webrtc, 2017-07-10 15:49:05]

sprang@webrtc.org changed reviewers:
+ pbos@webrtc.org

[pbos-webrtc, 2017-07-10 22:40:24]

lgtm

https://codereview.webrtc.org/2978503002/diff/1/webrtc/call/call.h
File webrtc/call/call.h (right):

https://codereview.webrtc.org/2978503002/diff/1/webrtc/call/call.h#newcode117
webrtc/call/call.h:117: // network time-out events.
Is there a RFC you can point to here?

[sprang_webrtc, 2017-07-11 08:39:58]

https://codereview.webrtc.org/2978503002/diff/1/webrtc/call/call.h
File webrtc/call/call.h (right):

https://codereview.webrtc.org/2978503002/diff/1/webrtc/call/call.h#newcode117
webrtc/call/call.h:117: // network time-out events.
On 2017/07/10 22:40:23, pbos-webrtc wrote:
> Is there a RFC you can point to here?

Done.

[sprang_webrtc, 2017-07-11 08:40:17]

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

[commit-bot: I haz the power, 2017-07-11 08:40:23]

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-07-11 09:22:54]

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

[commit-bot: I haz the power, 2017-07-11 09:22:55]

Dry run: This issue passed the CQ dry run.

[sprang_webrtc, 2017-07-11 10:41:31]

The CQ bit was checked by sprang@webrtc.org

[sprang_webrtc, 2017-07-11 10:41:31]

The patchset sent to the CQ was uploaded after l-g-t-m from pbos@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2978503002/#ps40001 (title:
"Typo in comment")

[commit-bot: I haz the power, 2017-07-11 10:41:37]

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-07-11 10:44:18]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1499769691224740,
"parent_rev": "2910357621dee4368bd3eaa0040cec82ac230dad", "commit_rev":
"e5c4a810e21e74acec09c987f20fb0e6788062fd"}

[commit-bot: I haz the power, 2017-07-11 10:44:22]

Description was changed from

==========
Move RTP keep-alive config from VideoSendStream::Config to Call::Config

This makes more sense since logically it's a transport level feature,
not a media stream feature. Even if the implementation details forces it
to be an rtp stream detail, for the moment.

BUG=webrtc:7907
==========

to

==========
Move RTP keep-alive config from VideoSendStream::Config to Call::Config

This makes more sense since logically it's a transport level feature,
not a media stream feature. Even if the implementation details forces it
to be an rtp stream detail, for the moment.

BUG=webrtc:7907

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

[commit-bot: I haz the power, 2017-07-11 10:44:23]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/external/webrtc/+/e5c4a810e21e74acec09c987f...