An assert was being hit because WaitableEvent's dtor was calling CloseHandle ...

An assert was being hit because WaitableEvent's dtor was calling CloseHandle on a handle that we already closed.

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

[agl, 2009-03-06 00:57:14]

LGTM

http://codereview.chromium.org/40195/diff/1/4
File chrome/common/child_process_host.cc (right):

http://codereview.chromium.org/40195/diff/1/4#newcode61
Line 61: process_event_.release();
Capital R on release?

[jam, 2009-03-06 01:09:21]

http://codereview.chromium.org/40195/diff/1/4
File chrome/common/child_process_host.cc (right):

http://codereview.chromium.org/40195/diff/1/4#newcode61
Line 61: process_event_.release();
On 2009/03/06 00:57:14, agl wrote:
> Capital R on release?

oops, that should have been a ->Release so that the WaitableEvent function is
called, not the scoped_ptr's!  Thanks for pointing it out.