[LayoutNG] Use NGLogicalSize in NGBlockLayoutAlgorithm

[LayoutNG] Use NGLogicalSize in NGBlockLayoutAlgorithm

Try to simplify coordinate handling in NGBlockLayoutAlgorithm::Layout by
using NGLogicalSize instead of raw LayoutUnits for the block/inline dir.

R=glebl@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng

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

[eae, 2017-04-10 14:52:08]

Description was changed from

==========
[LayoutNG] Use NGLogicalSize in NGBlockLayoutAlgorithm

Try to simplify coordinate handling in NGBlockLayoutAlgorithm::Layout by
using NGLogicalSize instead of raw LayoutUnits for the block/inline dir.

R=glebl@chromium.org
==========

to

==========
[LayoutNG] Use NGLogicalSize in NGBlockLayoutAlgorithm

Try to simplify coordinate handling in NGBlockLayoutAlgorithm::Layout by
using NGLogicalSize instead of raw LayoutUnits for the block/inline dir.

R=glebl@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng
==========

[eae, 2017-04-10 14:53:06]

When merged with https://codereview.chromium.org/2807113002/ it is further
simplified as that allows the full size to be set directly.

[eae, 2017-04-10 14:53:10]

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

[commit-bot: I haz the power, 2017-04-10 14:53:23]

Dry run: 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-04-10 16:18:41]

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

[commit-bot: I haz the power, 2017-04-10 16:18:42]

Dry run: This issue passed the CQ dry run.

[Gleb Lanbin, 2017-04-10 16:31:43]

lgtm

[eae, 2017-04-10 18:57:44]

The CQ bit was checked by eae@chromium.org

[commit-bot: I haz the power, 2017-04-10 18:58:30]

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-04-10 19:18:18]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491850664409100, "parent_rev":
"85be9792615f94cd01c32c7232314e969ff03880", "commit_rev":
"f768bcf3d097e89d75c394e9e39e3139f87348fe"}

[commit-bot: I haz the power, 2017-04-10 19:19:27]

Description was changed from

==========
[LayoutNG] Use NGLogicalSize in NGBlockLayoutAlgorithm

Try to simplify coordinate handling in NGBlockLayoutAlgorithm::Layout by
using NGLogicalSize instead of raw LayoutUnits for the block/inline dir.

R=glebl@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng
==========

to

==========
[LayoutNG] Use NGLogicalSize in NGBlockLayoutAlgorithm

Try to simplify coordinate handling in NGBlockLayoutAlgorithm::Layout by
using NGLogicalSize instead of raw LayoutUnits for the block/inline dir.

R=glebl@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng

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

[commit-bot: I haz the power, 2017-04-10 19:19:30]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/f768bcf3d097e89d75c394e9e39e...

[cbiesinger, 2017-04-10 20:59:37]

thanks for this change! lgtm