• Christian Fremerey's avatar
    [Video Capture Service] Break dependency of video capture service Mojo API from media_types.mojom · 1cecc366
    Christian Fremerey authored
    The API of the video capture service currently depends on a native enum VideoPixelFormat
    and a struct media.mojom.MailboxVideoFrameData declared in media_types.mojom. This CL breaks
    this dependency by defining a (non-native) Mojo enum VideoCapturePixelFormat and a struct
    media.mojom.MailboxBufferHandleSet specifically for the video capture service. This has
    two benefits:
    1.) It makes it clear from the API that the video capture service actually only supports a
    much more limited set of pixel formats.
    2.) It allows the reuse of the API definition in the mojom files in contexts where media/
    and type mappings to native enums are not available. In particular, this is the case
    when trying to use the video capture service API from ChromiumOS.
    
    This is purely an interface refactoring. There is no intent for functional change.
    
    TBR=xjz@chromium.org
    
    Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel
    Change-Id: I418e5bb95bd92da0600afd874b6fde71e982761a
    Reviewed-on: https://chromium-review.googlesource.com/1026795Reviewed-by: default avatarChristian Fremerey <chfremer@chromium.org>
    Reviewed-by: default avatarKen Rockot <rockot@chromium.org>
    Reviewed-by: default avatarTom Sepez <tsepez@chromium.org>
    Reviewed-by: default avatarEmircan Uysaler <emircan@chromium.org>
    Commit-Queue: Christian Fremerey <chfremer@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#555718}
    1cecc366
BUILD.gn 665 Bytes