Migrate content/*/p2p/* code to net::IPAddress.

Migrate content/*/p2p/* code to net::IPAddress.

BUG=496258
Committed: https://crrev.com/92c439fbcf4850b1ea8f90d80074e9150ab94f42
Cr-Commit-Position: refs/heads/master@{#383287}

[martijnc, 2016-03-24 19:34:28]

Patchset #1 (id:1) has been deleted

[martijnc, 2016-03-24 19:43:40]

martijn@martijnc.be changed reviewers:
+ dcheng@chromium.org, eroman@chromium.org, sergeyu@chromium.org

[martijnc, 2016-03-24 20:35:01]

martijn@martijnc.be changed reviewers:
- eroman@chromium.org

[martijnc, 2016-03-24 20:38:16]

martijn@martijnc.be changed reviewers:
+ eroman@chromium.org

[martijnc, 2016-03-24 20:38:16]

sergeyu: can you take a look at the p2p and jingle changes?
dcheng: can you take a look at the IPC change (p2p_messages.h)?

eroman: fyi

Thanks!

https://codereview.chromium.org/1833523002/diff/20001/content/renderer/p2p/so...
File content/renderer/p2p/socket_dispatcher.h (left):

https://codereview.chromium.org/1833523002/diff/20001/content/renderer/p2p/so...
content/renderer/p2p/socket_dispatcher.h:40: #include "net/base/ip_endpoint.h"
This was included and forward-declared.

[Sergey Ulanov, 2016-03-24 20:45:45]

lgtm after my nit about function name is addressed.

https://codereview.chromium.org/1833523002/diff/20001/jingle/glue/utils.h
File jingle/glue/utils.h (right):

https://codereview.chromium.org/1833523002/diff/20001/jingle/glue/utils.h#new...
jingle/glue/utils.h:35: rtc::IPAddress NetIPAddressToIPAddress(const
net::IPAddress& ip_address);
Call it NetIPAddressToRtcIPAddress() to make it less confusing please.

[eroman, 2016-03-24 21:36:04]

lgtm

[dcheng, 2016-03-24 21:58:48]

lgtm

[martijnc, 2016-03-25 16:01:04]

The CQ bit was checked by martijn@martijnc.be

[martijnc, 2016-03-25 16:01:04]

https://codereview.chromium.org/1833523002/diff/20001/jingle/glue/utils.h
File jingle/glue/utils.h (right):

https://codereview.chromium.org/1833523002/diff/20001/jingle/glue/utils.h#new...
jingle/glue/utils.h:35: rtc::IPAddress NetIPAddressToIPAddress(const
net::IPAddress& ip_address);
On 2016/03/24 at 20:45:45, Sergey Ulanov wrote:
> Call it NetIPAddressToRtcIPAddress() to make it less confusing please.

Done.

[martijnc, 2016-03-25 16:01:05]

The patchset sent to the CQ was uploaded after l-g-t-m from sergeyu@chromium.org
Link to the patchset: https://codereview.chromium.org/1833523002/#ps40001
(title: "comments sergeyu")

[commit-bot: I haz the power, 2016-03-25 16:01:15]

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

[commit-bot: I haz the power, 2016-03-25 16:48:52]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-03-25 16:49:52]

Description was changed from

==========
Migrate content/*/p2p/* code to net::IPAddress.

BUG=496258
==========

to

==========
Migrate content/*/p2p/* code to net::IPAddress.

BUG=496258

Committed: https://crrev.com/92c439fbcf4850b1ea8f90d80074e9150ab94f42
Cr-Commit-Position: refs/heads/master@{#383287}
==========

[commit-bot: I haz the power, 2016-03-25 16:49:53]

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