Notifications custom layouts: make the buttons look good in Material.

Notifications custom layouts: make the buttons look good in Material.

* Tweak the spacing to be identical to the standard layout
* Tweak the ripple to look good

BUG=541617
Committed: https://crrev.com/b0e8235529763657e30904be1cb34ec979495291
Cr-Commit-Position: refs/heads/master@{#357074}

[Michael van Ouwerkerk, 2015-10-29 17:01:31]

mvanouwerkerk@chromium.org changed reviewers:
+ newt@chromium.org, peter@chromium.org

[Michael van Ouwerkerk, 2015-10-29 17:01:32]

Peter, Newton, can you take a look please?

[Peter Beverloo, 2015-10-29 17:33:28]

lgtm

https://codereview.chromium.org/1418813010/diff/1/chrome/android/java/res/lay...
File chrome/android/java/res/layout/web_notification_big.xml (right):

https://codereview.chromium.org/1418813010/diff/1/chrome/android/java/res/lay...
chrome/android/java/res/layout/web_notification_big.xml:96: <ImageView
Is the 14dp reduction in height enough to make the footer visible?

[Michael van Ouwerkerk, 2015-10-29 17:41:07]

https://codereview.chromium.org/1418813010/diff/1/chrome/android/java/res/lay...
File chrome/android/java/res/layout/web_notification_big.xml (right):

https://codereview.chromium.org/1418813010/diff/1/chrome/android/java/res/lay...
chrome/android/java/res/layout/web_notification_big.xml:96: <ImageView
On 2015/10/29 17:33:28, Peter Beverloo wrote:
> Is the 14dp reduction in height enough to make the footer visible?

Yes, together with the timestamp spacing CL
(https://codereview.chromium.org/1418453003/) the layout is identical now in
Material.

[newt (away), 2015-10-29 20:43:05]

lgtm

[Michael van Ouwerkerk, 2015-10-30 09:48:54]

The CQ bit was checked by mvanouwerkerk@chromium.org

[commit-bot: I haz the power, 2015-10-30 09:49:14]

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

[commit-bot: I haz the power, 2015-10-30 10:29:09]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-10-30 10:29:58]

Patchset 1 (id:??) landed as
https://crrev.com/b0e8235529763657e30904be1cb34ec979495291
Cr-Commit-Position: refs/heads/master@{#357074}