include what you use: errno.h in process_memory_dump.cc

include what you use: errno.h in process_memory_dump.cc

Followup to https://codereview.chromium.org/1398163003 - include errno.h
since we use errno in process_memory_dump.cc.

BUG=542503
Committed: https://crrev.com/d017082377c2d6ff16de0724258974189c161970
Cr-Commit-Position: refs/heads/master@{#358024}

[Mostyn Bramley-Moore, 2015-11-05 09:46:14]

The CQ bit was checked by mostynb@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2015-11-05 09:46:28]

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

[Mostyn Bramley-Moore, 2015-11-05 09:48:15]

mostynb@opera.com changed reviewers:
+ primiano@chromium.org, ssid@chromium.org

[Mostyn Bramley-Moore, 2015-11-05 09:48:16]

@Primiano: please take a look at this minor followup to
https://codereview.chromium.org/1398163003 - this allows us to build this code
for a greater range of embedded targets.

[Primiano Tucci (use gerrit), 2015-11-05 09:54:07]

LGTM, but plz use errno.h just for consistency with the codebase

https://codereview.chromium.org/1417743004/diff/1/base/trace_event/process_me...
File base/trace_event/process_memory_dump.cc (right):

https://codereview.chromium.org/1417743004/diff/1/base/trace_event/process_me...
base/trace_event/process_memory_dump.cc:7: #include <cerrno>
can you make it errno.h ?
I think I've never seen the "c" prefix version of includes being used in
chromium/src.

[Mostyn Bramley-Moore, 2015-11-05 09:59:13]

Description was changed from

==========
include what you use: cerrno

Followup to https://codereview.chromium.org/1398163003 - include cerrno
since we use errno in process_memory_dump.cc.

BUG=542503
==========

to

==========
include what you use: errno.h

Followup to https://codereview.chromium.org/1398163003 - include errno.h
since we use errno in process_memory_dump.cc.

BUG=542503
==========

[Mostyn Bramley-Moore, 2015-11-05 09:59:51]

https://codereview.chromium.org/1417743004/diff/1/base/trace_event/process_me...
File base/trace_event/process_memory_dump.cc (right):

https://codereview.chromium.org/1417743004/diff/1/base/trace_event/process_me...
base/trace_event/process_memory_dump.cc:7: #include <cerrno>
On 2015/11/05 09:54:07, Primiano Tucci wrote:
> can you make it errno.h ?
> I think I've never seen the "c" prefix version of includes being used in
> chromium/src.

Done.

[Mostyn Bramley-Moore, 2015-11-05 09:59:57]

The CQ bit was checked by mostynb@opera.com

[Mostyn Bramley-Moore, 2015-11-05 09:59:58]

The patchset sent to the CQ was uploaded after l-g-t-m from
primiano@chromium.org
Link to the patchset: https://codereview.chromium.org/1417743004/#ps20001
(title: "use errno.h instead of cerrno")

[commit-bot: I haz the power, 2015-11-05 10:00:42]

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

[Mostyn Bramley-Moore, 2015-11-05 10:49:46]

Description was changed from

==========
include what you use: errno.h

Followup to https://codereview.chromium.org/1398163003 - include errno.h
since we use errno in process_memory_dump.cc.

BUG=542503
==========

to

==========
include what you use: errno.h in process_memory_dump.cc

Followup to https://codereview.chromium.org/1398163003 - include errno.h
since we use errno in process_memory_dump.cc.

BUG=542503
==========

[commit-bot: I haz the power, 2015-11-05 11:03:17]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-05 11:04:19]

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