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

services/video_capture/mojo_shared_memory_buffer_tracker.cc

 // 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 "services/video_capture/mojo_shared_memory_buffer_tracker.h"
 
 #include "base/memory/ptr_util.h"
 #include "services/video_capture/mojo_shared_memory_buffer_handle.h"
 
 namespace video_capture {
@@ skipping 24 matching lines @@
   frame_ = media::MojoSharedBufferVideoFrame::CreateDefaultI420(
       dimensions, base::TimeDelta());
   return true;
 }
 
 std::unique_ptr<media::VideoCaptureBufferHandle>
 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