Update and unify nine-patch files for content suggestion and omnibox cards

Update and unify nine-patch files for content suggestion and omnibox cards

Screenshots: https://goo.gl/photos/GFukuPJesZCRHVaMA

Change the calculation in CardViewHolder to not depend on the exact size
of the card shadow, but get it instead from the drawable's padding.

Review-Url: https://codereview.chromium.org/2638353002
Cr-Commit-Position: refs/heads/master@{#444785}
Committed: https://chromium.googlesource.com/chromium/src/+/dfd4741a6504330ae91eea2a44f3b55eaa66430d

[Bernhard Bauer, 2017-01-18 12:19:23]

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

[commit-bot: I haz the power, 2017-01-18 12:19:35]

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

[Bernhard Bauer, 2017-01-19 10:37:43]

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

[commit-bot: I haz the power, 2017-01-19 10:37:54]

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

[Bernhard Bauer, 2017-01-19 10:38:14]

Description was changed from

==========
Update nine-patch files for content suggestion cards.

Change the calculation in CardViewHolder to not depend on the exact size
of the card shadow, but get it instead from the drawable's padding.
==========

to

==========
Update and unify nine-patch files for content suggestion and omnibox cards

Change the calculation in CardViewHolder to not depend on the exact size
of the card shadow, but get it instead from the drawable's padding.
==========

[Bernhard Bauer, 2017-01-19 10:44:18]

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

[Bernhard Bauer, 2017-01-19 10:44:20]

Michael, can you take a look (in particular at CardViewHolder)? Thanks!

[Bernhard Bauer, 2017-01-19 11:16:38]

Description was changed from

==========
Update and unify nine-patch files for content suggestion and omnibox cards

Change the calculation in CardViewHolder to not depend on the exact size
of the card shadow, but get it instead from the drawable's padding.
==========

to

==========
Update and unify nine-patch files for content suggestion and omnibox cards

Screenshots: https://goo.gl/photos/GFukuPJesZCRHVaMA

Change the calculation in CardViewHolder to not depend on the exact size
of the card shadow, but get it instead from the drawable's padding.
==========

[Michael van Ouwerkerk, 2017-01-19 11:43:14]

lgtm \o/

[Bernhard Bauer, 2017-01-19 14:45:03]

Thanks! +Matt, Ted FYI.

[Bernhard Bauer, 2017-01-19 14:45:11]

The CQ bit was checked by bauerb@chromium.org

[commit-bot: I haz the power, 2017-01-19 14:45:25]

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-01-19 14:47:32]

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

[commit-bot: I haz the power, 2017-01-19 14:47:34]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[Bernhard Bauer, 2017-01-19 17:26:37]

The CQ bit was checked by bauerb@chromium.org

[Bernhard Bauer, 2017-01-19 17:26:38]

The patchset sent to the CQ was uploaded after l-g-t-m from
mvanouwerkerk@chromium.org
Link to the patchset: https://codereview.chromium.org/2638353002/#ps100001
(title: "sync")

[commit-bot: I haz the power, 2017-01-19 17:26:59]

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-01-19 18:03:42]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1484846797370920,
"parent_rev": "4b9b580a3159188e0708392406b8ccb36725f479", "commit_rev":
"dfd4741a6504330ae91eea2a44f3b55eaa66430d"}

[commit-bot: I haz the power, 2017-01-19 18:04:16]

Description was changed from

==========
Update and unify nine-patch files for content suggestion and omnibox cards

Screenshots: https://goo.gl/photos/GFukuPJesZCRHVaMA

Change the calculation in CardViewHolder to not depend on the exact size
of the card shadow, but get it instead from the drawable's padding.
==========

to

==========
Update and unify nine-patch files for content suggestion and omnibox cards

Screenshots: https://goo.gl/photos/GFukuPJesZCRHVaMA

Change the calculation in CardViewHolder to not depend on the exact size
of the card shadow, but get it instead from the drawable's padding.

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

[commit-bot: I haz the power, 2017-01-19 18:04:17]

Committed patchset #6 (id:100001) as
https://chromium.googlesource.com/chromium/src/+/dfd4741a6504330ae91eea2a44f3...

[Theresa, 2017-01-21 00:01:12]

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

[Theresa, 2017-01-21 00:01:13]

I'm working on the new Android history page, which should have similar assets as
the NTP (or share assets where possible). It looks like the ntp_card* assets are
no longer used. Is that correct, and if so, can they be removed entirely?

[Bernhard Bauer, 2017-01-24 13:09:15]

On 2017/01/21 00:01:13, Theresa wrote:
> I'm working on the new Android history page, which should have similar assets
as
> the NTP (or share assets where possible). It looks like the ntp_card* assets
are
> no longer used. Is that correct, and if so, can they be removed entirely?

Oops, yes. Those assets should have been removed in this CL; the card_*.9.png
assets are the ones to use going forward. I've sent out
https://codereview.chromium.org/2651653004/ to delete the old assets.