Revert of Rearrange SkRecord with small N in mind (patchset #8 id:120001 of https...
authormtklein <mtklein@google.com>
Wed, 8 Apr 2015 21:09:41 +0000 (14:09 -0700)
committerCommit bot <commit-bot@chromium.org>
Wed, 8 Apr 2015 21:09:41 +0000 (14:09 -0700)
commit35f55764b81390a085fb90f624082c196fbd6229
treee16cf8961b814300b4c3f02551aff5cd869ffd06
parente2dd9408cd711777afaa9410427fb0d761ab004a
Revert of Rearrange SkRecord with small N in mind (patchset #8 id:120001 of https://codereview.chromium.org/1061783002/)

Reason for revert:
https://uberchromegw.corp.google.com/i/client.skia/builders/Test-Ubuntu-GCC-GCE-CPU-AVX2-x86-Debug/builds/149/steps/dm/logs/stdio

Original issue's description:
> Rearrange SkRecord with small N in mind
>
> This rearranges the record pointers and types so they can go in a single array, then preallocates some space for them and for the SkVarAlloc.
>
> picture_overhead_draw bench drops from ~1000ns to 500-600ns, with no effect on picture_overhead_nodraw.
>
> I don't see any significant effect on large picture recording times from our .skps.
>
> BUG=chromium:470553
>
> Committed: https://skia.googlesource.com/skia/+/e2dd9408cd711777afaa9410427fb0d761ab004a

TBR=reed@google.com,mtklein@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:470553

Review URL: https://codereview.chromium.org/1068383003
src/core/SkRecord.cpp
src/core/SkRecord.h
src/core/SkVarAlloc.cpp
src/core/SkVarAlloc.h
tests/PictureTest.cpp