Ntp: adjust footer spacing.

Ntp: adjust footer spacing.

BUG=643104
Committed: https://crrev.com/480feb5b4f17cca6a327cf1640ca318b64a92216
Cr-Commit-Position: refs/heads/master@{#416225}

[Michael van Ouwerkerk, 2016-09-01 13:37:19]

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

[commit-bot: I haz the power, 2016-09-01 13:37:39]

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

[Michael van Ouwerkerk, 2016-09-01 13:39:44]

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

[Michael van Ouwerkerk, 2016-09-01 13:39:45]

Peter, could you take a look please?

[commit-bot: I haz the power, 2016-09-01 14:29:37]

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

[commit-bot: I haz the power, 2016-09-01 14:29:38]

Dry run: This issue passed the CQ dry run.

[PEConn, 2016-09-01 15:38:11]

https://codereview.chromium.org/2301023002/diff/1/chrome/android/java/res/val...
File chrome/android/java/res/values/dimens.xml (right):

https://codereview.chromium.org/2301023002/diff/1/chrome/android/java/res/val...
chrome/android/java/res/values/dimens.xml:280: <dimen
name="ntp_min_bottom_spacing_recycler_view">0dp</dimen>
Do we need this anymore?

[Michael van Ouwerkerk, 2016-09-01 15:53:07]

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

[commit-bot: I haz the power, 2016-09-01 15:53:24]

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

[Michael van Ouwerkerk, 2016-09-01 15:54:33]

Thanks Peter. Please take another look.

https://codereview.chromium.org/2301023002/diff/1/chrome/android/java/res/val...
File chrome/android/java/res/values/dimens.xml (right):

https://codereview.chromium.org/2301023002/diff/1/chrome/android/java/res/val...
chrome/android/java/res/values/dimens.xml:280: <dimen
name="ntp_min_bottom_spacing_recycler_view">0dp</dimen>
On 2016/09/01 15:38:11, PEConn wrote:
> Do we need this anymore?

We don't. Gone.

[PEConn, 2016-09-01 15:56:03]

LGTM

[Michael van Ouwerkerk, 2016-09-01 16:07:24]

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

[Michael van Ouwerkerk, 2016-09-01 16:07:24]

Dan: please review as OWNER for files in chrome/android/java/res/

[commit-bot: I haz the power, 2016-09-01 16:29:18]

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

[commit-bot: I haz the power, 2016-09-01 16:29:19]

Dry run: This issue passed the CQ dry run.

[gone, 2016-09-01 17:33:44]

OWNERS lgtm

[Michael van Ouwerkerk, 2016-09-02 09:38:32]

The CQ bit was checked by mvanouwerkerk@chromium.org

[commit-bot: I haz the power, 2016-09-02 09:38:46]

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, 2016-09-02 09:42:25]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-02 09:44:47]

Description was changed from

==========
Ntp: adjust footer spacing.

BUG=643104
==========

to

==========
Ntp: adjust footer spacing.

BUG=643104

Committed: https://crrev.com/480feb5b4f17cca6a327cf1640ca318b64a92216
Cr-Commit-Position: refs/heads/master@{#416225}
==========

[commit-bot: I haz the power, 2016-09-02 09:44:48]

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