[Android, Video, Autoplay] Only autoplay muted videos, ignore audio tags.

[Android, Video, Autoplay] Only autoplay muted videos, ignore audio tags.

Added a test for audio muted autoplay

BUG=619969
TEST=manual + layout tests

Committed: https://crrev.com/14b944fc8d348a17eaf24dd7a8b66662bf3d1ca8
Cr-Commit-Position: refs/heads/master@{#400149}

[whywhat, 2016-06-15 09:21:33]

The CQ bit was checked by avayvod@chromium.org to run a CQ dry run

[whywhat, 2016-06-15 09:21:44]

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

[whywhat, 2016-06-15 09:21:44]

PTaL

[commit-bot: I haz the power, 2016-06-15 09:21:46]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2065363002/1

[commit-bot: I haz the power, 2016-06-15 11:03:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-15 11:03:54]

Dry run: This issue passed the CQ dry run.

[mlamouri (slow - plz ping), 2016-06-15 20:29:12]

lgtm with test fix.

Thanks! \o/

https://codereview.chromium.org/2065363002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/media/autoplay-muted.html (right):

https://codereview.chromium.org/2065363002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/autoplay-muted.html:67: }), 100);
You should avoid setTimeout() with other values than 0 (ie. no values, 0 is
default). I think it's fine to remove 100, paused should have changed.

[whywhat, 2016-06-16 12:38:43]

Changed setTimeout to 0

[whywhat, 2016-06-16 12:39:45]

The CQ bit was checked by avayvod@chromium.org

[whywhat, 2016-06-16 12:39:46]

https://codereview.chromium.org/2065363002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/media/autoplay-muted.html (right):

https://codereview.chromium.org/2065363002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/autoplay-muted.html:67: }), 100);
On 2016/06/15 at 20:29:12, Mounir Lamouri (very slow) wrote:
> You should avoid setTimeout() with other values than 0 (ie. no values, 0 is
default). I think it's fine to remove 100, paused should have changed.

Done.

[whywhat, 2016-06-16 12:39:46]

The patchset sent to the CQ was uploaded after l-g-t-m from
mlamouri@chromium.org
Link to the patchset: https://codereview.chromium.org/2065363002/#ps20001
(title: "Changed setTimeout to 0")

[commit-bot: I haz the power, 2016-06-16 12:39:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2065363002/20001

[commit-bot: I haz the power, 2016-06-16 14:13:21]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-16 14:13:24]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-16 14:14:43]

Description was changed from

==========
[Android, Video, Autoplay] Only autoplay muted videos, ignore audio tags.

Added a test for audio muted autoplay

BUG=619969
TEST=manual + layout tests
==========

to

==========
[Android, Video, Autoplay] Only autoplay muted videos, ignore audio tags.

Added a test for audio muted autoplay

BUG=619969
TEST=manual + layout tests

Committed: https://crrev.com/14b944fc8d348a17eaf24dd7a8b66662bf3d1ca8
Cr-Commit-Position: refs/heads/master@{#400149}
==========

[commit-bot: I haz the power, 2016-06-16 14:14:44]

Patchset 2 (id:??) landed as
https://crrev.com/14b944fc8d348a17eaf24dd7a8b66662bf3d1ca8
Cr-Commit-Position: refs/heads/master@{#400149}