[LayoutNG] Don't use the margin-strut in the OOF static-position if zero-height.

[LayoutNG] Don't use the margin-strut in the OOF static-position if zero-height.

This patch does two things:

 - Doesn't include the margin strut in the OOF static-position if we are
   currently going to be a zero-height-div.

   This gives a static block offset of 0 relative to its immediate
   parent, which is correct.

 - Positions zero-height fragments using *their* end margin strut. Not
   the previous margin strut.

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

[ikilpatrick, 2017-05-05 20:06:28]

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

[ikilpatrick, 2017-05-05 20:06:39]

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

[commit-bot: I haz the power, 2017-05-05 20:07:19]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Gleb Lanbin, 2017-05-05 20:26:26]

lgtm

[ikilpatrick, 2017-05-05 21:35:29]

The CQ bit was unchecked by ikilpatrick@chromium.org

[ikilpatrick, 2017-05-05 21:35:33]

The CQ bit was checked by ikilpatrick@chromium.org

[commit-bot: I haz the power, 2017-05-05 21:35:39]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[ikilpatrick, 2017-05-05 21:59:04]

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

[commit-bot: I haz the power, 2017-05-05 21:59:46]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[ikilpatrick, 2017-05-05 23:05:43]

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

[commit-bot: I haz the power, 2017-05-05 23:06:16]

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-05-06 00:53:29]

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

[commit-bot: I haz the power, 2017-05-06 00:53:29]

Dry run: This issue passed the CQ dry run.

[ikilpatrick, 2017-05-06 05:11:47]

The CQ bit was checked by ikilpatrick@chromium.org

[ikilpatrick, 2017-05-06 05:11:47]

The patchset sent to the CQ was uploaded after l-g-t-m from glebl@chromium.org
Link to the patchset: https://codereview.chromium.org/2868523002/#ps60001
(title: "rebase.")

[commit-bot: I haz the power, 2017-05-06 05:12:23]

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-05-06 19:21:02]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1494047507357260,
"parent_rev": "5a006e9c8d2f5049f39f49a9280cc6cebb1a4ec3", "commit_rev":
"32b0510d7d81ee6bb988d9283336320306debbdf"}

[commit-bot: I haz the power, 2017-05-06 19:21:17]

Description was changed from

==========
[LayoutNG] Don't use the margin-strut in the OOF static-position if zero-height.

This patch does two things:

 - Doesn't include the margin strut in the OOF static-position if we are
   currently going to be a zero-height-div.

   This gives a static block offset of 0 relative to its immediate
   parent, which is correct.

 - Positions zero-height fragments using *their* end margin strut. Not
   the previous margin strut.

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

to

==========
[LayoutNG] Don't use the margin-strut in the OOF static-position if zero-height.

This patch does two things:

 - Doesn't include the margin strut in the OOF static-position if we are
   currently going to be a zero-height-div.

   This gives a static block offset of 0 relative to its immediate
   parent, which is correct.

 - Positions zero-height fragments using *their* end margin strut. Not
   the previous margin strut.

BUG=635619

Review-Url: https://codereview.chromium.org/2868523002
Cr-Commit-Position: refs/heads/master@{#469882}
Committed:
https://chromium.googlesource.com/chromium/src/+/32b0510d7d81ee6bb988d9283336...
==========

[commit-bot: I haz the power, 2017-05-06 19:21:18]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/32b0510d7d81ee6bb988d9283336...