IPC: Use ScopedHandle instead of a raw HANDLE for the private members.

IPC: Use ScopedHandle instead of a raw HANDLE for the private members.

BUG=387876
R=cpu@chromium.org

Committed: https://crrev.com/8c2d75c0c7f873096d9c6948338dd8a237c3edb7
Cr-Commit-Position: refs/heads/master@{#297011}

[cpu_(ooo_6.6-7.5), 2014-09-25 23:07:27]

lgtm

https://codereview.chromium.org/588383003/diff/1/ipc/ipc_channel_win.cc
File ipc/ipc_channel_win.cc (right):

https://codereview.chromium.org/588383003/diff/1/ipc/ipc_channel_win.cc#newco...
ipc/ipc_channel_win.cc:143: 
morita has a lgtmed change here.

https://codereview.chromium.org/588383003/diff/1/ipc/ipc_channel_win.cc#newco...
ipc/ipc_channel_win.cc:255: // only type with a base::File member. In any case,
this code should not
is this justin thing I was talking about?

[rvargas (doing something else), 2014-09-25 23:26:20]

thanks

https://codereview.chromium.org/588383003/diff/1/ipc/ipc_channel_win.cc
File ipc/ipc_channel_win.cc (right):

https://codereview.chromium.org/588383003/diff/1/ipc/ipc_channel_win.cc#newco...
ipc/ipc_channel_win.cc:143: 
On 2014/09/25 23:07:27, cpu wrote:
> morita has a lgtmed change here.

Yes, I know. This part is just for completeness.

https://codereview.chromium.org/588383003/diff/1/ipc/ipc_channel_win.cc#newco...
ipc/ipc_channel_win.cc:255: // only type with a base::File member. In any case,
this code should not
On 2014/09/25 23:07:27, cpu wrote:
> is this justin thing I was talking about?

Nope. Amit added this long time ago... it's not used by Chrome proper, but it
has a few calls from remoting and cloud print.

[rvargas (doing something else), 2014-09-26 18:42:42]

The CQ bit was checked by rvargas@chromium.org

[commit-bot: I haz the power, 2014-09-26 18:44:14]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/588383003/20001

[commit-bot: I haz the power, 2014-09-26 19:50:35]

Committed patchset #2 (id:20001) as ff8c7991002880b338e86b645a637a7f3b159298

[commit-bot: I haz the power, 2014-09-26 19:51:20]

Patchset 2 (id:??) landed as
https://crrev.com/8c2d75c0c7f873096d9c6948338dd8a237c3edb7
Cr-Commit-Position: refs/heads/master@{#297011}