Enable new media playback UI for clank.

Enable new media playback UI for clank.

This CL enables the new media UI for clank, and the associated
clank-specific behavior to hide volme controls more often to
save screen space.  Other platforms should not be affected.

To revert to the original UI, please revert only this commit.

The new UI code itself can be found in
https://codereview.chromium.org/1156993013 .  The default behavior
of that CL, without this one, will be the original UI for all
platforms, including clank.

BUG=446350, 488625, 487344
Committed: https://crrev.com/2f1a7e0ebd63cbaebba6f72f65df7a9d2557dd09
Cr-Commit-Position: refs/heads/master@{#342627}

[liberato (no reviews please), 2015-08-05 16:29:19]

liberato@chromium.org changed reviewers:
+ jochen@chromium.org

[liberato (no reviews please), 2015-08-05 16:43:05]

Please let me know if there's a better way to enable the new media playback UI
for clank only.

It was much easier when it was only a WebKit change.  I suppose that I could
#ifdef it there, to atomically enable and set test expectations for android. 
i'm not sure which subset we actually run, or if any of them will break.

xhwang (cc'd) was kind enough to help find out what subset it is.

thanks
-fl

[jochen (gone - plz use gerrit), 2015-08-06 07:44:25]

please send an intent to ship to blink-dev@ before we can land this cl

https://codereview.chromium.org/1266813002/diff/20001/content/renderer/render...
File content/renderer/render_view_impl.cc (right):

https://codereview.chromium.org/1266813002/diff/20001/content/renderer/render...
content/renderer/render_view_impl.cc:1165:
WebRuntimeFeatures::enableNewMediaPlaybackUi(true);
this should go into content/child/runtime_features.cc

[liberato (no reviews please), 2015-08-06 14:09:39]

there was an intent to implement and ship:

https://groups.google.com/a/chromium.org/forum/#!searchin/blink-dev/liberato/...

or does this require a separate ship-only?  i apologize in case i'm getting
things a bit muddled.

thanks
-fl

https://codereview.chromium.org/1266813002/diff/20001/content/renderer/render...
File content/renderer/render_view_impl.cc (right):

https://codereview.chromium.org/1266813002/diff/20001/content/renderer/render...
content/renderer/render_view_impl.cc:1165:
WebRuntimeFeatures::enableNewMediaPlaybackUi(true);
On 2015/08/06 07:44:25, jochen wrote:
> this should go into content/child/runtime_features.cc

Done.

[jochen (gone - plz use gerrit), 2015-08-06 14:13:10]

On 2015/08/06 at 14:09:39, liberato wrote:
> there was an intent to implement and ship:
> 
>
https://groups.google.com/a/chromium.org/forum/#!searchin/blink-dev/liberato/...
> 
> or does this require a separate ship-only?  i apologize in case i'm getting
things a bit muddled.
> 

ah, that's the right thread, but it needs 3 lgtms. mind pinging it again?

[jochen (gone - plz use gerrit), 2015-08-06 14:33:36]

lgtm

[liberato (no reviews please), 2015-08-06 20:20:37]

The CQ bit was checked by liberato@chromium.org

[commit-bot: I haz the power, 2015-08-06 20:21:26]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1266813002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1266813002/40001

[commit-bot: I haz the power, 2015-08-06 20:34:00]

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

[commit-bot: I haz the power, 2015-08-06 20:34:01]

Try jobs failed on following builders:
  android_chromium_gn_compile_dbg on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/android_chromiu...)
  android_chromium_gn_compile_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/android_chromiu...)
  linux_android_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_android_r...)

[liberato (no reviews please), 2015-08-10 15:57:24]

The CQ bit was checked by liberato@chromium.org

[liberato (no reviews please), 2015-08-10 15:57:25]

The patchset sent to the CQ was uploaded after l-g-t-m from jochen@chromium.org
Link to the patchset: https://codereview.chromium.org/1266813002/#ps60001
(title: "rebased.")

[commit-bot: I haz the power, 2015-08-10 15:57:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1266813002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1266813002/60001

[commit-bot: I haz the power, 2015-08-10 16:01:49]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-08-10 16:02:18]

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