Allow multiple exits from LaunchUpdateCommand() for safety and clarity.

Allow multiple exits from LaunchUpdateCommand() for safety and clarity.

BUG=none
TEST=none

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

[grt (UTC plus 2), 2013-04-15 23:32:54]

https://codereview.chromium.org/14251022/diff/1/chrome_frame/update_launcher.cc
File chrome_frame/update_launcher.cc (right):

https://codereview.chromium.org/14251022/diff/1/chrome_frame/update_launcher....
chrome_frame/update_launcher.cc:62: ULONG_PTR phandle = NULL;
can this be a base::win::ScopedHandle, using its Receive() method on line 64? if
not, how about making |process| on line 67 a ScopedHandle?

[Peter Kasting, 2013-04-16 00:09:22]

https://codereview.chromium.org/14251022/diff/1/chrome_frame/update_launcher.cc
File chrome_frame/update_launcher.cc (right):

https://codereview.chromium.org/14251022/diff/1/chrome_frame/update_launcher....
chrome_frame/update_launcher.cc:62: ULONG_PTR phandle = NULL;
On 2013/04/15 23:32:54, grt wrote:
> can this be a base::win::ScopedHandle, using its Receive() method on line 64?
if
> not, how about making |process| on line 67 a ScopedHandle?

I don't think using Receive() will work because the callee expects a ULONG_PTR*.
 But I converted |process| to be a ScopedHandle.

[grt (UTC plus 2), 2013-04-16 13:24:30]

lgtm

[Peter Kasting, 2013-04-16 16:43:36]

Committed patchset #2 manually as r194370 (presubmit successful).