Linux: Correctly count the number of open file descriptors.

Linux: Correctly count the number of open file descriptors.

The initial implementation of ProcessMetrics::GetOpenFdCount() counted
"." and ".." as well.

Add a unit test to verify the counting is correct.

BUG=496957
Committed: https://crrev.com/cf4c85b1897bf9b1f62e4c48a4faf122958c3790
Cr-Commit-Position: refs/heads/master@{#365504}

[Lei Zhang, 2015-12-16 03:09:34]

thestig@chromium.org changed reviewers:
+ afakhry@chromium.org, rickyz@chromium.org

[Lei Zhang, 2015-12-16 03:09:34]

To punish myself for being a bad reviewer, here's a follow up CL with a unit
test to make sure our new code actually does what we think it does.

With only the unit test, it fails at runtime with an actual value of 2 vs the
expected value of 0.

[rickyz (no longer on Chrome), 2015-12-16 03:42:58]

lgtm

You're awesome - thanks for picking this up/adding a test!

[Lei Zhang, 2015-12-16 07:16:58]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2015-12-16 07:17:14]

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

[commit-bot: I haz the power, 2015-12-16 08:33:38]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-16 08:34:22]

Description was changed from

==========
Linux: Correctly count the number of open file descriptors.

The initial implementation of ProcessMetrics::GetOpenFdCount() counted
"." and ".." as well.

Add a unit test to verify the counting is correct.

BUG=496957
==========

to

==========
Linux: Correctly count the number of open file descriptors.

The initial implementation of ProcessMetrics::GetOpenFdCount() counted
"." and ".." as well.

Add a unit test to verify the counting is correct.

BUG=496957

Committed: https://crrev.com/cf4c85b1897bf9b1f62e4c48a4faf122958c3790
Cr-Commit-Position: refs/heads/master@{#365504}
==========

[commit-bot: I haz the power, 2015-12-16 08:34:23]

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

[afakhry, 2015-12-16 18:22:58]

On 2015/12/16 08:34:23, commit-bot: I haz the power wrote:
> Patchset 2 (id:??) landed as
> https://crrev.com/cf4c85b1897bf9b1f62e4c48a4faf122958c3790
> Cr-Commit-Position: refs/heads/master@{#365504}

Thank you so much! I like the test!