Use fast malloc for heap-allocated property tree builder context

Use fast malloc for heap-allocated property tree builder context

This patch switches to using fast malloc for the property tree
builder context which should bring performance back to before
https://codereview.chromium.org/2733143004 landed.

Locally this improves blink_perf.paint complex-content-slow-scroll by
12.5% and blink_perf.paint prePaint by 12.4%.

BUG=700996, 698653
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2744243010
Cr-Commit-Position: refs/heads/master@{#458131}
Committed: https://chromium.googlesource.com/chromium/src/+/9d29c6383dcd86368b16b2993cb0eacb062d5a58

[pdr., 2017-03-18 02:34:03]

Description was changed from

==========
Use fast malloc for heap-allocated property tree builder context

This patch switches to using fast malloc for the property tree
builder context which should bring performance back to before
https://codereview.chromium.org/2733143004 landed.

Locally this improves blink_perf.paint complex-content-slow-scroll by
12.5% and blink_perf.paint prePaint by 12.4%.

BUG=700996, 698653
==========

to

==========
Use fast malloc for heap-allocated property tree builder context

This patch switches to using fast malloc for the property tree
builder context which should bring performance back to before
https://codereview.chromium.org/2733143004 landed.

Locally this improves blink_perf.paint complex-content-slow-scroll by
12.5% and blink_perf.paint prePaint by 12.4%.

BUG=700996, 698653
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[pdr., 2017-03-18 02:34:38]

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

[commit-bot: I haz the power, 2017-03-18 02:35:05]

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

[pdr., 2017-03-18 03:47:10]

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

[commit-bot: I haz the power, 2017-03-18 04:11:24]

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

[commit-bot: I haz the power, 2017-03-18 04:11:25]

Dry run: This issue passed the CQ dry run.

[Xianzhu, 2017-03-20 16:20:24]

lgtm

https://codereview.chromium.org/2744243010/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilder.h (right):

https://codereview.chromium.org/2744243010/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilder.h:68: const
LayoutObject* containerForAbsolutePosition;
Why change these?

[pdr., 2017-03-20 16:35:32]

https://codereview.chromium.org/2744243010/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilder.h (right):

https://codereview.chromium.org/2744243010/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilder.h:68: const
LayoutObject* containerForAbsolutePosition;
On 2017/03/20 at 16:20:24, Xianzhu wrote:
> Why change these?

These seem to be required for fast malloc. A few other callsites do this as
well.

[pdr., 2017-03-20 16:35:36]

The CQ bit was checked by pdr@chromium.org

[commit-bot: I haz the power, 2017-03-20 16:35:59]

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-03-20 18:31:51]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490027736178490, "parent_rev":
"82b5d950824501a233e607c6b48e0f5bc8c7e0b1", "commit_rev":
"9d29c6383dcd86368b16b2993cb0eacb062d5a58"}

[commit-bot: I haz the power, 2017-03-20 18:34:33]

Description was changed from

==========
Use fast malloc for heap-allocated property tree builder context

This patch switches to using fast malloc for the property tree
builder context which should bring performance back to before
https://codereview.chromium.org/2733143004 landed.

Locally this improves blink_perf.paint complex-content-slow-scroll by
12.5% and blink_perf.paint prePaint by 12.4%.

BUG=700996, 698653
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Use fast malloc for heap-allocated property tree builder context

This patch switches to using fast malloc for the property tree
builder context which should bring performance back to before
https://codereview.chromium.org/2733143004 landed.

Locally this improves blink_perf.paint complex-content-slow-scroll by
12.5% and blink_perf.paint prePaint by 12.4%.

BUG=700996, 698653
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2744243010
Cr-Commit-Position: refs/heads/master@{#458131}
Committed:
https://chromium.googlesource.com/chromium/src/+/9d29c6383dcd86368b16b2993cb0...
==========

[commit-bot: I haz the power, 2017-03-20 18:34:35]

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