Don't store resources with a unique key in GrResourceCache's fScratchMap

Don't store resources with a unique key in GrResourceCache's fScratchMap

The reasoning here is that resources with a unique key are never selected from fScratchMap and just clog up the search for an available resource.

This knocks a 200x loop over the SVGbouncingrects case from 264ms down to 164ms.

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

Committed: https://skia.googlesource.com/skia/+/c4ed68426649dd4ca2c3119cdafdd562d3c3ba28

[robertphillips, 2016-05-24 20:03:29]

Description was changed from

==========
Don't store resources with a unique key in GrResourceCache's fScratchMap

BUG=603969
==========

to

==========
Don't store resources with a unique key in GrResourceCache's fScratchMap

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

[robertphillips, 2016-05-24 20:11:30]

Description was changed from

==========
Don't store resources with a unique key in GrResourceCache's fScratchMap

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

to

==========
Don't store resources with a unique key in GrResourceCache's fScratchMap

The reasoning here is that resources with a unique key are never selected from
fScratchMap and just clog up the search for an available resource.

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

[robertphillips, 2016-05-24 20:11:31]

robertphillips@google.com changed reviewers:
+ bsalomon@google.com

[robertphillips, 2016-05-24 20:17:52]

Description was changed from

==========
Don't store resources with a unique key in GrResourceCache's fScratchMap

The reasoning here is that resources with a unique key are never selected from
fScratchMap and just clog up the search for an available resource.

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

to

==========
Don't store resources with a unique key in GrResourceCache's fScratchMap

The reasoning here is that resources with a unique key are never selected from
fScratchMap and just clog up the search for an available resource.

This knocks a 200x loop over the SVGbouncingrects case from 264ms to 164ms.

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

[robertphillips, 2016-05-24 20:22:20]

Description was changed from

==========
Don't store resources with a unique key in GrResourceCache's fScratchMap

The reasoning here is that resources with a unique key are never selected from
fScratchMap and just clog up the search for an available resource.

This knocks a 200x loop over the SVGbouncingrects case from 264ms to 164ms.

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

to

==========
Don't store resources with a unique key in GrResourceCache's fScratchMap

The reasoning here is that resources with a unique key are never selected from
fScratchMap and just clog up the search for an available resource.

This knocks a 200x loop over the SVGbouncingrects case from 264ms down to 164ms.

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

[robertphillips, 2016-05-24 20:22:47]

The CQ bit was checked by robertphillips@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-05-24 20:22:52]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2008083002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2008083002/40001

[bsalomon, 2016-05-24 20:35:13]

Assuming this is covered adequately by existing unit tests, lgtm.

[commit-bot: I haz the power, 2016-05-24 20:40:07]

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

[commit-bot: I haz the power, 2016-05-24 20:40:08]

Dry run: This issue passed the CQ dry run.

[robertphillips, 2016-05-24 21:16:09]

The CQ bit was checked by robertphillips@google.com

[commit-bot: I haz the power, 2016-05-24 21:16:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2008083002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2008083002/40001

[commit-bot: I haz the power, 2016-05-24 21:17:15]

Description was changed from

==========
Don't store resources with a unique key in GrResourceCache's fScratchMap

The reasoning here is that resources with a unique key are never selected from
fScratchMap and just clog up the search for an available resource.

This knocks a 200x loop over the SVGbouncingrects case from 264ms down to 164ms.

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

to

==========
Don't store resources with a unique key in GrResourceCache's fScratchMap

The reasoning here is that resources with a unique key are never selected from
fScratchMap and just clog up the search for an available resource.

This knocks a 200x loop over the SVGbouncingrects case from 264ms down to 164ms.

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

Committed:
https://skia.googlesource.com/skia/+/c4ed68426649dd4ca2c3119cdafdd562d3c3ba28
==========

[commit-bot: I haz the power, 2016-05-24 21:17:16]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/c4ed68426649dd4ca2c3119cdafdd562d3c3ba28