[LayoutNG] Remove last NGConstraintSpace constructor.

[LayoutNG] Remove last NGConstraintSpace constructor.

The other patch is: https://codereview.chromium.org/2456973002/
Main usages was in tests and CreateFromLayoutObject.

BUG=635619
Committed: https://crrev.com/631ba4493ab55194d0ed2e2482c296f0aeadcb94
Cr-Commit-Position: refs/heads/master@{#428292}

[ikilpatrick, 2016-10-27 21:47:35]

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

[ikilpatrick, 2016-10-27 21:48:40]

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

[Gleb Lanbin, 2016-10-27 22:29:55]

https://codereview.chromium.org/2456193002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/layout/ng/ng_constraint_space_test.cc
(right):

https://codereview.chromium.org/2456193002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/layout/ng/ng_constraint_space_test.cc:18:
NGPhysicalSize(LayoutUnit(200), LayoutUnit(100)), true, false, true,
can you add comments for every bool here? and below.

[ikilpatrick, 2016-10-27 23:54:44]

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

[ikilpatrick, 2016-10-27 23:54:52]

https://codereview.chromium.org/2456193002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/layout/ng/ng_constraint_space_test.cc
(right):

https://codereview.chromium.org/2456193002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/layout/ng/ng_constraint_space_test.cc:18:
NGPhysicalSize(LayoutUnit(200), LayoutUnit(100)), true, false, true,
On 2016/10/27 22:29:54, glebl wrote:
> can you add comments for every bool here? and below.

Done.

[commit-bot: I haz the power, 2016-10-27 23:55:15]

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, 2016-10-28 01:06:35]

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

[commit-bot: I haz the power, 2016-10-28 01:06:35]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[Gleb Lanbin, 2016-10-28 04:08:50]

The CQ bit was checked by glebl@chromium.org

[Gleb Lanbin, 2016-10-28 04:08:51]

lgtm

[commit-bot: I haz the power, 2016-10-28 04:09:06]

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-10-28 05:21:53]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-28 05:24:08]

Description was changed from

==========
[LayoutNG] Remove last NGConstraintSpace constructor.

The other patch is: https://codereview.chromium.org/2456973002/
Main usages was in tests and CreateFromLayoutObject.

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

to

==========
[LayoutNG] Remove last NGConstraintSpace constructor.

The other patch is: https://codereview.chromium.org/2456973002/
Main usages was in tests and CreateFromLayoutObject.

BUG=635619

Committed: https://crrev.com/631ba4493ab55194d0ed2e2482c296f0aeadcb94
Cr-Commit-Position: refs/heads/master@{#428292}
==========

[commit-bot: I haz the power, 2016-10-28 05:24:09]

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

[cbiesinger, 2016-10-28 19:05:07]

lgtm