Add PERSISTENT_MAP usage for GpuMemoryBuffers.

content/common/child_process_host_impl.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 "content/common/child_process_host_impl.h"
 
 #include <limits>
 
 #include "base/atomic_sequence_num.h"
 #include "base/command_line.h"
@@ skipping 304 matching lines @@
     uint32 width,
     uint32 height,
     gfx::GpuMemoryBuffer::Format format,
     gfx::GpuMemoryBuffer::Usage usage,
     gfx::GpuMemoryBufferHandle* handle) {
   // TODO(reveman): Add support for other types of GpuMemoryBuffers.
 
   // AllocateForChildProcess() will check if |width| and |height| are valid
   // and handle failure in a controlled way when not. We just need to make
   // sure |format| and |usage| are supported here.
-  if (GpuMemoryBufferImplSharedMemory::IsFormatSupported(format) &&
-      usage == gfx::GpuMemoryBuffer::MAP) {

[reveman, 2015/05/20 14:26:08]

just change this line to
"GpuMemoryBufferImplSharedMemory::IsUsageSupported(usage)) {"
instead

[danakj, 2015/05/20 17:51:14]

Done.

-    *handle = GpuMemoryBufferImplSharedMemory::AllocateForChildProcess(
-        g_next_gpu_memory_buffer_id.GetNext(),
-        gfx::Size(width, height),
-        format,
-        peer_process_.Handle());
-  }
+  if (!GpuMemoryBufferImplSharedMemory::IsFormatSupported(format))
+    return;
+  if (usage != gfx::GpuMemoryBuffer::MAP &&
+      usage != gfx::GpuMemoryBuffer::PERSISTENT_MAP)
+    return;
+  *handle = GpuMemoryBufferImplSharedMemory::AllocateForChildProcess(
+      g_next_gpu_memory_buffer_id.GetNext(), gfx::Size(width, height), format,
+      peer_process_.Handle());
 }
 
 void ChildProcessHostImpl::OnDeletedGpuMemoryBuffer(
     gfx::GpuMemoryBufferId id,
     uint32 sync_point) {
   // Note: Nothing to do here as ownership of shared memory backed
   // GpuMemoryBuffers is passed with IPC.
 }
 
 }  // namespace content