Change MessagePumpForGpu and SharedMemory to ScopedHandle

base/memory/shared_memory_unittest.cc

 // 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.
 
 #include "base/memory/shared_memory.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <memory>
@@ skipping 574 matching lines @@
       ::CreateFileMappingA(file_handle.Get(), nullptr,
                            PAGE_READONLY | SEC_IMAGE, 0, 0, kTestSectionName));
   EXPECT_TRUE(section_handle.IsValid());
 
   // Check direct opening by name, from handle and duplicated from handle.
   SharedMemory shared_memory_open;
   EXPECT_TRUE(shared_memory_open.Open(kTestSectionName, true));
   EXPECT_FALSE(shared_memory_open.Map(1));
   EXPECT_EQ(nullptr, shared_memory_open.memory());
 
-  SharedMemory shared_memory_handle_dup(

[stanisc, 2016/06/29 18:36:18]

This part of the test is redundant (identical to the next one). And it was
causing a problem due to double tracking / double closing of the handle.

-      SharedMemoryHandle(section_handle.Get(), ::GetCurrentProcessId()), true);
-  EXPECT_FALSE(shared_memory_handle_dup.Map(1));
-  EXPECT_EQ(nullptr, shared_memory_handle_dup.memory());
-
   SharedMemory shared_memory_handle_local(
       SharedMemoryHandle(section_handle.Take(), ::GetCurrentProcessId()), true);
   EXPECT_FALSE(shared_memory_handle_local.Map(1));
   EXPECT_EQ(nullptr, shared_memory_handle_local.memory());
 
   // Check that a handle without SECTION_QUERY also can't be mapped as it can't
   // be checked.
   SharedMemory shared_memory_handle_dummy;
   SharedMemoryCreateOptions options;
   options.size = 0x1000;
@@ skipping 83 matching lines @@
   memory.Close();
   SharedMemoryProcessTest::CleanUp();
 }
 
 MULTIPROCESS_TEST_MAIN(SharedMemoryTestMain) {
   return SharedMemoryProcessTest::TaskTestMain();
 }
 #endif  // !defined(OS_IOS) && !defined(OS_ANDROID) && !defined(OS_MACOSX)
 
 }  // namespace base