• Thomas Guilbert's avatar
    Remove MediaResourceGetter · b625d909
    Thomas Guilbert authored
    Before playing media using the MediaPlayerBridge, we try to parse the
    metadata, using the MediaResourceGetter. This class uses the Android
    MediaMetadataRetriever API, which can be be resource intensive. We
    eventually get all of the metadata we need from the Android
    MediaPlayer, and pre-parsing the metadata sometimes does nothing but
    delay the start of playback.
    
    This CL removes the java MediaResourceGetter, and removes metadata
    extraction capabilities from the C++ portion of the
    MediaResourceGetter interface.
    
    NB: This is unlikely to have a performance effect on HLS playback,
    since the MediaResourceGetter did not support parsing HLS.
    
    Bug: 739914
    Change-Id: I460d5aea02bf1a0bf099f66a781a40ca6374d4d0
    Reviewed-on: https://chromium-review.googlesource.com/988681Reviewed-by: default avatarPeter Wen <wnwen@chromium.org>
    Reviewed-by: default avatarDale Curtis <dalecurtis@chromium.org>
    Reviewed-by: default avatarYaron Friedman <yfriedman@chromium.org>
    Commit-Queue: Thomas Guilbert <tguilbert@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#548221}
    b625d909
BUILD.gn 30.8 KB