content: Use type-parameterized tests for GpuMemoryBuffer implementations.

content/common/gpu/client/gpu_memory_buffer_impl_shared_memory_unittest.cc

Index: content/common/gpu/client/gpu_memory_buffer_impl_shared_memory_unittest.cc
diff --git a/content/common/gpu/client/gpu_memory_buffer_impl_shared_memory_unittest.cc b/content/common/gpu/client/gpu_memory_buffer_impl_shared_memory_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..836e898c025eafb94f3fdd710719df77bf0a67dd
--- /dev/null
+++ b/content/common/gpu/client/gpu_memory_buffer_impl_shared_memory_unittest.cc
@@ -0,0 +1,43 @@
+// Copyright 2015 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_shared_memory.h"
+#include "content/test/gpu_memory_buffer_impl_test_template.h"
+
+namespace content {
+namespace {
+
+INSTANTIATE_TYPED_TEST_CASE_P(GpuMemoryBufferImplSharedMemory,
+                              GpuMemoryBufferImplTest,
+                              GpuMemoryBufferImplSharedMemory);
+
+void BufferDestroyed(bool* destroyed, uint32 sync_point) {
+  *destroyed = true;
+}
+
+TEST(GpuMemoryBufferImplSharedMemoryTest, Create) {
+  const gfx::GpuMemoryBufferId kBufferId(1);
+
+  gfx::Size buffer_size(8, 8);
+
+  for (auto format : gfx::GetBufferFormats()) {
+    if (!GpuMemoryBufferImplSharedMemory::IsFormatSupported(format))
+      continue;
+
+    bool destroyed = false;
+    scoped_ptr<GpuMemoryBufferImplSharedMemory> buffer(
+        GpuMemoryBufferImplSharedMemory::Create(
+            kBufferId, buffer_size, format,
+            base::Bind(&BufferDestroyed, base::Unretained(&destroyed))));
+    ASSERT_TRUE(buffer);
+    EXPECT_EQ(buffer->GetFormat(), format);
+
+    // Check if destruction callback is executed when deleting the buffer.
+    buffer.reset();
+    EXPECT_TRUE(destroyed);

[piman, 2015/10/07 01:20:17]

nit: ASSERT_TRUE. If not true, the callback has a reference to a dangling stack
object, you definitely want to abort at that point.

[reveman, 2015/10/07 09:05:01]

Done.

+  }
+}
+
+}  // namespace
+}  // namespace content