Clean up the GrTHashTable API.

Clean up the GrTHashTable API.

removeAt() and slowFindIndex() were only used by GrFontCache, and didn't really need to exist: they can be replaced by a simple remove() call.  This new code should actually be faster; slowFindIndex() was O(N), and this should be O(lg N).

BUG=

Committed: http://code.google.com/p/skia/source/detail?r=11973

[bsalomon, 2013-10-25 20:48:46]

lgtm w/ one q

https://codereview.chromium.org/43383006/diff/30001/src/gpu/GrTextStrike_impl.h
File src/gpu/GrTextStrike_impl.h (right):

https://codereview.chromium.org/43383006/diff/30001/src/gpu/GrTextStrike_impl...
src/gpu/GrTextStrike_impl.h:19: explicit Key(const GrKey* fontScalarKey) {
does it make sense to have the GrFontScaler cons if we have this one?

[mtklein, 2013-10-25 21:08:21]

https://codereview.chromium.org/43383006/diff/30001/src/gpu/GrTextStrike_impl.h
File src/gpu/GrTextStrike_impl.h (right):

https://codereview.chromium.org/43383006/diff/30001/src/gpu/GrTextStrike_impl...
src/gpu/GrTextStrike_impl.h:19: explicit Key(const GrKey* fontScalarKey) {
On 2013/10/25 20:48:47, bsalomon wrote:
> does it make sense to have the GrFontScaler cons if we have this one?

Nope.  Gone.

[bsalomon, 2013-10-25 21:10:56]

lgtm

[commit-bot: I haz the power, 2013-10-27 19:21:48]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/mtklein@google.com/43383006/80001

[commit-bot: I haz the power, 2013-10-27 19:28:44]

Retried try job too often on Build-Ubuntu12-GCC-x86_64-Release-Trybot for
step(s) BuildBench, BuildEverything, BuildTools
http://skiabot-master.pogerlabs.com:10117/buildstatus?builder=Build-Ubuntu12-...

[commit-bot: I haz the power, 2013-10-27 20:38:48]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/mtklein@google.com/43383006/190001

[commit-bot: I haz the power, 2013-10-27 20:50:26]

Change committed as 11973