Using different sequence numbers for different SSRCs

Using different sequence numbers for different SSRCs

This seems to solve the unexpected behavior when selecting lower layers.
Also, this replaces https://codereview.webrtc.org/1327153002/

Committed: https://crrev.com/e78e2c714bdbecb910526746d9e3678a245a8f8b
Cr-Commit-Position: refs/heads/master@{#10206}

[ivica, 2015-09-22 12:50:49]

ivica@webrtc.org changed reviewers:
+ mflodman@webrtc.org, sprang@webrtc.org

[sprang_webrtc, 2015-09-23 13:58:19]

Looks good. Perhaps you should add a unit test as well?

https://codereview.webrtc.org/1350383004/diff/20001/webrtc/test/layer_filteri...
File webrtc/test/layer_filtering_transport.cc (right):

https://codereview.webrtc.org/1350383004/diff/20001/webrtc/test/layer_filteri...
webrtc/test/layer_filtering_transport.cc:39: else
don't need this else

https://codereview.webrtc.org/1350383004/diff/20001/webrtc/test/layer_filteri...
webrtc/test/layer_filtering_transport.cc:98: }
Remove {}

[mflodman, 2015-10-02 09:08:37]

LGTM with Erik's comments addressed.

[ivica, 2015-10-07 12:07:42]

Unit tests would require simulating packets and catching them, which seems to be
unexpectedly difficult. If you agree, I would like to leave that for later.

https://codereview.webrtc.org/1350383004/diff/20001/webrtc/test/layer_filteri...
File webrtc/test/layer_filtering_transport.cc (right):

https://codereview.webrtc.org/1350383004/diff/20001/webrtc/test/layer_filteri...
webrtc/test/layer_filtering_transport.cc:39: else
On 2015/09/23 13:58:19, språng wrote:
> don't need this else

Done.

https://codereview.webrtc.org/1350383004/diff/20001/webrtc/test/layer_filteri...
webrtc/test/layer_filtering_transport.cc:98: }
On 2015/09/23 13:58:19, språng wrote:
> Remove {}

Done.

[sprang_webrtc, 2015-10-07 13:50:46]

lgtm

[ivica, 2015-10-07 14:03:54]

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

[commit-bot: I haz the power, 2015-10-07 14:04:07]

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

[commit-bot: I haz the power, 2015-10-07 14:59:54]

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

[commit-bot: I haz the power, 2015-10-07 14:59:55]

Dry run: This issue passed the CQ dry run.

[ivica, 2015-10-08 06:43:02]

The CQ bit was checked by ivica@webrtc.org

[ivica, 2015-10-08 06:43:02]

The patchset sent to the CQ was uploaded after l-g-t-m from mflodman@webrtc.org
Link to the patchset: https://codereview.webrtc.org/1350383004/#ps80001 (title:
"Adding a comment")

[commit-bot: I haz the power, 2015-10-08 06:43:19]

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

[commit-bot: I haz the power, 2015-10-08 06:44:35]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-10-08 06:44:46]

Patchset 5 (id:??) landed as
https://crrev.com/e78e2c714bdbecb910526746d9e3678a245a8f8b
Cr-Commit-Position: refs/heads/master@{#10206}