• mcasas's avatar
    VideoCapture: more migration IPC-->mojo, part 6 · 0abf0b0b
    mcasas authored
    This CL does the migration of the last Vid Capture IPC
    message to mojo, namely OnBufferCreated().  This
    message shared a SharedMemoryHandle, which is now
    send using mojo::{Wrap,Unwrap}SharedMemoryHandle().
    This needs a few small changes elsewhere in the code,
    in particular there is no need to ShareToProcess() a given
    ShMem (and this in turn makes us think that the whole
    data-sending can be simplified in later CLs).
    
    Following suggestion in PS1, VideoCaptureBufferTracker()
    is changed to
    mojo::ScopedSharedBufferHandle GetHandleForTransit();,
    which causes changes in a few other classes;
    - SharedMemoryBufferTracker, MojoSharedMemoryBufferTracker
    - VideoCaptureBufferPool & Impl,
    - VideoCaptureController (no need for render process id).
    (all these changes are in services/video_capture/ and
    content/browser/renderer_host/media/)
    
    VideoCaptureHost is still a BrowserMessageFilter,
    and VideoCaptureMessageFilter is not completely
    gone yet, but will disappear soon -- I didn't want to
    mix the changes in the ShMem management with
    other changes just in case.
    
    BUG=651897
    TEST=local capture, {content,capture,video_capture}_unittests
    
    Review-Url: https://chromiumcodereview.appspot.com/2410383002
    Cr-Commit-Position: refs/heads/master@{#426500}
    0abf0b0b
video_capture_messages.h 548 Bytes