VideoCapture: more migration IPC-->mojo, part 6

services/video_capture/mojo_shared_memory_buffer_tracker.cc

Index: services/video_capture/mojo_shared_memory_buffer_tracker.cc
diff --git a/services/video_capture/mojo_shared_memory_buffer_tracker.cc b/services/video_capture/mojo_shared_memory_buffer_tracker.cc
index 42170e81d6ea361d270b6e37fddc739c9e955c0a..4de53971b9de82ef401f827a7908d44b60e43e22 100644
--- a/services/video_capture/mojo_shared_memory_buffer_tracker.cc
+++ b/services/video_capture/mojo_shared_memory_buffer_tracker.cc
@@ -42,10 +42,9 @@ MojoSharedMemoryBufferTracker::GetBufferHandle() {
   return base::MakeUnique<MojoSharedMemoryBufferHandle>(this);
 }
 
-bool MojoSharedMemoryBufferTracker::ShareToProcess(
-    base::ProcessHandle process_handle,
-    base::SharedMemoryHandle* new_handle) {
-  return false;
+mojo::ScopedSharedBufferHandle
+MojoSharedMemoryBufferTracker::GetHandleForTransit() {
+  return mojo::ScopedSharedBufferHandle();

[dcheng, 2016/10/20 08:37:56]

I'm not entirely sure I understand the purpose of this class. Is this a
NOTREACHED() kind of thing?

(Perhaps a class-level comment could be useful)

[mcasas, 2016/10/20 15:09:51]

//services/video_capture is still under development [1],
but when it's ready, we should consider better mojo
ways to share data from service impl to client, e.g.
a mojo DataPipe, but for the time being it needs to
comply to the VideoCaptureBufferTracker, which 
was designed in a pre-mojo world.

[1] https://crbug.com/584797

 }
 
 }  // namespace video_capture