Plugin Power Saver: Implement UI by Chrome designers.

Plugin Power Saver: Implement UI by Chrome designers.

This patch:

a) Implements the new UI by our Chrome designers for PPS.
b) No longer tiles the poster image in case it's the wrong size. Now it centers and 'contain's it.

BUG=443810, 403800
Committed: https://crrev.com/8543891ac05c45e247da267d43d8926bafc1c95f
Cr-Commit-Position: refs/heads/master@{#315601}

[tommycli, 2015-02-10 00:03:47]

tommycli@chromium.org changed reviewers:
+ sgabriel@chromium.org, thestig@chromium.org

[tommycli, 2015-02-10 00:04:54]

tommycli@chromium.org changed reviewers:
- sgabriel@chromium.org

[tommycli, 2015-02-10 00:04:55]

thestig: ptal

This is a video of what it looks like:
https://drive.google.com/open?id=0B-Use9bxumRNQnBDZFhsZUZBMlk

[tommycli, 2015-02-10 00:05:20]

thestig: Sebastien Gabriel is the designer. I also linked the video to him.

[tommycli, 2015-02-10 02:31:11]

bauerb: Designer asked me to consider some CSS transitions on the opacity.

I noticed that transitions DO NOT work on WebViewPlugin, while the same code
works on jsfiddle.

Thoughts?

[Lei Zhang, 2015-02-10 03:31:10]

If Sebastien is happy with tihs, then lgtm.

[Bernhard Bauer, 2015-02-10 09:08:15]

bauerb@chromium.org changed reviewers:
+ bauerb@chromium.org

[Bernhard Bauer, 2015-02-10 09:08:16]

LGTM with some opportunities to clean up my messes below:

(No idea about the animations, BTW)

https://codereview.chromium.org/896133005/diff/1/chrome/renderer/resources/pl...
File chrome/renderer/resources/plugin_poster.html (right):

https://codereview.chromium.org/896133005/diff/1/chrome/renderer/resources/pl...
chrome/renderer/resources/plugin_poster.html:44: <div
i18n-values="title:name;.style.background-image:background" id="outer"
onclick="plugin.load()">
Can you break this line to make it fit into 80 columns?

https://codereview.chromium.org/896133005/diff/1/chrome/renderer/resources/pl...
chrome/renderer/resources/plugin_poster.html:48: <p id="debug"> </p>
While you're making changes, can you remove this line? I originally added it for
debugging purposes (I think console.log doesn't work inside of the inner
WebView), but it requires logging code anyway, so whoever needs it can just add
it back in in a local build.

[tommycli, 2015-02-10 17:54:34]

New patchsets have been uploaded after l-g-t-m from
thestig@chromium.org,bauerb@chromium.org

[tommycli, 2015-02-10 17:56:11]

bauerb: Thanks!

https://codereview.chromium.org/896133005/diff/1/chrome/renderer/resources/pl...
File chrome/renderer/resources/plugin_poster.html (right):

https://codereview.chromium.org/896133005/diff/1/chrome/renderer/resources/pl...
chrome/renderer/resources/plugin_poster.html:44: <div
i18n-values="title:name;.style.background-image:background" id="outer"
onclick="plugin.load()">
On 2015/02/10 09:08:16, Bernhard Bauer wrote:
> Can you break this line to make it fit into 80 columns?

Done.

https://codereview.chromium.org/896133005/diff/1/chrome/renderer/resources/pl...
chrome/renderer/resources/plugin_poster.html:48: <p id="debug"> </p>
On 2015/02/10 09:08:16, Bernhard Bauer wrote:
> While you're making changes, can you remove this line? I originally added it
for
> debugging purposes (I think console.log doesn't work inside of the inner
> WebView), but it requires logging code anyway, so whoever needs it can just
add
> it back in in a local build.

Done.

[tommycli, 2015-02-10 18:01:05]

The CQ bit was checked by tommycli@chromium.org

[commit-bot: I haz the power, 2015-02-10 18:01:36]

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

[commit-bot: I haz the power, 2015-02-10 18:50:25]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-10 18:51:34]

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