harmony: use Harmony sizing for ViewsDelegate sizing functions

harmony: use Harmony sizing for ViewsDelegate sizing functions

BUG=635173
Committed: https://crrev.com/50ef64ec787d2d721df865f74c22596e2ac0601e
Cr-Commit-Position: refs/heads/master@{#433946}

[Elly Fong-Jones, 2016-11-17 14:36:34]

Description was changed from

==========
harmony: use Harmony sizing for ViewsDelegate sizing functions

BUG=635173
==========

to

==========
harmony: use Harmony sizing for ViewsDelegate sizing functions

BUG=635173
==========

[Elly Fong-Jones, 2016-11-17 14:36:34]

ellyjones@chromium.org changed reviewers:
+ sky@chromium.org

[Elly Fong-Jones, 2016-11-17 14:36:40]

sky: ptal? :)

[sky, 2016-11-17 18:18:34]

LGTM

https://codereview.chromium.org/2505973003/diff/1/chrome/browser/ui/views/har...
File chrome/browser/ui/views/harmony/harmony_layout_delegate.h (right):

https://codereview.chromium.org/2505973003/diff/1/chrome/browser/ui/views/har...
chrome/browser/ui/views/harmony/harmony_layout_delegate.h:19: static const int
kHarmonyLayoutUnit = 16;
style guide says constants should be before constructor/destructor.

[Elly Fong-Jones, 2016-11-22 15:46:59]

The CQ bit was checked by ellyjones@chromium.org

[Elly Fong-Jones, 2016-11-22 15:46:59]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/2505973003/#ps20001
(title: "move constant")

[commit-bot: I haz the power, 2016-11-22 15:47:13]

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-11-22 18:12:53]

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

[commit-bot: I haz the power, 2016-11-22 18:12:54]

Internal error: failed to checkout. Please try again.

[Elly Fong-Jones, 2016-11-22 19:37:08]

The CQ bit was checked by ellyjones@chromium.org

[commit-bot: I haz the power, 2016-11-22 19:37:40]

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-11-22 19:48:16]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1479843428014850,
"parent_rev": "8537207eef79ce5759bb156d1dc66adf62705a72", "commit_rev":
"9603e2c406b1f2d7ef049bdd1d8a2455ab645b29"}

[commit-bot: I haz the power, 2016-11-22 19:48:54]

Description was changed from

==========
harmony: use Harmony sizing for ViewsDelegate sizing functions

BUG=635173
==========

to

==========
harmony: use Harmony sizing for ViewsDelegate sizing functions

BUG=635173
==========

[commit-bot: I haz the power, 2016-11-22 19:48:55]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-22 19:52:42]

Description was changed from

==========
harmony: use Harmony sizing for ViewsDelegate sizing functions

BUG=635173
==========

to

==========
harmony: use Harmony sizing for ViewsDelegate sizing functions

BUG=635173

Committed: https://crrev.com/50ef64ec787d2d721df865f74c22596e2ac0601e
Cr-Commit-Position: refs/heads/master@{#433946}
==========

[commit-bot: I haz the power, 2016-11-22 19:52:43]

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