[Mojo Video Capture] Adapt video_capture service to refactored video capture stack

media/capture/video/shared_memory_buffer_handle.cc

Index: media/capture/video/shared_memory_buffer_handle.cc
diff --git a/media/capture/video/shared_memory_buffer_handle.cc b/media/capture/video/shared_memory_buffer_handle.cc
new file mode 100644
index 0000000000000000000000000000000000000000..fb2801aa7c316c245c5d02a1ac2b26d52f8b33e7
--- /dev/null
+++ b/media/capture/video/shared_memory_buffer_handle.cc
@@ -0,0 +1,31 @@
+// Copyright 2016 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 "media/capture/video/shared_memory_buffer_handle.h"
+
+#include "base/memory/ptr_util.h"
+#include "mojo/public/cpp/system/platform_handle.h"
+
+namespace media {
+
+SharedMemoryBufferHandle::SharedMemoryBufferHandle(
+    base::SharedMemory* shared_memory,
+    size_t mapped_size)
+    : shared_memory_(shared_memory), mapped_size_(mapped_size) {}
+
+SharedMemoryBufferHandle::~SharedMemoryBufferHandle() = default;
+
+size_t SharedMemoryBufferHandle::mapped_size() const {
+  return mapped_size_;
+}
+
+uint8_t* SharedMemoryBufferHandle::data() {
+  return static_cast<uint8_t*>(shared_memory_->memory());

[emircan, 2017/04/19 18:16:48]

We should add checks to make sure |share_memory_| is not null at any stage,
since we don't have a mechanism to determine its lifetime. Also, please add
comments about lifetime.

[chfremer, 2017/04/19 23:03:37]

I don't think checks would work here. 
Added comment to the class declaration.

+}
+
+const uint8_t* SharedMemoryBufferHandle::data() const {
+  return static_cast<const uint8_t*>(shared_memory_->memory());
+}
+
+}  // namespace media