[BinarySize] Filter duplicate lines in nm output.

[BinarySize] Filter duplicate lines in nm output.

nm just outputs the symbols it finds in the debug sections of the binary
and sometimes the same symbol appears more than once, with the exact
same data.

The binary_size tool would think that it was two different symbols that
shared the same address so it would get the numbers right, but the
output would be confusing since the memory would be split in two
halves.

BUG=

Committed: https://crrev.com/9a8e892c20f14492af3cade9c6444de4e1267c1a
Cr-Commit-Position: refs/heads/master@{#372359}

[Daniel Bratell, 2016-01-29 13:21:13]

bratell@opera.com changed reviewers:
+ andrewhayden@chromium.org

[Daniel Bratell, 2016-01-29 13:21:13]

Can you please take a look? Just something I noticed when trying to make sense
of some strange output after a binary size reduction experiment (that didn't
work out).

[Andrew Hayden (chromium.org), 2016-01-29 13:41:04]

On 2016/01/29 13:21:13, Daniel Bratell wrote:
> Can you please take a look? Just something I noticed when trying to make sense
> of some strange output after a binary size reduction experiment (that didn't
> work out).

LGTM! Thanks.

[Daniel Bratell, 2016-01-29 13:48:52]

The CQ bit was checked by bratell@opera.com

[commit-bot: I haz the power, 2016-01-29 13:49:19]

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

[commit-bot: I haz the power, 2016-01-29 13:58:00]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-01-29 13:58:01]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Daniel Bratell, 2016-01-29 15:59:41]

bratell@opera.com changed reviewers:
+ thakis@chromium.org

[Daniel Bratell, 2016-01-29 15:59:41]

thakis, can you please take a look? The code is already reviewed but this tools
subdirectory didn't have an OWNERS file so I've added one with the people that
more or less (mostly less) change this code.

[Nico, 2016-01-29 16:04:39]

rs-lgtm

[Daniel Bratell, 2016-01-29 16:26:00]

The CQ bit was checked by bratell@opera.com

[Daniel Bratell, 2016-01-29 16:26:01]

The patchset sent to the CQ was uploaded after l-g-t-m from
andrewhayden@chromium.org
Link to the patchset: https://codereview.chromium.org/1645843004/#ps20001
(title: "Also add an OWNERS file")

[commit-bot: I haz the power, 2016-01-29 16:26:27]

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

[commit-bot: I haz the power, 2016-01-29 16:49:11]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-29 16:50:47]

Description was changed from

==========
[BinarySize] Filter duplicate lines in nm output.

nm just outputs the symbols it finds in the debug sections of the binary
and sometimes the same symbol appears more than once, with the exact
same data.

The binary_size tool would think that it was two different symbols that
shared the same address so it would get the numbers right, but the
output would be confusing since the memory would be split in two
halves.

BUG=
==========

to

==========
[BinarySize] Filter duplicate lines in nm output.

nm just outputs the symbols it finds in the debug sections of the binary
and sometimes the same symbol appears more than once, with the exact
same data.

The binary_size tool would think that it was two different symbols that
shared the same address so it would get the numbers right, but the
output would be confusing since the memory would be split in two
halves.

BUG=

Committed: https://crrev.com/9a8e892c20f14492af3cade9c6444de4e1267c1a
Cr-Commit-Position: refs/heads/master@{#372359}
==========

[commit-bot: I haz the power, 2016-01-29 16:50:47]

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