[Offline pages] Showing more than one row in saved offline filter

[Offline pages] Showing more than one row in saved offline filter

Saved offline filter view shows the size of offline copy below the
title of the page, which prompts for spacing between the rows.
Bug here is that resource storing dimensions was not properly
converted to a number of pixels.

Also, the size of spacing was too large (once it applied properly).

BUG=536945
Committed: https://crrev.com/dcb8fa6715d1717c8eb75c8df88edd98cb5ae655
Cr-Commit-Position: refs/heads/master@{#351591}

[fgorski, 2015-09-29 20:32:52]

fgorski@chromium.org changed reviewers:
+ aurimas@chromium.org, kkimlabs@chromium.org

[fgorski, 2015-09-29 20:32:56]

kkimlabs@chromium.org: Please review changes in enhanced bookmarks

aurimas@chromium.org: Please review changes in dimensions:
Given this is spacing between the rows, 40dp was far too much. 10 looks
reasonable, but I'll send a screenshot to UX and CC you for confirmation.

[aurimas (slooooooooow), 2015-09-29 20:37:03]

chrome/android/java/res/values/dimens.xml LGTM

[Kibeom Kim (inactive), 2015-09-29 21:09:49]

lgtm

(I guess I missed this in some code review sorry!)

[fgorski, 2015-09-30 16:51:46]

Aurimas, per UX I am updating to 16dp (you were looped in)

Kibeom, no worries... took me long to find, because I didn't know about changes
to that file, but at least I learned a bit about recycler view and warmed up
after vacation.

[fgorski, 2015-09-30 16:57:03]

The CQ bit was checked by fgorski@chromium.org

[fgorski, 2015-09-30 16:57:03]

The patchset sent to the CQ was uploaded after l-g-t-m from
aurimas@chromium.org, kkimlabs@chromium.org
Link to the patchset: https://codereview.chromium.org/1376023002/#ps20001
(title: "Updating row spacing in filter view to 16dp per UX recommendation")

[commit-bot: I haz the power, 2015-09-30 16:57:20]

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

[commit-bot: I haz the power, 2015-09-30 17:46:15]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-30 17:47:01]

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