Remove an unnecessary leak annotation.

Remove an unnecessary leak annotation.
Add memory leak suppressing for StatisticsRecorderTest.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=149989

[jar (doing other things), 2012-08-03 21:33:10]

lgtm

[commit-bot: I haz the power, 2012-08-03 21:55:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kaiwang@chromium.org/10827132/1

[commit-bot: I haz the power, 2012-08-04 01:15:09]

Change committed as 149989

[Alexander Potapenko, 2012-08-06 12:48:18]

Kai, Jim, can you elaborate on why the leak annotation is unnecessary here?

[kaiwang, 2012-08-06 22:10:35]

See
http://src.chromium.org/viewvc/chrome/trunk/src/base/debug/leak_annotations.h...

ANNOTATE_LEAKING_OBJECT_PTR will annotate the object and all objects referenced
by it as intentional leak.
Here, checksum_matching_list is already annotated as leak, so it's not necessary
to annotate the ranges

[Alexander Potapenko, 2012-08-07 10:08:56]

Am I missing something? Where is checksum_matching_list annotated as such?

[kaiwang, 2012-08-07 20:09:37]

On 2012/08/07 10:08:56, Alexander Potapenko wrote:
> Am I missing something? Where is checksum_matching_list annotated as such?

See line 110:
ANNOTATE_LEAKING_OBJECT_PTR(checksum_matching_list);

[Alexander Potapenko, 2012-08-08 09:30:19]

Oh, you're right. Sorry for being inattentive.