Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(167)

Unified Diff: content/common/gpu/client/gpu_memory_buffer_impl_shared_memory.cc

Issue 331723003: gpu: Remove Create/DeleteImage IPC by adding an X11_PIXMAP_BUFFER GpuMemoryBuffer type. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/common/gpu/client/gpu_memory_buffer_impl_shared_memory.cc
diff --git a/content/common/gpu/client/gpu_memory_buffer_impl_shm.cc b/content/common/gpu/client/gpu_memory_buffer_impl_shared_memory.cc
similarity index 66%
rename from content/common/gpu/client/gpu_memory_buffer_impl_shm.cc
rename to content/common/gpu/client/gpu_memory_buffer_impl_shared_memory.cc
index 8420931420697ffaf739e52e867358d4e0c0d1a0..42daebc0089b4ae8a602ea946ea826758c1a7150 100644
--- a/content/common/gpu/client/gpu_memory_buffer_impl_shm.cc
+++ b/content/common/gpu/client/gpu_memory_buffer_impl_shared_memory.cc
@@ -1,22 +1,25 @@
-// Copyright 2013 The Chromium Authors. All rights reserved.
+// 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/common/gpu/client/gpu_memory_buffer_impl_shm.h"
+#include "content/common/gpu/client/gpu_memory_buffer_impl_shared_memory.h"
#include "base/numerics/safe_math.h"
#include "ui/gl/gl_bindings.h"
namespace content {
-GpuMemoryBufferImplShm::GpuMemoryBufferImplShm(const gfx::Size& size,
- unsigned internalformat)
- : GpuMemoryBufferImpl(size, internalformat) {}
+GpuMemoryBufferImplSharedMemory::GpuMemoryBufferImplSharedMemory(
+ const gfx::Size& size,
+ unsigned internalformat)
+ : GpuMemoryBufferImpl(size, internalformat) {
+}
-GpuMemoryBufferImplShm::~GpuMemoryBufferImplShm() {}
+GpuMemoryBufferImplSharedMemory::~GpuMemoryBufferImplSharedMemory() {
+}
// static
-void GpuMemoryBufferImplShm::AllocateSharedMemoryForChildProcess(
+void GpuMemoryBufferImplSharedMemory::AllocateSharedMemoryForChildProcess(
const gfx::Size& size,
unsigned internalformat,
base::ProcessHandle child_process,
@@ -35,8 +38,9 @@ void GpuMemoryBufferImplShm::AllocateSharedMemoryForChildProcess(
}
// static
-bool GpuMemoryBufferImplShm::IsLayoutSupported(const gfx::Size& size,
- unsigned internalformat) {
+bool GpuMemoryBufferImplSharedMemory::IsLayoutSupported(
+ const gfx::Size& size,
+ unsigned internalformat) {
base::CheckedNumeric<int> buffer_size = size.width();
buffer_size *= size.height();
buffer_size *= BytesPerPixel(internalformat);
@@ -44,7 +48,7 @@ bool GpuMemoryBufferImplShm::IsLayoutSupported(const gfx::Size& size,
}
// static
-bool GpuMemoryBufferImplShm::IsUsageSupported(unsigned usage) {
+bool GpuMemoryBufferImplSharedMemory::IsUsageSupported(unsigned usage) {
switch (usage) {
case GL_IMAGE_MAP_CHROMIUM:
return true;
@@ -54,13 +58,14 @@ bool GpuMemoryBufferImplShm::IsUsageSupported(unsigned usage) {
}
// static
-bool GpuMemoryBufferImplShm::IsConfigurationSupported(const gfx::Size& size,
- unsigned internalformat,
- unsigned usage) {
+bool GpuMemoryBufferImplSharedMemory::IsConfigurationSupported(
+ const gfx::Size& size,
+ unsigned internalformat,
+ unsigned usage) {
return IsLayoutSupported(size, internalformat) && IsUsageSupported(usage);
}
-bool GpuMemoryBufferImplShm::Initialize() {
+bool GpuMemoryBufferImplSharedMemory::Initialize() {
DCHECK(IsLayoutSupported(size_, internalformat_));
scoped_ptr<base::SharedMemory> shared_memory(new base::SharedMemory());
if (!shared_memory->CreateAnonymous(size_.GetArea() *
@@ -71,7 +76,7 @@ bool GpuMemoryBufferImplShm::Initialize() {
return true;
}
-bool GpuMemoryBufferImplShm::InitializeFromHandle(
+bool GpuMemoryBufferImplSharedMemory::InitializeFromHandle(
gfx::GpuMemoryBufferHandle handle) {
DCHECK(IsLayoutSupported(size_, internalformat_));
if (!base::SharedMemory::IsHandleValid(handle.handle))
@@ -81,7 +86,7 @@ bool GpuMemoryBufferImplShm::InitializeFromHandle(
return true;
}
-void* GpuMemoryBufferImplShm::Map() {
+void* GpuMemoryBufferImplSharedMemory::Map() {
DCHECK(!mapped_);
if (!shared_memory_->Map(size_.GetArea() * BytesPerPixel(internalformat_)))
return NULL;
@@ -89,17 +94,17 @@ void* GpuMemoryBufferImplShm::Map() {
return shared_memory_->memory();
}
-void GpuMemoryBufferImplShm::Unmap() {
+void GpuMemoryBufferImplSharedMemory::Unmap() {
DCHECK(mapped_);
shared_memory_->Unmap();
mapped_ = false;
}
-uint32 GpuMemoryBufferImplShm::GetStride() const {
+uint32 GpuMemoryBufferImplSharedMemory::GetStride() const {
return size_.width() * BytesPerPixel(internalformat_);
}
-gfx::GpuMemoryBufferHandle GpuMemoryBufferImplShm::GetHandle() const {
+gfx::GpuMemoryBufferHandle GpuMemoryBufferImplSharedMemory::GetHandle() const {
gfx::GpuMemoryBufferHandle handle;
handle.type = gfx::SHARED_MEMORY_BUFFER;
handle.handle = shared_memory_->handle();

Powered by Google App Engine
This is Rietveld 408576698