Use std::vector in the PayloadRouter interface.

Use std::vector in the PayloadRouter interface.

Prevents copying a vector to a list that gets copied back to a vector,
which makes calling code a bit easier.

BUG=webrtc:5494
R=danilchap@webrtc.org

Committed: https://crrev.com/404686a3377bc74456a8e369426f0b1e39bafad7
Cr-Commit-Position: refs/heads/master@{#11589}

[pbos-webrtc, 2016-02-11 16:14:06]

PTAL

[danilchap, 2016-02-11 16:22:07]

https://codereview.webrtc.org/1686323003/diff/1/webrtc/video/vie_channel.cc
File webrtc/video/vie_channel.cc (right):

https://codereview.webrtc.org/1686323003/diff/1/webrtc/video/vie_channel.cc#n...
webrtc/video/vie_channel.cc:389:
send_payload_router_->SetSendingRtpModules(registered_modules);
why drop if(sender_) ?

[pbos-webrtc, 2016-02-11 16:38:32]

https://codereview.webrtc.org/1686323003/diff/1/webrtc/video/vie_channel.cc
File webrtc/video/vie_channel.cc (right):

https://codereview.webrtc.org/1686323003/diff/1/webrtc/video/vie_channel.cc#n...
webrtc/video/vie_channel.cc:389:
send_payload_router_->SetSendingRtpModules(registered_modules);
On 2016/02/11 16:22:07, danilchap wrote:
> why drop if(sender_) ?

DCHECKed above.

[danilchap, 2016-02-11 16:40:53]

On 2016/02/11 16:38:32, pbos-webrtc wrote:
> https://codereview.webrtc.org/1686323003/diff/1/webrtc/video/vie_channel.cc
> File webrtc/video/vie_channel.cc (right):
> 
>
https://codereview.webrtc.org/1686323003/diff/1/webrtc/video/vie_channel.cc#n...
> webrtc/video/vie_channel.cc:389:
> send_payload_router_->SetSendingRtpModules(registered_modules);
> On 2016/02/11 16:22:07, danilchap wrote:
> > why drop if(sender_) ?
> 
> DCHECKed above.

lgtm

[commit-bot: I haz the power, 2016-02-11 22:37:40]

Description was changed from

==========
Use std::vector in the PayloadRouter interface.

Prevents copying a vector to a list that gets copied back to a vector,
which makes calling code a bit easier.

BUG=webrtc:5494
R=danilchap@webrtc.org
==========

to

==========
Use std::vector in the PayloadRouter interface.

Prevents copying a vector to a list that gets copied back to a vector,
which makes calling code a bit easier.

BUG=webrtc:5494
R=danilchap@webrtc.org

Committed: https://crrev.com/404686a3377bc74456a8e369426f0b1e39bafad7
Cr-Commit-Position: refs/heads/master@{#11589}
==========

[pbos-webrtc, 2016-02-11 22:37:41]

Description was changed from

==========
Use std::vector in the PayloadRouter interface.

Prevents copying a vector to a list that gets copied back to a vector,
which makes calling code a bit easier.

BUG=webrtc:5494
R=danilchap@webrtc.org

Committed: https://crrev.com/404686a3377bc74456a8e369426f0b1e39bafad7
Cr-Commit-Position: refs/heads/master@{#11589}
==========

to

==========
Use std::vector in the PayloadRouter interface.

Prevents copying a vector to a list that gets copied back to a vector,
which makes calling code a bit easier.

BUG=webrtc:5494
R=danilchap@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/404686a3377bc74456a8e3694...
==========

[commit-bot: I haz the power, 2016-02-11 22:37:41]

Patchset 1 (id:??) landed as
https://crrev.com/404686a3377bc74456a8e369426f0b1e39bafad7
Cr-Commit-Position: refs/heads/master@{#11589}

[pbos-webrtc, 2016-02-11 22:37:42]

Committed patchset #1 (id:1) manually as
404686a3377bc74456a8e369426f0b1e39bafad7 (presubmit successful).