Reland Connect TurnPort and TCPPort to AsyncPacketSocket::SignalSentPacket.

Reland Connect TurnPort and TCPPort to AsyncPacketSocket::SignalSentPacket.

Chromium reported errors when building libjingle_nacl due to some methods used virtual instead of override when they were overriding the base class. My guess is that when one method starts using override, all other in the same class must too.

R=tommi@webrtc.org
TBR=pthatcher@webtrc.org

BUG=4173
Committed: https://crrev.com/55674ffb32307c6f3efaab442340d3c5c075073b
Cr-Commit-Position: refs/heads/master@{#11251}

[stefan-webrtc, 2016-01-14 14:33:36]

Fix build issues when building with Chromium.

[stefan-webrtc, 2016-01-14 14:35:41]

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

[stefan-webrtc, 2016-01-14 14:37:02]

Description was changed from

==========
Reland Connect TurnPort and TCPPort to AsyncPacketSocket::SignalSentPacket.

BUG=4173
==========

to

==========
Reland Connect TurnPort and TCPPort to AsyncPacketSocket::SignalSentPacket.

Chromium reported errors when building libjingle_nacl due to some methods used
virtual instead of override when they were overriding the base class. My guess
is that when one method starts using override, all other in the same class must
too.

BUG=4173
==========

[stefan-webrtc, 2016-01-14 14:37:26]

Description was changed from

==========
Reland Connect TurnPort and TCPPort to AsyncPacketSocket::SignalSentPacket.

Chromium reported errors when building libjingle_nacl due to some methods used
virtual instead of override when they were overriding the base class. My guess
is that when one method starts using override, all other in the same class must
too.

BUG=4173
==========

to

==========
Reland Connect TurnPort and TCPPort to AsyncPacketSocket::SignalSentPacket.

Chromium reported errors when building libjingle_nacl due to some methods used
virtual instead of override when they were overriding the base class. My guess
is that when one method starts using override, all other in the same class must
too.

TBR=pthatcher@webtrc.org

BUG=4173
==========

[stefan-webrtc, 2016-01-14 14:38:43]

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

[commit-bot: I haz the power, 2016-01-14 14:38:49]

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

[tommi, 2016-01-14 14:41:48]

lgtm

[stefan-webrtc, 2016-01-14 14:49:28]

Description was changed from

==========
Reland Connect TurnPort and TCPPort to AsyncPacketSocket::SignalSentPacket.

Chromium reported errors when building libjingle_nacl due to some methods used
virtual instead of override when they were overriding the base class. My guess
is that when one method starts using override, all other in the same class must
too.

TBR=pthatcher@webtrc.org

BUG=4173
==========

to

==========
Reland Connect TurnPort and TCPPort to AsyncPacketSocket::SignalSentPacket.

Chromium reported errors when building libjingle_nacl due to some methods used
virtual instead of override when they were overriding the base class. My guess
is that when one method starts using override, all other in the same class must
too.

R=tommi@webrtc.org
TBR=pthatcher@webtrc.org

BUG=4173

Committed:
https://chromium.googlesource.com/external/webrtc/+/55674ffb32307c6f3efaab442...
==========

[stefan-webrtc, 2016-01-14 14:49:30]

Committed patchset #2 (id:20001) manually as
55674ffb32307c6f3efaab442340d3c5c075073b (presubmit successful).

[commit-bot: I haz the power, 2016-01-14 14:49:32]

Description was changed from

==========
Reland Connect TurnPort and TCPPort to AsyncPacketSocket::SignalSentPacket.

Chromium reported errors when building libjingle_nacl due to some methods used
virtual instead of override when they were overriding the base class. My guess
is that when one method starts using override, all other in the same class must
too.

R=tommi@webrtc.org
TBR=pthatcher@webtrc.org

BUG=4173

Committed:
https://chromium.googlesource.com/external/webrtc/+/55674ffb32307c6f3efaab442...
==========

to

==========
Reland Connect TurnPort and TCPPort to AsyncPacketSocket::SignalSentPacket.

Chromium reported errors when building libjingle_nacl due to some methods used
virtual instead of override when they were overriding the base class. My guess
is that when one method starts using override, all other in the same class must
too.

R=tommi@webrtc.org
TBR=pthatcher@webtrc.org

BUG=4173

Committed: https://crrev.com/55674ffb32307c6f3efaab442340d3c5c075073b
Cr-Commit-Position: refs/heads/master@{#11251}
==========

[commit-bot: I haz the power, 2016-01-14 14:49:33]

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