Start USING_FAST_MALLOC for more paint data structures

Start USING_FAST_MALLOC for more paint data structures

This patch starts using fast malloc (aka partition alloc) for the following
paint-related data structures:
 PaintInfo
 PaintLayerRareData
 PaintArtifactCompositor
 GeometryMapper
 GeometryMapperClipCache
 GeometryMapperTransformCache
 PropertyTreeState

For more information about USING_FAST_MALLOC, see the Allocator.md docs.

Review-Url: https://codereview.chromium.org/2783913002
Cr-Commit-Position: refs/heads/master@{#460591}
Committed: https://chromium.googlesource.com/chromium/src/+/43d841d52e2c28931f80ab41adf564e8d0912bb0

[pdr., 2017-03-29 15:56:01]

Description was changed from

==========
Start USING_FAST_MALLOC for more paint data structures

This patch starts using fast malloc (aka partition alloc) for the following
paint-related data structures:
 PaintInfo
 PaintLayerRareData
 PaintArtifactCompositor
 GeometryMapper
 GeometryMapperClipCache
 GeometryMapperTransformCache
 PropertyTreeState

For more information about USING_FAST_MALLOC, see the Allocator.md docs.
==========

to

==========
Start USING_FAST_MALLOC for more paint data structures

This patch starts using fast malloc (aka partition alloc) for the following
paint-related data structures:
 PaintInfo
 PaintLayerRareData
 PaintArtifactCompositor
 GeometryMapper
 GeometryMapperClipCache
 GeometryMapperTransformCache
 PropertyTreeState

For more information about USING_FAST_MALLOC, see the Allocator.md docs.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[pdr., 2017-03-29 15:56:27]

Description was changed from

==========
Start USING_FAST_MALLOC for more paint data structures

This patch starts using fast malloc (aka partition alloc) for the following
paint-related data structures:
 PaintInfo
 PaintLayerRareData
 PaintArtifactCompositor
 GeometryMapper
 GeometryMapperClipCache
 GeometryMapperTransformCache
 PropertyTreeState

For more information about USING_FAST_MALLOC, see the Allocator.md docs.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Start USING_FAST_MALLOC for more paint data structures

This patch starts using fast malloc (aka partition alloc) for the following
paint-related data structures:
 PaintInfo
 PaintLayerRareData
 PaintArtifactCompositor
 GeometryMapper
 GeometryMapperClipCache
 GeometryMapperTransformCache
 PropertyTreeState

For more information about USING_FAST_MALLOC, see the Allocator.md docs.
==========

[pdr., 2017-03-29 15:56:27]

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

[pdr., 2017-03-29 15:58:04]

This isn't an exhaustive list but it gets some of the big ones such as
PropertyTreeState and the GeometryMapper caches. If this shows a perf benefit,
I'll file a bug to go through all the classes.

[chrishtr, 2017-03-29 16:08:57]

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

[chrishtr, 2017-03-29 16:08:58]

+haraken as FYI/double-check.

https://codereview.chromium.org/2783913002/diff/1/third_party/WebKit/Source/p...
File third_party/WebKit/Source/platform/graphics/paint/PropertyTreeState.h
(right):

https://codereview.chromium.org/2783913002/diff/1/third_party/WebKit/Source/p...
third_party/WebKit/Source/platform/graphics/paint/PropertyTreeState.h:21: class
PLATFORM_EXPORT PropertyTreeState {
*PaintPropertyNode also?

[pdr., 2017-03-29 16:16:26]

https://codereview.chromium.org/2783913002/diff/1/third_party/WebKit/Source/p...
File third_party/WebKit/Source/platform/graphics/paint/PropertyTreeState.h
(right):

https://codereview.chromium.org/2783913002/diff/1/third_party/WebKit/Source/p...
third_party/WebKit/Source/platform/graphics/paint/PropertyTreeState.h:21: class
PLATFORM_EXPORT PropertyTreeState {
On 2017/03/29 at 16:08:58, chrishtr wrote:
> *PaintPropertyNode also?

I think we're good here because PaintPropertyNodes should be in the partition
alloc due to being RefCounted.

[chrishtr, 2017-03-29 16:17:37]

lgtm

Ah.

[pdr., 2017-03-29 16:24:11]

@haraken, can you take a look as well?

[pdr., 2017-03-29 16:27:42]

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

[commit-bot: I haz the power, 2017-03-29 16:28:34]

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-03-29 18:34:34]

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

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

Dry run: This issue passed the CQ dry run.

[haraken, 2017-03-29 23:18:08]

LGTM

[pdr., 2017-03-30 00:02:12]

The CQ bit was checked by pdr@chromium.org

[commit-bot: I haz the power, 2017-03-30 00:03:48]

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-30 00:15:22]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490832132072020, "parent_rev":
"4cbd5e215ed47201e8914cb93269051eaf41fc96", "commit_rev":
"43d841d52e2c28931f80ab41adf564e8d0912bb0"}

[commit-bot: I haz the power, 2017-03-30 00:16:51]

Description was changed from

==========
Start USING_FAST_MALLOC for more paint data structures

This patch starts using fast malloc (aka partition alloc) for the following
paint-related data structures:
 PaintInfo
 PaintLayerRareData
 PaintArtifactCompositor
 GeometryMapper
 GeometryMapperClipCache
 GeometryMapperTransformCache
 PropertyTreeState

For more information about USING_FAST_MALLOC, see the Allocator.md docs.
==========

to

==========
Start USING_FAST_MALLOC for more paint data structures

This patch starts using fast malloc (aka partition alloc) for the following
paint-related data structures:
 PaintInfo
 PaintLayerRareData
 PaintArtifactCompositor
 GeometryMapper
 GeometryMapperClipCache
 GeometryMapperTransformCache
 PropertyTreeState

For more information about USING_FAST_MALLOC, see the Allocator.md docs.

Review-Url: https://codereview.chromium.org/2783913002
Cr-Commit-Position: refs/heads/master@{#460591}
Committed:
https://chromium.googlesource.com/chromium/src/+/43d841d52e2c28931f80ab41adf5...
==========

[commit-bot: I haz the power, 2017-03-30 00:16:52]

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