• boliu's avatar
    Revert of Clean up WebMediaPlayerAndroid needs_establish_peer_ (patchset #7... · 515ae254
    boliu authored
    Revert of Clean up WebMediaPlayerAndroid needs_establish_peer_ (patchset #7 id:120001 of https://codereview.chromium.org/557593002/)
    
    Reason for revert:
    Suspected to break tough_video_cases. See crbug.com/412897 for details.
    
    BUG=412897
    
    Original issue's description:
    > Clean up WebMediaPlayerAndroid needs_establish_peer_
    >
    > needs_establish_peer_ has been used to skip actual estalishing a surface
    > texture peer based on many conditions. Not all places that modify
    > actually checks all these conditions so there are bound to be some bugs.
    >
    > This CL aims to tease out all these different conditions and put them in
    > a single function EstablishSurfaceTexturePeerIfNeeded. This function
    > should be called any time these conditions are changed. Makes for much
    > saner reading.
    >
    > BUG=412578
    >
    > Committed: https://crrev.com/701cb640efb326aa107b5b0c25dcf91766fe765f
    > Cr-Commit-Position: refs/heads/master@{#294068}
    
    TBR=qinmin@chromium.org,avayvod@chromium.org,ycheo@chromium.org
    NOTREECHECKS=true
    NOTRY=true
    BUG=412578
    
    Review URL: https://codereview.chromium.org/562803002
    
    Cr-Commit-Position: refs/heads/master@{#294271}
    515ae254
webmediaplayer_android.cc 62.5 KB