Fixed invalid filtering of SCTP datachannel packets on high ports.

Fixed invalid filtering of SCTP datachannel packets on high ports.

Packets on source ports 32768-49151 got identified as RTP packets by
"IsRtpPacket" and were ignored by the SCTP transport.

This CL changes this to check the packet flags for "PF_SRTP_BYPASS".

BUG=webrtc:6959
Review-Url: https://codereview.webrtc.org/2743653005
Cr-Commit-Position: refs/heads/master@{#17179}
Committed: https://chromium.googlesource.com/external/webrtc/+/46d2457debb617b5ff4dfdbe61e3c637b9fb787b

[joachim, 2017-03-10 00:21:25]

jbauch@webrtc.org changed reviewers:
+ deadbeef@webrtc.org

[joachim, 2017-03-10 00:21:26]

Ptal

[Taylor Brandstetter, 2017-03-10 06:22:01]

lgtm

[joachim, 2017-03-10 06:40:22]

The CQ bit was checked by jbauch@webrtc.org

[commit-bot: I haz the power, 2017-03-10 06:40:25]

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-10 06:44:54]

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

[commit-bot: I haz the power, 2017-03-10 06:44:55]

Try jobs failed on following builders:
  presubmit on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/presubmit/builds/14685)

[Taylor Brandstetter, 2017-03-10 07:02:15]

On 2017/03/10 06:44:55, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   presubmit on master.tryserver.webrtc (JOB_FAILED,
> http://build.chromium.org/p/tryserver.webrtc/builders/presubmit/builds/14685)

Oh, I guess I'm not an owner of media/, and media/sctp/ doesn't have its own
OWNERS. I really should move this code into p2p/, I just haven't gotten around
to it.

[joachim, 2017-03-10 07:25:10]

jbauch@webrtc.org changed reviewers:
+ pthatcher@webrtc.org

[joachim, 2017-03-10 07:25:10]

+pthatcher for owner review. Ptal.

[pthatcher1, 2017-03-10 23:41:37]

lgtm

[joachim, 2017-03-11 00:17:00]

The CQ bit was checked by jbauch@webrtc.org

[commit-bot: I haz the power, 2017-03-11 00:17:12]

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-11 00:20:05]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489191420080290, "parent_rev":
"42a426372878850160e1e627478335a880cebd9f", "commit_rev":
"46d2457debb617b5ff4dfdbe61e3c637b9fb787b"}

[commit-bot: I haz the power, 2017-03-11 00:20:07]

Description was changed from

==========
Fixed invalid filtering of SCTP datachannel packets on high ports.

Packets on source ports 32768-49151 got identified as RTP packets by
"IsRtpPacket" and were ignored by the SCTP transport.

This CL changes this to check the packet flags for "PF_SRTP_BYPASS".

BUG=webrtc:6959
==========

to

==========
Fixed invalid filtering of SCTP datachannel packets on high ports.

Packets on source ports 32768-49151 got identified as RTP packets by
"IsRtpPacket" and were ignored by the SCTP transport.

This CL changes this to check the packet flags for "PF_SRTP_BYPASS".

BUG=webrtc:6959

Review-Url: https://codereview.webrtc.org/2743653005
Cr-Commit-Position: refs/heads/master@{#17179}
Committed:
https://chromium.googlesource.com/external/webrtc/+/46d2457debb617b5ff4dfdbe6...
==========

[commit-bot: I haz the power, 2017-03-11 00:20:08]

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