Sandbox: remove raw handles from SharedMemIPCServer::ServerControl

sandbox/win/src/sharedmem_ipc_server.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef SANDBOX_SRC_SHAREDMEM_IPC_SERVER_H_
 #define SANDBOX_SRC_SHAREDMEM_IPC_SERVER_H_
 
 #include <list>
 
 #include "base/basictypes.h"
 #include "base/gtest_prod_util.h"
+#include "base/win/scoped_handle.h"
 #include "sandbox/win/src/crosscall_params.h"
 #include "sandbox/win/src/crosscall_server.h"
 #include "sandbox/win/src/sharedmem_ipc_client.h"
 
 // IPC transport implementation that uses shared memory.
 // This is the server side
 //
 // The server side has knowledge about the layout of the shared memory
 // and the state transitions. Both are explained in sharedmem_ipc_client.h
 //
@@ skipping 34 matching lines @@
   // do not work with sandbox tests.
   FRIEND_TEST_ALL_PREFIXES(IPCTest, SharedMemServerTests);
   // When an event fires (IPC request). A thread from the ThreadProvider
   // will call this function. The context parameter should be the same as
   // provided when ThreadProvider::RegisterWait was called.
   static void __stdcall ThreadPingEventReady(void* context,
                                              unsigned char);
 
   // Makes the client and server events. This function is called once
   // per channel.
-  bool MakeEvents(HANDLE* server_ping, HANDLE* server_pong,
+  bool MakeEvents(base::win::ScopedHandle* server_ping,
+                  base::win::ScopedHandle* server_pong,
                   HANDLE* client_ping, HANDLE* client_pong);
 
   // A copy this structure is maintained per channel.
   // Note that a lot of the fields are just the same of what we have in the IPC
   // object itself. It is better to have the copies since we can dispatch in the
   // static method without worrying about converting back to a member function
   // call or about threading issues.
   struct ServerControl {
     // This channel server ping event.
-    HANDLE ping_event;
+    base::win::ScopedHandle ping_event;
     // This channel server pong event.
-    HANDLE pong_event;
+    base::win::ScopedHandle pong_event;
     // The size of this channel.
     uint32 channel_size;
     // The pointer to the actual channel data.
     char* channel_buffer;
     // The pointer to the base of the shared memory.
     char* shared_base;
     // A pointer to this channel's client-side control structure this structure
     // lives in the shared memory.
     ChannelControl* channel;
     // the IPC dispatcher associated with this channel.
@@ skipping 29 matching lines @@
 
   // The dispatcher handles 'ready' IPC calls.
   Dispatcher* call_dispatcher_;
 
   DISALLOW_COPY_AND_ASSIGN(SharedMemIPCServer);
 };
 
 }  // namespace sandbox
 
 #endif  // SANDBOX_SRC_SHAREDMEM_IPC_SERVER_H_