DescriptionRevert of Propagate audible state from player to the containing tab (patchset #16 id:310001 of https://codereview.chromium.org/896673003/)
Reason for revert:
This (Android only) change breaks the downstream Android test CastSwitchVideoTest#testNewVideoNewPageSameTab.
Reverting as Android Sheriff.
BUG=463445
Original issue's description:
> Propagate audible state from player to the containing tab.
>
> This is part 1 (chromium),
> part 2 (clank) is https://chrome-internal-review.googlesource.com/#/c/195455/
>
> This CL is only about propagation/plumbing.
> Neither the reporting of audible state my players nor the consumption
> (UI notification) are implemented.
>
> This patch creates a new object AudioMonitorAndroid and
> attaches it to MediaWebContentsObserver. The AudioMonitorAndroid
> maintains the audible state for the tab. It receives notifications
> from MediaPlayerAndroid objects and sends the resulting
> notification to the WebContents.
>
> BUG=414810
>
> Committed: https://crrev.com/6ad356d3e845ed9b67c5fb799b2ee8290f317268
> Cr-Commit-Position: refs/heads/master@{#318823}
TBR=qinmin@chromium.org,sievers@chromium.org,tedchoc@chromium.org,aurimas@chromium.org,timav@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=414810
Committed: https://crrev.com/6232fa583f944ee5f52be99b6eb9cee6fd3fdeb8
Cr-Commit-Position: refs/heads/master@{#318862}
Patch Set 1 #
Created: 5 years, 9 months ago
(Patch set is too large to download)
Messages
Total messages: 5 (1 generated)
|