• Matt Wolenetz's avatar
    [webcodecs] [MSE]: Keep Encoded{Audio,Video}Chunks non-dictionary · b5d0fe17
    Matt Wolenetz authored
    Drops a couple obsolete TODOs which previously tracked changing
    EncodedAudioChunk and EncodedVideoChunk to IDL dictionary types.
    This doesn't preclude adding parallel dictionary types that the APIs
    can consume later, if needed.
    
    For the MSE case, keeping them as non-IDL dictionaries retains the
    current ability for the bindings generator to precisely disambiguate
    them (for example in matching to an overload or union). This CL
    intends to increase certainty that that disambiguation capability
    will remain; otherwise new API shapes depending on it might need to
    change.
    
    Note that keeping them as non-dictionary types is nice for webcodecs;
    it better retains their immutability and enables potential for
    reduced/zero-copy scenarios (sandersd@ mentioned this, along with
    mentioning improved implementation control over behavior like
    construction and transferring).
    
    BUG=1144908
    
    Change-Id: Ib9fb6bcd50b6fe2900908c3380b75b798c71b667
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2535735Reviewed-by: default avatarChrome Cunningham <chcunningham@chromium.org>
    Reviewed-by: default avatarDan Sanders <sandersd@chromium.org>
    Commit-Queue: Matthew Wolenetz <wolenetz@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#827392}
    b5d0fe17
video_decoder.idl 2.98 KB