include what you use: errno.h and string.h in net/

include what you use: errno.h and string.h in net/

Adding some missing errno.h and string.h includes.

Committed: https://crrev.com/82ccf4656fbc6819e5dcbe507751e1d29691fc31
Cr-Commit-Position: refs/heads/master@{#358844}

[Mostyn Bramley-Moore, 2015-11-07 00:14:13]

mostynb@opera.com changed reviewers:
+ xunjieli@chromium.org

[Mostyn Bramley-Moore, 2015-11-07 00:14:14]

@xunjieli: here are some include-what-you-use fixes.

[xunjieli, 2015-11-09 21:53:29]

https://codereview.chromium.org/1427143003/diff/1/net/socket/tcp_socket_win.cc
File net/socket/tcp_socket_win.cc (right):

https://codereview.chromium.org/1427143003/diff/1/net/socket/tcp_socket_win.c...
net/socket/tcp_socket_win.cc:8: #include <errno.h>
Why do we need this header for this file?

[Mostyn Bramley-Moore, 2015-11-09 23:13:08]

https://codereview.chromium.org/1427143003/diff/1/net/socket/tcp_socket_win.cc
File net/socket/tcp_socket_win.cc (right):

https://codereview.chromium.org/1427143003/diff/1/net/socket/tcp_socket_win.c...
net/socket/tcp_socket_win.cc:8: #include <errno.h>
On 2015/11/09 21:53:29, xunjieli wrote:
> Why do we need this header for this file?

Because the errno variable is used (on line 623).

https://codereview.chromium.org/1427143003/diff/1/net/socket/tcp_socket_win.c...
net/socket/tcp_socket_win.cc:623: return MapSystemError(errno);
Note the errno variable being used here.

[xunjieli, 2015-11-10 15:43:17]

On 2015/11/09 23:13:08, Mostyn Bramley-Moore wrote:
> https://codereview.chromium.org/1427143003/diff/1/net/socket/tcp_socket_win.cc
> File net/socket/tcp_socket_win.cc (right):
> 
>
https://codereview.chromium.org/1427143003/diff/1/net/socket/tcp_socket_win.c...
> net/socket/tcp_socket_win.cc:8: #include <errno.h>
> On 2015/11/09 21:53:29, xunjieli wrote:
> > Why do we need this header for this file?
> 
> Because the errno variable is used (on line 623).
> 
>
https://codereview.chromium.org/1427143003/diff/1/net/socket/tcp_socket_win.c...
> net/socket/tcp_socket_win.cc:623: return MapSystemError(errno);
> Note the errno variable being used here.

lgtm. Thanks. I didn't realize that is a macro defined variable.

[Mostyn Bramley-Moore, 2015-11-10 17:08:47]

The CQ bit was checked by mostynb@opera.com

[commit-bot: I haz the power, 2015-11-10 17:09:07]

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

[commit-bot: I haz the power, 2015-11-10 17:51:21]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-11-10 17:52:04]

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