Add IceConfig and IceConfigRequest.

Add IceConfig and IceConfigRequest.

The new IceConfig struct will be used to pass TURN configuration to
PortAllocator. JingleInfoRequest has been moved to remoting/protocol
and now implements IceConfigRequest.

BUG=577954
Committed: https://crrev.com/58e801a5b7d4a5bfb300280cdf5b8eab6251abae
Cr-Commit-Position: refs/heads/master@{#375328}

[Sergey Ulanov, 2016-02-12 18:58:06]

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

[Jamie, 2016-02-12 22:48:28]

lgtm

https://codereview.chromium.org/1694613002/diff/1/remoting/protocol/ice_config.h
File remoting/protocol/ice_config.h (right):

https://codereview.chromium.org/1694613002/diff/1/remoting/protocol/ice_confi...
remoting/protocol/ice_config.h:27: std::vector<rtc::SocketAddress> stun_hosts;
s/hosts/servers/, here and below, for consistency with |turn_servers| (and
because we use "host" to refer to a CRD host)? Unless these terms are standard
nomenclature.

[Sergey Ulanov, 2016-02-13 01:11:02]

https://codereview.chromium.org/1694613002/diff/1/remoting/protocol/ice_config.h
File remoting/protocol/ice_config.h (right):

https://codereview.chromium.org/1694613002/diff/1/remoting/protocol/ice_confi...
remoting/protocol/ice_config.h:27: std::vector<rtc::SocketAddress> stun_hosts;
On 2016/02/12 22:48:28, Jamie wrote:
> s/hosts/servers/, here and below, for consistency with |turn_servers| (and
> because we use "host" to refer to a CRD host)? Unless these terms are standard
> nomenclature.

Done.

[Sergey Ulanov, 2016-02-13 01:12:25]

The CQ bit was checked by sergeyu@chromium.org

[Sergey Ulanov, 2016-02-13 01:12:25]

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/1694613002/#ps20001
(title: " ")

[commit-bot: I haz the power, 2016-02-13 01:14:22]

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

[commit-bot: I haz the power, 2016-02-13 02:07:29]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-16 22:47:02]

Description was changed from

==========
Add IceConfig and IceConfigRequest.

The new IceConfig struct will be used to pass TURN configuration to
PortAllocator. JingleInfoRequest has been moved to remoting/protocol
and now implements IceConfigRequest.

BUG=577954
==========

to

==========
Add IceConfig and IceConfigRequest.

The new IceConfig struct will be used to pass TURN configuration to
PortAllocator. JingleInfoRequest has been moved to remoting/protocol
and now implements IceConfigRequest.

BUG=577954

Committed: https://crrev.com/58e801a5b7d4a5bfb300280cdf5b8eab6251abae
Cr-Commit-Position: refs/heads/master@{#375328}
==========

[commit-bot: I haz the power, 2016-02-16 22:47:03]

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