[Blink>Media] Allow autoplay muted on Android by default

[Blink>Media] Allow autoplay muted on Android by default

There was a mistake causing autoplay muted is shipped on Android
but it will be disabled if the chromium embedder doesn't specify
content setting for "AllowAutoplay" preference. This CL makes the
AllowAutoplay preference true by default so that it is allowed by
embedders (including AndroidWebView) unless they explicitly
disable it.

Intent to ship:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/Q1cnzNI2GpI/AL_eyUNABgAJ

BUG=689018
Review-Url: https://codereview.chromium.org/2677173002
Cr-Commit-Position: refs/heads/master@{#448423}
Committed: https://chromium.googlesource.com/chromium/src/+/faaa2fd0a05f1622d9a8806da118d4f3b602e707

[Zhiqiang Zhang (Slow), 2017-02-06 12:44:01]

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

[mlamouri (slow - plz ping), 2017-02-06 12:47:58]

I would rename the CL title to not be WebView specific but be about Blink
embedders. Basically, it enables autoplay muted by default unless the embedders
disables it.

[mlamouri (slow - plz ping), 2017-02-06 12:48:04]

oh, and lgtm :)

[Zhiqiang Zhang (Slow), 2017-02-06 14:08:19]

Description was changed from

==========
[Blink>Media] Allow autoplay muted on Android WebView

There was a mistake causing autoplay muted is shipped on Android but not Android
WebView. This CL fixes the issue.

Intent to ship:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/Q1cnzNI2GpI/AL_eyUNA...

BUG=689018
==========

to

==========
[Blink>Media] Allow autoplay muted on Android by default

There was a mistake causing autoplay muted is shipped on Android
but it will be disabled if the chromium embedder doesn't specify
content setting for "AllowAutoplay" preference. This CL makes the
AllowAutoplay preference true by default so that it is allowed by
embedders (including AndroidWebView) unless they explicitly
disable it.

Intent to ship:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/Q1cnzNI2GpI/AL_eyUNA...

BUG=689018
==========

[Zhiqiang Zhang (Slow), 2017-02-06 14:08:28]

On 2017/02/06 12:47:58, mlamouri wrote:
> I would rename the CL title to not be WebView specific but be about Blink
> embedders. Basically, it enables autoplay muted by default unless the
embedders
> disables it.

Done.

[Zhiqiang Zhang (Slow), 2017-02-06 14:08:33]

The CQ bit was checked by zqzhang@chromium.org

[commit-bot: I haz the power, 2017-02-06 14:08:54]

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-02-06 15:34:43]

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

[commit-bot: I haz the power, 2017-02-06 15:34:45]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Zhiqiang Zhang (Slow), 2017-02-06 15:55:02]

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

[commit-bot: I haz the power, 2017-02-06 15:55:24]

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

[Zhiqiang Zhang (Slow), 2017-02-06 16:56:33]

zqzhang@chromium.org changed reviewers:
+ dpranke@chromium.org

[Zhiqiang Zhang (Slow), 2017-02-06 16:56:33]

+dpranke: mock_content_settings_client.cc

[commit-bot: I haz the power, 2017-02-06 17:41:44]

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

[commit-bot: I haz the power, 2017-02-06 17:41:46]

Dry run: This issue passed the CQ dry run.

[Dirk Pranke, 2017-02-06 20:54:30]

lgtm

[Zhiqiang Zhang (Slow), 2017-02-06 22:40:29]

The CQ bit was checked by zqzhang@chromium.org

[Zhiqiang Zhang (Slow), 2017-02-06 22:40:30]

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/2677173002/#ps20001
(title: "fixed layout tests")

[commit-bot: I haz the power, 2017-02-06 22:41:23]

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-02-06 22:48:46]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1486420829932130,
"parent_rev": "c5e88f0990ad395ae9c078903099ac02af08ea01", "commit_rev":
"faaa2fd0a05f1622d9a8806da118d4f3b602e707"}

[commit-bot: I haz the power, 2017-02-06 22:49:19]

Description was changed from

==========
[Blink>Media] Allow autoplay muted on Android by default

There was a mistake causing autoplay muted is shipped on Android
but it will be disabled if the chromium embedder doesn't specify
content setting for "AllowAutoplay" preference. This CL makes the
AllowAutoplay preference true by default so that it is allowed by
embedders (including AndroidWebView) unless they explicitly
disable it.

Intent to ship:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/Q1cnzNI2GpI/AL_eyUNA...

BUG=689018
==========

to

==========
[Blink>Media] Allow autoplay muted on Android by default

There was a mistake causing autoplay muted is shipped on Android
but it will be disabled if the chromium embedder doesn't specify
content setting for "AllowAutoplay" preference. This CL makes the
AllowAutoplay preference true by default so that it is allowed by
embedders (including AndroidWebView) unless they explicitly
disable it.

Intent to ship:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/Q1cnzNI2GpI/AL_eyUNA...

BUG=689018

Review-Url: https://codereview.chromium.org/2677173002
Cr-Commit-Position: refs/heads/master@{#448423}
Committed:
https://chromium.googlesource.com/chromium/src/+/faaa2fd0a05f1622d9a8806da118...
==========

[commit-bot: I haz the power, 2017-02-06 22:49:20]

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