commit c99b6c793653b57cdf462e37109b1ed4d3addeea
author Zhi Huang <zhihuang@webrtc.org> Fri Nov 10 16:44:46 2017 -0800
committer Commit Bot <commit-bot@chromium.org> Sat Nov 11 01:14:35 2017 +0000
tree b1191650c077f26498302ff8007060c692bca212
parent 83119dd83387a2a6abdbca2c3ee98b4353e11bc9

Remove the SetEncryptedHeaderExtensionIds methods.

The existing methods SetEncrypedHeaderExtensionIds in SrtpTransport and SrtpSession
are removed because those methods could be confusing. When these methods are called
the head extension IDs are not actually updated and the user need to call SetRtpParams
again to make that happen. The existing setter just caches the new IDs.

To make it less confusing, the SetEncryptedHeaderExtensionIds is removed and the new
extension IDs will be set immediately when setting the crypto params.

For SDES, the crypto params and the header extension IDs will be set at the same time.

For DTLS, the new header extensions are cached in BaseChannel and will be set when
the DTLS handshake is completed.

Another major change is that when doing DTLS-SRTP, the encrypted header extension
IDs will be updated only when they are changed.

Bug: webrtc:7013
Change-Id: Ib70d4797456ae5ecb61b3dfff15c7e3e7ede89bd
Reviewed-on: https://webrtc-review.googlesource.com/15860
Commit-Queue: Zhi Huang <zhihuang@webrtc.org>
Reviewed-by: Peter Thatcher <pthatcher@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#20639}