Update pepper to not assume that SharedMemoryHandle is an int.

content/renderer/pepper/pepper_compositor_host.cc

 // Copyright 2014 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 "content/renderer/pepper/pepper_compositor_host.h"
 
 #include <limits>
 
 #include "base/logging.h"
 #include "base/memory/shared_memory.h"
@@ skipping 98 matching lines @@
       return PP_ERROR_BADARGUMENT;
     }
 
     // Make sure the source rect is not beyond the dimensions of the
     // image.
     if (!CheckPPFloatRect(new_layer->image->source_rect,
                           desc.size.width, desc.size.height)) {
       return PP_ERROR_BADARGUMENT;
     }
 
-    int handle;
+    base::SharedMemoryHandle handle;
     uint32_t byte_count;
     if (enter.object()->GetSharedMemory(&handle, &byte_count) != PP_OK)
       return PP_ERROR_FAILED;
 
 #if defined(OS_WIN)
     base::SharedMemoryHandle shm_handle;
     if (!::DuplicateHandle(::GetCurrentProcess(),
                            reinterpret_cast<base::SharedMemoryHandle>(handle),
                            ::GetCurrentProcess(),
                            &shm_handle,
                            0,
                            FALSE,
                            DUPLICATE_SAME_ACCESS)) {
       return PP_ERROR_FAILED;
     }
 #else
-    base::SharedMemoryHandle shm_handle(dup(handle), false);
+    base::SharedMemoryHandle shm_handle =
+        base::SharedMemory::DeepCopyHandle(handle, false);

[piman, 2015/06/01 22:16:08]

Can you do this in both OS_WIN and !OS_WIN paths?

[erikchen, 2015/06/01 23:48:53]

Yes. I will do so in a follow up CL. DeepCopyHandle() isn't defined on OS_WIN
yet, so I'll need to add that to base/memory first.

I removed an unnecessary cast to base::SharedMemoryHandle.

 #endif
     image_shm->reset(new base::SharedMemory(shm_handle, true));
     if (!(*image_shm)->Map(desc.stride * desc.size.height)) {
       image_shm->reset();
       return PP_ERROR_NOMEMORY;
     }
     return PP_OK;
   }
 
   return PP_ERROR_BADARGUMENT;
@@ skipping 267 matching lines @@
 
   // If the host is not bound to the instance, return PP_OK immediately.
   if (!bound_instance_)
     return PP_OK;
 
   commit_layers_reply_context_ = context->MakeReplyMessageContext();
   return PP_OK_COMPLETIONPENDING;
 }
 
 }  // namespace content