Calc overflow on positioned descendants of tables

Calc overflow on the positioned descendants of tables

Tables can be the containing block for positioned objects too, so when we take over
the overflow calc duties from LayoutBlock we need to cater for positioned objects
like it does too.

I also spotted an error in LayoutTable::recalcChildOverflowAfterStyleChange() - we
need to recalc overflow for all sections regardless of whether a section has already
changed its overflow.

BUG=619510
Committed: https://crrev.com/488de59908c1faab11124da600aabd5708df6e75
Cr-Commit-Position: refs/heads/master@{#402626}

[rhogan, 2016-06-28 17:55:31]

Description was changed from

==========
Calc overflow on positioned descendants of tables

BUG=619510
==========

to

==========
Calc overflow on the positioned descendants of tables

Tables can be the containing block for positioned objects too, so when we take
over
the overflow calc duties from LayoutBlock we need to cater for positioned
objects
like it does too.

I also spotted an error in LayoutTable::recalcChildOverflowAfterStyleChange() -
we need to recalc overflow for all sections regardless of whether a section has
already
changed its overflow.

BUG=619510
==========

[rhogan, 2016-06-28 17:55:42]

Description was changed from

==========
Calc overflow on the positioned descendants of tables

Tables can be the containing block for positioned objects too, so when we take
over
the overflow calc duties from LayoutBlock we need to cater for positioned
objects
like it does too.

I also spotted an error in LayoutTable::recalcChildOverflowAfterStyleChange() -
we need to recalc overflow for all sections regardless of whether a section has
already
changed its overflow.

BUG=619510
==========

to

==========
Calc overflow on the positioned descendants of tables

Tables can be the containing block for positioned objects too, so when we take
over
the overflow calc duties from LayoutBlock we need to cater for positioned
objects
like it does too.

I also spotted an error in LayoutTable::recalcChildOverflowAfterStyleChange() -
we
need to recalc overflow for all sections regardless of whether a section has
already
changed its overflow.

BUG=619510
==========

[rhogan, 2016-06-28 17:55:55]

robhogan@gmail.com changed reviewers:
+ wangxianzhu@chromium.org

[Xianzhu, 2016-06-28 18:00:00]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2016-06-28 18:00:00]

lgtm

[commit-bot: I haz the power, 2016-06-28 18:00: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, 2016-06-28 19:58:43]

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

[commit-bot: I haz the power, 2016-06-28 19:58:44]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[rhogan, 2016-06-28 21:09:39]

The CQ bit was checked by robhogan@gmail.com

[rhogan, 2016-06-28 21:09:39]

The patchset sent to the CQ was uploaded after l-g-t-m from
wangxianzhu@chromium.org
Link to the patchset: https://codereview.chromium.org/2100343002/#ps40001
(title: "bug 619510")

[commit-bot: I haz the power, 2016-06-28 21:10:46]

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-06-29 00:57:40]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-06-29 00:59:25]

Description was changed from

==========
Calc overflow on the positioned descendants of tables

Tables can be the containing block for positioned objects too, so when we take
over
the overflow calc duties from LayoutBlock we need to cater for positioned
objects
like it does too.

I also spotted an error in LayoutTable::recalcChildOverflowAfterStyleChange() -
we
need to recalc overflow for all sections regardless of whether a section has
already
changed its overflow.

BUG=619510
==========

to

==========
Calc overflow on the positioned descendants of tables

Tables can be the containing block for positioned objects too, so when we take
over
the overflow calc duties from LayoutBlock we need to cater for positioned
objects
like it does too.

I also spotted an error in LayoutTable::recalcChildOverflowAfterStyleChange() -
we
need to recalc overflow for all sections regardless of whether a section has
already
changed its overflow.

BUG=619510

Committed: https://crrev.com/488de59908c1faab11124da600aabd5708df6e75
Cr-Commit-Position: refs/heads/master@{#402626}
==========

[commit-bot: I haz the power, 2016-06-29 00:59:26]

Patchset 3 (id:??) landed as
https://crrev.com/488de59908c1faab11124da600aabd5708df6e75
Cr-Commit-Position: refs/heads/master@{#402626}