GrTessellator: don't reuse the previous vertex buffer on a cache miss.

GrTessellator: don't reuse the previous vertex buffer on a cache miss.

Just let the cache give us the correct entry for the given size.
This simplifies the code (no mysterious non-const ref member).
I don't see any performance effect from this change, but will watch
the bots closely.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1800623002

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

[Stephen White, 2016-03-14 14:38:15]

Description was changed from

==========
GrTessellator: don't reuse the previous vertex buffer on a cache miss.

Just let the cache give us the correct entry for the given size.
This simplifies the code (no mysterious non-const ref member).
I don't see any performance effect from this change, but will watch
the bots closely.

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

to

==========
GrTessellator: don't reuse the previous vertex buffer on a cache miss.

Just let the cache give us the correct entry for the given size.
This simplifies the code (no mysterious non-const ref member).
I don't see any performance effect from this change, but will watch
the bots closely.

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[Stephen White, 2016-03-14 15:40:00]

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

[commit-bot: I haz the power, 2016-03-14 15:40:08]

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

[Stephen White, 2016-03-14 15:42:11]

Description was changed from

==========
GrTessellator: don't reuse the previous vertex buffer on a cache miss.

Just let the cache give us the correct entry for the given size.
This simplifies the code (no mysterious non-const ref member).
I don't see any performance effect from this change, but will watch
the bots closely.

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
GrTessellator: don't reuse the previous vertex buffer on a cache miss.

Just let the cache give us the correct entry for the given size.
This simplifies the code (no mysterious non-const ref member).
I don't see any performance effect from this change, but will watch
the bots closely.

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[Stephen White, 2016-03-14 15:42:11]

senorblanco@chromium.org changed reviewers:
+ bsalomon@google.com

[Stephen White, 2016-03-14 15:42:17]

Brian: PTAL. Thanks!

[commit-bot: I haz the power, 2016-03-14 15:48:38]

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

[commit-bot: I haz the power, 2016-03-14 15:48:39]

Dry run: This issue passed the CQ dry run.

[Stephen White, 2016-03-15 20:44:05]

bsalomon: ping? (it's small)

[bsalomon, 2016-03-16 14:01:21]

On 2016/03/15 20:44:05, Stephen White wrote:
> bsalomon: ping? (it's small)

lgtm (sorry was OOO until today)

[Stephen White, 2016-03-16 17:01:10]

The CQ bit was checked by senorblanco@chromium.org

[commit-bot: I haz the power, 2016-03-16 17:01:18]

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

[commit-bot: I haz the power, 2016-03-16 17:26:00]

Description was changed from

==========
GrTessellator: don't reuse the previous vertex buffer on a cache miss.

Just let the cache give us the correct entry for the given size.
This simplifies the code (no mysterious non-const ref member).
I don't see any performance effect from this change, but will watch
the bots closely.

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
GrTessellator: don't reuse the previous vertex buffer on a cache miss.

Just let the cache give us the correct entry for the given size.
This simplifies the code (no mysterious non-const ref member).
I don't see any performance effect from this change, but will watch
the bots closely.

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

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

[commit-bot: I haz the power, 2016-03-16 17:26:01]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/af1e21e7ebb155d2505da0eb974c672953dfefef