Fix a leak in client_cert_resolver

Fix a leak in client_cert_resolver

BUG=272596
TEST=chromeos_unittests under valgrind

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=247558

[pneubeck (no reviews), 2014-01-28 08:13:48]

Cool that you tracked that down!
lgtm

[commit-bot: I haz the power, 2014-01-28 18:39:35]

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

[commit-bot: I haz the power, 2014-01-28 18:39:40]

Failed to apply patch for tools/valgrind/memcheck/suppressions.txt:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file tools/valgrind/memcheck/suppressions.txt
  Hunk #1 FAILED at 5886.
  1 out of 1 hunk FAILED -- saving rejects to file
tools/valgrind/memcheck/suppressions.txt.rej

Patch:       tools/valgrind/memcheck/suppressions.txt
Index: tools/valgrind/memcheck/suppressions.txt
diff --git a/tools/valgrind/memcheck/suppressions.txt
b/tools/valgrind/memcheck/suppressions.txt
index
832b8ef56cc7bd3d85d8bb8ac95f0a856614a0ae..27c76c1a461c8bd218ea9ba60b934d8de84be41f
100644
--- a/tools/valgrind/memcheck/suppressions.txt
+++ b/tools/valgrind/memcheck/suppressions.txt
@@ -5886,17 +5886,6 @@
   
fun:_ZN4base11MessageLoop8PostTaskERKN15tracked_objects8LocationERKNS_8CallbackIFvvEEE
 }
 {
-   bug_272596
-   Memcheck:Leak
-   ...
-   fun:nssPKIObjectCollection_AddInstanceAsObject
-   fun:nssToken_TraverseCertificates
-   fun:NSSTrustDomain_TraverseCertificates
-   fun:PK11_ListCerts
-  
fun:_ZN3net15NSSCertDatabase9ListCertsEPSt6vectorI13scoped_refptrINS_15X509CertificateEESaIS4_EE
-   fun:_ZN8chromeos12_GLOBAL__N_119LoadNSSCertificates*
-}
-{
    bug_273398
    Memcheck:Leak
    ...

[commit-bot: I haz the power, 2014-01-28 19:23:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tbarzic@chromium.org/145273015/100001

[commit-bot: I haz the power, 2014-01-29 01:12:36]

Change committed as 247558