SkLiteDL: control reuse by a single parameter

SkLiteDL: control reuse by a single parameter

This simply caps the number of times a display list can be reused.
As this number goes up, the average amount of memory we cache goes up
and the expected number of mallocs per SkLiteDL::New() goes down.

This strategy does not need a hard-coded cap on how many display lists
to cache, or how big they can grow.

TBR=herb@google.com

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2226813002

Committed: https://skia.googlesource.com/skia/+/765b6e654802b770784becf3f53e5bd8beea4c9d

[mtklein_C, 2016-08-08 18:17:52]

Description was changed from

==========
SkLiteDL: control reuse by a single parameter

This simply caps the number of times a display list can be reused.
As this number goes up, the average amount of memory we cache goes up
and the expected number of mallocs per SkLiteDL::New() goes down.

This strategy does not need a hard-coded cap on how many display lists
to cache, or how big they can grow.

BUG=skia:
==========

to

==========
SkLiteDL: control reuse by a single parameter

This simply caps the number of times a display list can be reused.
As this number goes up, the average amount of memory we cache goes up
and the expected number of mallocs per SkLiteDL::New() goes down.

This strategy does not need a hard-coded cap on how many display lists
to cache, or how big they can grow.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2226813002
==========

[mtklein_C, 2016-08-08 18:21:54]

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

[commit-bot: I haz the power, 2016-08-08 18:21:59]

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

[mtklein, 2016-08-08 18:53:33]

Description was changed from

==========
SkLiteDL: control reuse by a single parameter

This simply caps the number of times a display list can be reused.
As this number goes up, the average amount of memory we cache goes up
and the expected number of mallocs per SkLiteDL::New() goes down.

This strategy does not need a hard-coded cap on how many display lists
to cache, or how big they can grow.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2226813002
==========

to

==========
SkLiteDL: control reuse by a single parameter

This simply caps the number of times a display list can be reused.
As this number goes up, the average amount of memory we cache goes up
and the expected number of mallocs per SkLiteDL::New() goes down.

This strategy does not need a hard-coded cap on how many display lists
to cache, or how big they can grow.

TBR=herb@google.com

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2226813002
==========

[mtklein, 2016-08-08 18:53:42]

mtklein@google.com changed reviewers:
+ herb@google.com, mtklein@google.com

[mtklein_C, 2016-08-08 18:53:59]

mtklein@chromium.org changed reviewers:
- mtklein@google.com

[mtklein_C, 2016-08-08 18:54:02]

The CQ bit was unchecked by mtklein@chromium.org

[mtklein_C, 2016-08-08 18:54:03]

The CQ bit was checked by mtklein@chromium.org

[commit-bot: I haz the power, 2016-08-08 18:54:08]

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-08-08 19:23:04]

Description was changed from

==========
SkLiteDL: control reuse by a single parameter

This simply caps the number of times a display list can be reused.
As this number goes up, the average amount of memory we cache goes up
and the expected number of mallocs per SkLiteDL::New() goes down.

This strategy does not need a hard-coded cap on how many display lists
to cache, or how big they can grow.

TBR=herb@google.com

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2226813002
==========

to

==========
SkLiteDL: control reuse by a single parameter

This simply caps the number of times a display list can be reused.
As this number goes up, the average amount of memory we cache goes up
and the expected number of mallocs per SkLiteDL::New() goes down.

This strategy does not need a hard-coded cap on how many display lists
to cache, or how big they can grow.

TBR=herb@google.com

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2226813002

Committed:
https://skia.googlesource.com/skia/+/765b6e654802b770784becf3f53e5bd8beea4c9d
==========

[commit-bot: I haz the power, 2016-08-08 19:23:05]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/765b6e654802b770784becf3f53e5bd8beea4c9d