[Video] MediaSession API event handlers can resume background video.

[Video] MediaSession API event handlers can resume background video.

ShouldPauseVideoWhenHidden is now used for videos with audio too so that
HTMLMediaElement::play() actually reaches WMPI because the video is
paused.

WMPI::UpdatePlayState_ComputePlayState now only cares about the presence
of the session and allows playback of anything as long as it got play()
command.

WMPI now tracks whether background video playback has been unlocked by a
user gesture. RendererWebMediaPlayerDelegate emulates user gesture in
OnPlay/OnPause to behave the same way as MediaSession.

BUG=689637
TEST=manual+updated tests

Review-Url: https://codereview.chromium.org/2681863005
Cr-Commit-Position: refs/heads/master@{#452700}
(cherry picked from commit 65fad2795b571f868c559941187f785aaa0414a0)

R=sandersd@chromium.org

Review-Url: https://codereview.chromium.org/2705213007 .
Cr-Commit-Position: refs/branch-heads/2987@{#682}
Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943}
Committed: https://chromium.googlesource.com/chromium/src/+/a8ce00278da90b8abbdf39b0f74ddbb417e0035e

[whywhat, 2017-02-24 01:33:58]

Removed non-existing argument in the test

[whywhat, 2017-02-24 18:25:48]

Fixed WMPA

[whywhat, 2017-02-24 18:27:27]

avayvod@chromium.org changed reviewers:
+ sandersd@chromium.org

[whywhat, 2017-02-24 18:27:27]

Dan, would you please take a look at the WMPA change for a sanity check?
I tried it locally and it seems to work.

[sandersd (OOO until July 31), 2017-02-24 20:23:53]

lgtm

[whywhat, 2017-02-24 21:51:15]

Description was changed from

==========
[Video] MediaSession API event handlers can resume background video.

ShouldPauseVideoWhenHidden is now used for videos with audio too so that
HTMLMediaElement::play() actually reaches WMPI because the video is
paused.

WMPI::UpdatePlayState_ComputePlayState now only cares about the presence
of the session and allows playback of anything as long as it got play()
command.

WMPI now tracks whether background video playback has been unlocked by a
user gesture. RendererWebMediaPlayerDelegate emulates user gesture in
OnPlay/OnPause to behave the same way as MediaSession.

BUG=689637
TEST=manual+updated tests

Review-Url: https://codereview.chromium.org/2681863005
Cr-Commit-Position: refs/heads/master@{#452700}
(cherry picked from commit 65fad2795b571f868c559941187f785aaa0414a0)
==========

to

==========
[Video] MediaSession API event handlers can resume background video.

ShouldPauseVideoWhenHidden is now used for videos with audio too so that
HTMLMediaElement::play() actually reaches WMPI because the video is
paused.

WMPI::UpdatePlayState_ComputePlayState now only cares about the presence
of the session and allows playback of anything as long as it got play()
command.

WMPI now tracks whether background video playback has been unlocked by a
user gesture. RendererWebMediaPlayerDelegate emulates user gesture in
OnPlay/OnPause to behave the same way as MediaSession.

BUG=689637
TEST=manual+updated tests

Review-Url: https://codereview.chromium.org/2681863005
Cr-Commit-Position: refs/heads/master@{#452700}
(cherry picked from commit 65fad2795b571f868c559941187f785aaa0414a0)

R=sandersd@chromium.org

Review-Url: https://codereview.chromium.org/2705213007 .
Cr-Commit-Position: refs/branch-heads/2987@{#682}
Cr-Branched-From:
ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943}
Committed:
https://chromium.googlesource.com/chromium/src/+/a8ce00278da90b8abbdf39b0f74d...
==========

[whywhat, 2017-02-24 21:51:16]

Committed patchset #3 (id:40001) manually as
a8ce00278da90b8abbdf39b0f74ddbb417e0035e (presubmit successful).