Add heapcheck suppression for NSS trust leak.

Add heapcheck suppression for NSS certificate trust leak.

BUG=66941
TEST=none

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

[Ryan Sleevi, 2010-12-16 04:45:21]

TBR jhawkins, as you are listed as the current memory sheriff.

This is just the heapcheck suppression to the memcheck suppression landed as
part of http://codereview.chromium.org/4646001/. No heapcheck trybots meant I
didn't catch this before commit.

Unfortunately extra-broad because heapcheck is not picking up the NSS functions
doing the leaking like memcheck does.

[James Hawkins, 2010-12-16 04:46:30]

LGTM

[wtc, 2010-12-17 02:46:59]

http://codereview.chromium.org/5904002/diff/4001/tools/heapcheck/suppressions...
File tools/heapcheck/suppressions.txt (right):

http://codereview.chromium.org/5904002/diff/4001/tools/heapcheck/suppressions...
tools/heapcheck/suppressions.txt:107: fun:PR_NewLock
The suppressions should all include CERT_ChangeCertTrust
in the call stacks.

In fact, I think it's fine to merge these into a single
suppression with this call stack:
  fun:CERT_ChangeCertTrust
  fun:net::TestRootCerts::*

[Ryan Sleevi, 2010-12-17 02:50:17]

http://codereview.chromium.org/5904002/diff/4001/tools/heapcheck/suppressions...
File tools/heapcheck/suppressions.txt (right):

http://codereview.chromium.org/5904002/diff/4001/tools/heapcheck/suppressions...
tools/heapcheck/suppressions.txt:107: fun:PR_NewLock
On 2010/12/17 02:46:59, wtc wrote:
> The suppressions should all include CERT_ChangeCertTrust
> in the call stacks.
> 
> In fact, I think it's fine to merge these into a single
> suppression with this call stack:
>   fun:CERT_ChangeCertTrust
>   fun:net::TestRootCerts::*

wtc: Unfortunately, heapcheck wasn't picking up CERT_ChangeCertTrust as part of
the callstack. The suppressions went straight from Add/Clear into these three
functions. Otherwise, I would have just suppressed CERT_ChangeCertTrust without
the TestRootCerts, which is what I did for the Valgrind suppression.