QUIC - fix memory leak.

QUIC - fix memory leak.

R=rch@chromium.org
BUG=232777
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=195037

[Derek Bruening, 2013-04-17 23:05:57]

Can you also remove the Valgrind and Heapcheck suppressions added in r194676 as
part of putting in this real fix?

[ramant (doing other things), 2013-04-17 23:15:16]

PTAL.

[Derek Bruening, 2013-04-17 23:16:13]

LGTM

[Ryan Hamilton, 2013-04-18 18:18:56]

lgtm

[commit-bot: I haz the power, 2013-04-18 18:21:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rtenneti@chromium.org/14328013/8001

[commit-bot: I haz the power, 2013-04-18 18:21:44]

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

Patch:       tools/heapcheck/suppressions.txt
Index: tools/heapcheck/suppressions.txt
diff --git a/tools/heapcheck/suppressions.txt b/tools/heapcheck/suppressions.txt
index
945c3c4f3e016f85e9a488c1fc48182c80196029..7c09fc429dccc1e9a37055460e437a4aa7d2d93f
100644
--- a/tools/heapcheck/suppressions.txt
+++ b/tools/heapcheck/suppressions.txt
@@ -1878,14 +1878,6 @@
    fun:cc::DelayBasedTimeSource::OnTimerFired
 }
 {
-   bug_232777
-   Heapcheck:Leak
-   ...
-   fun:net::QuicStreamFactory::GetOrCreateCryptoConfig
-   fun:net::QuicStreamFactory::CreateSession
-   fun:net::QuicStreamFactory::Job::DoConnect
-}
-{
    bug_232924
    Heapcheck:Leak
    ...

[commit-bot: I haz the power, 2013-04-18 18:40:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rtenneti@chromium.org/14328013/14001

[commit-bot: I haz the power, 2013-04-18 22:21:28]

Retried try job too often on win7_aura for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&...

[commit-bot: I haz the power, 2013-04-18 22:23:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rtenneti@chromium.org/14328013/14001

[commit-bot: I haz the power, 2013-04-19 01:01:06]

Change committed as 195037