Revert of Use SkTypeface::getBounds() in bounding-box calculations. (patchset #5...
authormtklein <mtklein@google.com>
Wed, 29 Oct 2014 23:07:10 +0000 (16:07 -0700)
committerCommit bot <commit-bot@chromium.org>
Wed, 29 Oct 2014 23:07:11 +0000 (16:07 -0700)
commited167acb7b494bf0aece05ac8a68b560782b027c
tree74dcd878e12b0f728f9da03e185dbc9c92ce1b46
parent67c71203a4e2e4f4be0a4e2293bec3a1bc1b07c8
Revert of Use SkTypeface::getBounds() in bounding-box calculations. (patchset #5 id:80001 of https://codereview.chromium.org/680363003/)

Reason for revert:
Mac mini asserting

Original issue's description:
> Use SkTypeface::getBounds() in bounding-box calculations.
>
> This should produce tighter conservative bounding boxes for text than the
> approximation code it replaces.
>
> Recording performance is neutral on my desktop.  Playback performance
> improves by up to 15% on text heavy pages, e.g.
>
>   desk_pokemonwiki.skp_1 3.24ms -> 2.83ms  0.87x
>         desk_baidu.skp_1 1.91ms -> 1.58ms  0.83x
>
> Committed: https://skia.googlesource.com/skia/+/bf8dc343df4fbdcb8af546eb68b640e011a33489
>
> CQ_EXTRA_TRYBOTS=client.skia:Test-Win7-ShuttleA-HD2000-x86-Debug-Trybot
>
> Committed: https://skia.googlesource.com/skia/+/c51add674dfb89b988a7fbc05f41838c203f9dcd

TBR=reed@google.com,mtklein@chromium.org
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/690833002
src/core/SkRecordDraw.cpp
tests/RecordDrawTest.cpp