DescriptionRevert 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
Committed: https://crrev.com/515ae254a0c8a4b29f87606c3464373e3b81400b
Cr-Commit-Position: refs/heads/master@{#294271}
Patch Set 1 #
Created: 6 years, 3 months ago
(Patch set is too large to download)
Messages
Total messages: 4 (0 generated)
|