Update ChromiumPacketSocket factory to apply PacketOptions it receives.

Update ChromiumPacketSocket factory to apply PacketOptions it receives.

In Chromium libjingle is compiled with ENABLED_EXTERNAL_AUTH. This means
that the sending PacketSocketFactory is responsible for updating send
time in the outgoing RTP packets and calculating HMAC.
ChromiumPacketSocket previously wasn't doing that, so the receiver could
not validate the RTP packets.

BUG=547158
Committed: https://crrev.com/8cf7497b319ee3b7c0fe6449790c13d78a8b61e5
Cr-Commit-Position: refs/heads/master@{#369885}

[Sergey Ulanov, 2016-01-12 19:32:32]

sergeyu@chromium.org changed reviewers:
+ jamiewalch@chromium.org

[Sergey Ulanov, 2016-01-12 19:32:32]

This change depends on https://codereview.chromium.org/1578323002/ and
https://codereview.chromium.org/1577123004/

[Jamie, 2016-01-12 22:08:09]

LGTM, but I think there are some errors in the CL description:

s/Thins/This/
s/could validate/could not validate/

[Sergey Ulanov, 2016-01-12 22:14:45]

Description was changed from

==========
Update ChromiumPacketSocket factory to apply PacketOptions it receives.

In Chromium libjingle is compiled with ENABLED_EXTERNAL_AUTH. Thins means
that the sending PacketSocketFactory is responsible for updating send
time in the outgoing RTP packets and calculating HMAC.
ChromiumPacketSocket previously wasn't doing that, so the receiver could
validate the RTP packets.

BUG=547158
==========

to

==========
Update ChromiumPacketSocket factory to apply PacketOptions it receives.

In Chromium libjingle is compiled with ENABLED_EXTERNAL_AUTH. This means
that the sending PacketSocketFactory is responsible for updating send
time in the outgoing RTP packets and calculating HMAC.
ChromiumPacketSocket previously wasn't doing that, so the receiver could
not validate the RTP packets.

BUG=547158
==========

[Sergey Ulanov, 2016-01-15 23:43:04]

The CQ bit was checked by sergeyu@chromium.org

[Sergey Ulanov, 2016-01-15 23:43:05]

The patchset sent to the CQ was uploaded after l-g-t-m from
jamiewalch@chromium.org
Link to the patchset: https://codereview.chromium.org/1574353003/#ps20001
(title: " ")

[commit-bot: I haz the power, 2016-01-15 23:43:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1574353003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1574353003/20001

[commit-bot: I haz the power, 2016-01-16 00:07:40]

Description was changed from

==========
Update ChromiumPacketSocket factory to apply PacketOptions it receives.

In Chromium libjingle is compiled with ENABLED_EXTERNAL_AUTH. This means
that the sending PacketSocketFactory is responsible for updating send
time in the outgoing RTP packets and calculating HMAC.
ChromiumPacketSocket previously wasn't doing that, so the receiver could
not validate the RTP packets.

BUG=547158
==========

to

==========
Update ChromiumPacketSocket factory to apply PacketOptions it receives.

In Chromium libjingle is compiled with ENABLED_EXTERNAL_AUTH. This means
that the sending PacketSocketFactory is responsible for updating send
time in the outgoing RTP packets and calculating HMAC.
ChromiumPacketSocket previously wasn't doing that, so the receiver could
not validate the RTP packets.

BUG=547158
==========

[commit-bot: I haz the power, 2016-01-16 00:07:40]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-16 00:08:47]

Description was changed from

==========
Update ChromiumPacketSocket factory to apply PacketOptions it receives.

In Chromium libjingle is compiled with ENABLED_EXTERNAL_AUTH. This means
that the sending PacketSocketFactory is responsible for updating send
time in the outgoing RTP packets and calculating HMAC.
ChromiumPacketSocket previously wasn't doing that, so the receiver could
not validate the RTP packets.

BUG=547158
==========

to

==========
Update ChromiumPacketSocket factory to apply PacketOptions it receives.

In Chromium libjingle is compiled with ENABLED_EXTERNAL_AUTH. This means
that the sending PacketSocketFactory is responsible for updating send
time in the outgoing RTP packets and calculating HMAC.
ChromiumPacketSocket previously wasn't doing that, so the receiver could
not validate the RTP packets.

BUG=547158

Committed: https://crrev.com/8cf7497b319ee3b7c0fe6449790c13d78a8b61e5
Cr-Commit-Position: refs/heads/master@{#369885}
==========

[commit-bot: I haz the power, 2016-01-16 00:08:48]

Patchset 2 (id:??) landed as
https://crrev.com/8cf7497b319ee3b7c0fe6449790c13d78a8b61e5
Cr-Commit-Position: refs/heads/master@{#369885}