[layoutng] Make sure to update m_isSelfCollapsing after ng layout

[layoutng] Make sure to update m_isSelfCollapsing after ng layout

This will otherwise cause assertion failures on many pages.

We could also compute this in ng code during margin collapsing and just set it
on the old object, but I'm not sure that would be worth it.

This and https://codereview.chromium.org/2337763005/ fixes most (but not all)
of the assertion failures during legacy/ng interop.

R=ikilpatrick@chromium.org,eae@chromium.org,glebl@chromium.org
BUG=635619
Committed: https://crrev.com/2eacf266f015eca06c70449f397d973bad57ae68
Cr-Commit-Position: refs/heads/master@{#418703}

[cbiesinger, 2016-09-14 21:02:21]

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

[commit-bot: I haz the power, 2016-09-14 21:02:57]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[ikilpatrick, 2016-09-14 21:04:45]

lgtm

[cbiesinger, 2016-09-14 21:16:55]

The CQ bit was unchecked by cbiesinger@chromium.org

[cbiesinger, 2016-09-14 21:16:58]

The CQ bit was checked by cbiesinger@chromium.org

[commit-bot: I haz the power, 2016-09-14 21:17:21]

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-09-14 22:44:17]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-14 22:46:22]

Description was changed from

==========
[layoutng] Make sure to update m_isSelfCollapsing after ng layout

This will otherwise cause assertion failures on many pages.

We could also compute this in ng code during margin collapsing and just set it
on the old object, but I'm not sure that would be worth it.

This and https://codereview.chromium.org/2337763005/ fixes most (but not all)
of the assertion failures during legacy/ng interop.

R=ikilpatrick@chromium.org,eae@chromium.org,glebl@chromium.org
BUG=635619
==========

to

==========
[layoutng] Make sure to update m_isSelfCollapsing after ng layout

This will otherwise cause assertion failures on many pages.

We could also compute this in ng code during margin collapsing and just set it
on the old object, but I'm not sure that would be worth it.

This and https://codereview.chromium.org/2337763005/ fixes most (but not all)
of the assertion failures during legacy/ng interop.

R=ikilpatrick@chromium.org,eae@chromium.org,glebl@chromium.org
BUG=635619

Committed: https://crrev.com/2eacf266f015eca06c70449f397d973bad57ae68
Cr-Commit-Position: refs/heads/master@{#418703}
==========

[commit-bot: I haz the power, 2016-09-14 22:46:24]

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

[eae, 2016-09-15 09:26:16]

Nice! LGTM