Move all ancestor-dependent properties out of PaintLayer and into an aux. struct.

Move all ancestor-dependent properties out of PaintLayer and into an aux. struct.

This makes PaintLayer smaller, which improves performance of code, such as hit
testing, that does not need to use these ancestor-dependent compositing properties.

This patch makes hit testing on paint layers >= 15% faster. In particular, it makes
the Events/hit-test-lots-of-layers.html performance test ~15% faster. It had
recently regressed when I added an IntRect.)

BUG=659059
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Committed: https://crrev.com/8af9d0bee7cdbd2db09562939b7d33696d9a7447
Cr-Commit-Position: refs/heads/master@{#430509}

[chrishtr, 2016-11-08 00:36:38]

Description was changed from

==========
none


none


none

BUG=
==========

to

==========
none


none


none

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[chrishtr, 2016-11-08 00:39:59]

Description was changed from

==========
none


none


none

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Move all ancestor-dependent properties out of PaintLayer and into an aux.
struct.

This makes PaintLayer smaller, which improves performance of code, such as hit
testing, that does not need to use these ancestor-dependent compositing
properties.

This patch makes hit testing on paint layers >= 15% faster. In particular, it
makes
the Events/hit-test-lots-of-layers.html performances test ~15% faster. It had
recently regressed when I added an IntRect.)

BUG=659059
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[chrishtr, 2016-11-08 00:47:29]

Description was changed from

==========
Move all ancestor-dependent properties out of PaintLayer and into an aux.
struct.

This makes PaintLayer smaller, which improves performance of code, such as hit
testing, that does not need to use these ancestor-dependent compositing
properties.

This patch makes hit testing on paint layers >= 15% faster. In particular, it
makes
the Events/hit-test-lots-of-layers.html performances test ~15% faster. It had
recently regressed when I added an IntRect.)

BUG=659059
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Move all ancestor-dependent properties out of PaintLayer and into an aux.
struct.

This makes PaintLayer smaller, which improves performance of code, such as hit
testing, that does not need to use these ancestor-dependent compositing
properties.

This patch makes hit testing on paint layers >= 15% faster. In particular, it
makes
the Events/hit-test-lots-of-layers.html performance test ~15% faster. It had
recently regressed when I added an IntRect.)

BUG=659059
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[chrishtr, 2016-11-08 00:48:42]

chrishtr@chromium.org changed reviewers:
+ wangxianzhu@chromium.org

[chrishtr, 2016-11-08 00:48:42]

The possible downside of this CL is that it increases the memory footprint of
PaintLayer.
However, I don't think that is a big problem given other data we have about
total Blink memory
use, and that PaintLayer is only a fraction of all LayoutObjects.

https://codereview.chromium.org/2479403002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/paint/PaintLayer.cpp (left):

https://codereview.chromium.org/2479403002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/paint/PaintLayer.cpp:104: void* pointers[10];
This was wrong before.

[chrishtr, 2016-11-08 00:48:48]

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

[commit-bot: I haz the power, 2016-11-08 00:49:29]

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-11-08 01:59:56]

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

[commit-bot: I haz the power, 2016-11-08 01:59:57]

Dry run: This issue passed the CQ dry run.

[Xianzhu, 2016-11-08 03:44:35]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2016-11-08 03:44:35]

lgtm

[commit-bot: I haz the power, 2016-11-08 03:44:50]

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-11-08 03:48:47]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-08 03:55:20]

Description was changed from

==========
Move all ancestor-dependent properties out of PaintLayer and into an aux.
struct.

This makes PaintLayer smaller, which improves performance of code, such as hit
testing, that does not need to use these ancestor-dependent compositing
properties.

This patch makes hit testing on paint layers >= 15% faster. In particular, it
makes
the Events/hit-test-lots-of-layers.html performance test ~15% faster. It had
recently regressed when I added an IntRect.)

BUG=659059
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Move all ancestor-dependent properties out of PaintLayer and into an aux.
struct.

This makes PaintLayer smaller, which improves performance of code, such as hit
testing, that does not need to use these ancestor-dependent compositing
properties.

This patch makes hit testing on paint layers >= 15% faster. In particular, it
makes
the Events/hit-test-lots-of-layers.html performance test ~15% faster. It had
recently regressed when I added an IntRect.)

BUG=659059
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Committed: https://crrev.com/8af9d0bee7cdbd2db09562939b7d33696d9a7447
Cr-Commit-Position: refs/heads/master@{#430509}
==========

[commit-bot: I haz the power, 2016-11-08 03:55:21]

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