[LayoutNG] Make NGBlockLayoutAlgorithm accept a NGBlockNode.

[LayoutNG] Make NGBlockLayoutAlgorithm accept a NGBlockNode.

Instead of the LayoutObject, Style, FirstChild tuple.

This is so that NGFragmentBuilder can access the node to construct a
break token correctly.

See prototype multi-col patch: crrev/2693193002.

BUG=635619
Review-Url: https://codereview.chromium.org/2692403003
Cr-Commit-Position: refs/heads/master@{#450843}
Committed: https://chromium.googlesource.com/chromium/src/+/a4d7e99f9043c0268d57258bb6ef1a6e206b6b1d

[ikilpatrick, 2017-02-15 01:58:59]

ikilpatrick@chromium.org changed reviewers:
+ cbiesinger@chromium.org, glebl@chromium.org

[ikilpatrick, 2017-02-15 01:58:59]

The change in the tests from stack allocated nodes to heap allocated, is that a
Persistent container has a DCHECK that it's object is heap allocated.

The more you know :).

[cbiesinger, 2017-02-15 17:33:14]

lgtm

[ikilpatrick, 2017-02-15 18:00:41]

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

[commit-bot: I haz the power, 2017-02-15 18:01:38]

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-02-15 19:42:29]

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

[commit-bot: I haz the power, 2017-02-15 19:42:30]

Dry run: This issue passed the CQ dry run.

[Gleb Lanbin, 2017-02-15 22:29:13]

lgtm

[Gleb Lanbin, 2017-02-15 22:29:14]

lgtm

[ikilpatrick, 2017-02-15 22:58:04]

The CQ bit was checked by ikilpatrick@chromium.org

[commit-bot: I haz the power, 2017-02-15 22:58:49]

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-02-16 00:45:24]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487199484296260,
"parent_rev": "5bbea159fd30c4f7f1af838e12bf5dccb39e5dae", "commit_rev":
"a4d7e99f9043c0268d57258bb6ef1a6e206b6b1d"}

[commit-bot: I haz the power, 2017-02-16 00:46:08]

Description was changed from

==========
[LayoutNG] Make NGBlockLayoutAlgorithm accept a NGBlockNode.

Instead of the LayoutObject, Style, FirstChild tuple.

This is so that NGFragmentBuilder can access the node to construct a
break token correctly.

See prototype multi-col patch: crrev/2693193002.

BUG=635619
==========

to

==========
[LayoutNG] Make NGBlockLayoutAlgorithm accept a NGBlockNode.

Instead of the LayoutObject, Style, FirstChild tuple.

This is so that NGFragmentBuilder can access the node to construct a
break token correctly.

See prototype multi-col patch: crrev/2693193002.

BUG=635619

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

[commit-bot: I haz the power, 2017-02-16 00:46:09]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/a4d7e99f9043c0268d57258bb6ef...