[layoutng] Use the override width even when we're not a flex item

[layoutng] Use the override width even when we're not a flex item

(And also use it for positioned boxes)

We want to use this for layoutng/legacy interop. For legacy layout,
there should be no change in behavior because flexbox is currently
the only caller of this function:
https://cs.chromium.org/search/?q=setOverrideLogicalContentWidth&sq=package:chromium&type=cs

Interestingly, the height version did not have this limitation already.

This patch also deleted an outdated FIXME comment.

R=atotic@chromium.org,eae@chromium.org
BUG=635619
Review-Url: https://codereview.chromium.org/2624143005
Cr-Commit-Position: refs/heads/master@{#443056}
Committed: https://chromium.googlesource.com/chromium/src/+/513f1f02c5c928367006c600cd1098f6315654ca

[cbiesinger, 2017-01-11 21:44:38]

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

[commit-bot: I haz the power, 2017-01-11 21:45:17]

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

[eae, 2017-01-11 22:03:33]

LGTM

[cbiesinger, 2017-01-11 22:08:31]

Description was changed from

==========
[layoutng] Use the override width even when we're not a flex item

We want to use this for layoutng/legacy interop. For legacy layout,
there should be no change in behavior because flexbox is currently
the only caller of this function:
https://cs.chromium.org/search/?q=setOverrideLogicalContentWidth&sq=package:c...

Interestingly, the height version did not have this limitation already.

This patch also deleted an outdated FIXME comment.

R=atotic@chromium.org,eae@chromium.org
BUG=635619
==========

to

==========
[layoutng] Use the override width even when we're not a flex item

(And also use it for positioned boxes)

We want to use this for layoutng/legacy interop. For legacy layout,
there should be no change in behavior because flexbox is currently
the only caller of this function:
https://cs.chromium.org/search/?q=setOverrideLogicalContentWidth&sq=package:c...

Interestingly, the height version did not have this limitation already.

This patch also deleted an outdated FIXME comment.

R=atotic@chromium.org,eae@chromium.org
BUG=635619
==========

[cbiesinger, 2017-01-11 22:08:35]

The CQ bit was unchecked by cbiesinger@chromium.org

[cbiesinger, 2017-01-11 22:08:44]

The CQ bit was checked by cbiesinger@chromium.org

[commit-bot: I haz the power, 2017-01-11 22:10:37]

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

[atotic, 2017-01-11 22:46:07]

lgtm

[commit-bot: I haz the power, 2017-01-11 23:44:14]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484172524349200, "parent_rev":
"ff407634505cc3c6203ed2f746000b6d6eea79b6", "commit_rev":
"513f1f02c5c928367006c600cd1098f6315654ca"}

[commit-bot: I haz the power, 2017-01-11 23:45:49]

Description was changed from

==========
[layoutng] Use the override width even when we're not a flex item

(And also use it for positioned boxes)

We want to use this for layoutng/legacy interop. For legacy layout,
there should be no change in behavior because flexbox is currently
the only caller of this function:
https://cs.chromium.org/search/?q=setOverrideLogicalContentWidth&sq=package:c...

Interestingly, the height version did not have this limitation already.

This patch also deleted an outdated FIXME comment.

R=atotic@chromium.org,eae@chromium.org
BUG=635619
==========

to

==========
[layoutng] Use the override width even when we're not a flex item

(And also use it for positioned boxes)

We want to use this for layoutng/legacy interop. For legacy layout,
there should be no change in behavior because flexbox is currently
the only caller of this function:
https://cs.chromium.org/search/?q=setOverrideLogicalContentWidth&sq=package:c...

Interestingly, the height version did not have this limitation already.

This patch also deleted an outdated FIXME comment.

R=atotic@chromium.org,eae@chromium.org
BUG=635619

Review-Url: https://codereview.chromium.org/2624143005
Cr-Commit-Position: refs/heads/master@{#443056}
Committed:
https://chromium.googlesource.com/chromium/src/+/513f1f02c5c928367006c600cd10...
==========

[commit-bot: I haz the power, 2017-01-11 23:45:50]

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