DescriptionRevert of Fix autoplay during prerendering for media elements on Android. (patchset #4 id:60001 of https://codereview.chromium.org/1293253003/ )
Reason for revert:
https://build.chromium.org/p/chromium.linux/builders/Linux%20Builder%20%28dbg%29/builds/92412/steps/compile/logs/stdio
This CL seems redefining MediaBrowserTest, which is already defined in chrome/browser/media/media_browsertest.h
and causing build breakage.
Original issue's description:
> Fix autoplay during prerendering for media elements on Android.
>
> If the user gesture for autoplay is disabled manually or via an
> experiment, prerendered media will play in a hidden frame.
>
> Within WebMediaPlayerImpl this behavior is deferred at load time
> via a callback, however WebMediaPlayerAndroid was never given
> this callback, so it would try to play media. Generally this
> fails because by default autoplay requires a user gesture.
>
> The correct fix is to refactor WMPA such that it uses a params
> structure like WMPI and can invoke the deferral callback when
> load() is called.
>
> BUG=522157
> TEST=clank no longer plays hidden a/v when user gesture disabled.
>
> Committed: https://crrev.com/98fc72cda10bf328ad55e460f6a7b06a356d5b3d
> Cr-Commit-Position: refs/heads/master@{#344676}
TBR=creis@chromium.org,qinmin@chromium.org,dalecurtis@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=522157
Committed: https://crrev.com/1b09bd614c0387158777f703cdb675384a35add3
Cr-Commit-Position: refs/heads/master@{#344682}
Patch Set 1 #
Created: 5 years, 4 months ago
(Patch set is too large to download)
Messages
Total messages: 5 (1 generated)
|