Don't pause suspended videos with no audio.

Don't pause suspended videos with no audio.

This came up in some earlier code review.
The purpose of the timer is to pause audible backgrounded videos so they
don't surprise user when the tab is foregrounded. Video only players don't
have this problem and they often don't have a play button to resume playback.

BUG=None

Review-Url: https://codereview.chromium.org/2696423003
Cr-Commit-Position: refs/heads/master@{#455224}
Committed: https://chromium.googlesource.com/chromium/src/+/52efd28a984d8b4f5afdd9a3a6be1654499db1a3

[whywhat, 2017-02-16 20:43:32]

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

[whywhat, 2017-02-16 20:43:33]

PTaL

[DaleCurtis, 2017-02-16 20:48:27]

Seems okay, WMPI test? lgtm

[whywhat, 2017-03-07 18:59:18]

Added WMPI tests

[whywhat, 2017-03-07 18:59:43]

On 2017/02/16 at 20:48:27, dalecurtis wrote:
> Seems okay, WMPI test? lgtm

Done.

[whywhat, 2017-03-07 18:59:47]

The CQ bit was checked by avayvod@chromium.org

[whywhat, 2017-03-07 18:59:48]

The patchset sent to the CQ was uploaded after l-g-t-m from
dalecurtis@chromium.org
Link to the patchset: https://codereview.chromium.org/2696423003/#ps20001
(title: "Added WMPI tests")

[commit-bot: I haz the power, 2017-03-07 19:01:05]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-07 21:13:10]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1488913187874850,
"parent_rev": "44b074ebba30c2f7c20c70f6a3a6a95ebc341037", "commit_rev":
"c1fd94b55dd10dd4afeea792fb23bb3adf8d289c"}

[commit-bot: I haz the power, 2017-03-07 21:17:50]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1488913187874850,
"parent_rev": "0164b8789c0946f0fa9aac63a6fca058c3b101ad", "commit_rev":
"52efd28a984d8b4f5afdd9a3a6be1654499db1a3"}

[commit-bot: I haz the power, 2017-03-07 21:18:31]

Description was changed from

==========
Don't pause suspended videos with no audio.

This came up in some earlier code review.
The purpose of the timer is to pause audible backgrounded videos so they
don't surprise user when the tab is foregrounded. Video only players don't
have this problem and they often don't have a play button to resume playback.

BUG=None
==========

to

==========
Don't pause suspended videos with no audio.

This came up in some earlier code review.
The purpose of the timer is to pause audible backgrounded videos so they
don't surprise user when the tab is foregrounded. Video only players don't
have this problem and they often don't have a play button to resume playback.

BUG=None

Review-Url: https://codereview.chromium.org/2696423003
Cr-Commit-Position: refs/heads/master@{#455224}
Committed:
https://chromium.googlesource.com/chromium/src/+/52efd28a984d8b4f5afdd9a3a6be...
==========

[commit-bot: I haz the power, 2017-03-07 21:18:32]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/52efd28a984d8b4f5afdd9a3a6be...