Attempting to re-land CL 3110006 which turned out to have ...

Attempting to re-land CL 3110006 which turned out to have 
a memory leak.

Add support for speaking SSL to an HTTP Proxy, to
HttpProxyClientSocketPool (and friends)

More information about an HTTPS Proxy can be found here:

http://dev.chromium.org/spdy/spdy-proxy

This implementation supports both http:// and https:// requests,
as well as support for both Proxy and Server auth.

BUG=29625
TEST=none


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

[Ryan Hamilton, 2010-08-25 17:50:12]

Hi Guys,

I had a memory leak in my earlier CL.  In hindsight it was blindingly obvious
(thanks Mike!).  I believe I have fixed it.  I've uploaded 2 patches in this CL.
 The first is the original CL (updated to work with trunk), and the second is
the patch to fix the memory leak.

I'd appreciate a review when you have a minute.  I've verified that valgrind
passes for me (on my box, where it was failing with the leak).  I'll also run it
against the trybots before committing.

Cheers,

Ryan

[vandebo (ex-Chrome), 2010-08-25 17:59:52]

LGTM

Your try jobs failed because lkgr is before the revert.  You can tell the try
servers to use a different revision with -r.

[rch (use chromium not google), 2010-08-25 18:09:31]

Ah!  I'll give that a shot.  Thanks...

On Aug 25, 2010 10:59:53 AM PDT, vandebo <vandebo@chromium.org> wrote:
> LGTM
> Your try jobs failed because lkgr is before the revert. You can tell
> the try
> servers to use a different revision with -r.