Update GrTRecorder to preserve memory allocations

src/gpu/GrInOrderDrawBuffer.h

 /*
  * Copyright 2011 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef GrInOrderDrawBuffer_DEFINED
 #define GrInOrderDrawBuffer_DEFINED
 
@@ skipping 283 matching lines @@
     void recordStateIfNecessary();
     // We lazily record clip changes in order to skip clips that have no effect.
     void recordClipIfNecessary();
     // Records any trace markers for a command after adding it to the buffer.
     void recordTraceMarkersIfNecessary();
 
     virtual bool isIssued(uint32_t drawID) { return drawID != fDrawID; }
 
     // TODO: Use a single allocator for commands and records
     enum {
-        kCmdBufferInitialSizeInBytes = 64 * 1024,
+        kCmdBufferInitialSizeInBytes = 8 * 1024,

[Chris Dalton, 2014/11/07 19:38:47]

This was arbitrary. The idea is that, since GrTRecorder preserves memory now, we
don't need to be quite so aggressive with the size of its initial allocation.
But it should still be relatively large for scenarios like alternating between
long and short draw calls.

[bsalomon, 2014/11/07 20:06:42]

makes sense to me

         kGeoPoolStatePreAllocCnt     = 4,
     };
 
     CmdBuffer                         fCmdBuffer;
     GrDrawState*                      fLastState;
     SkTArray<GrTraceMarkerSet, false> fGpuCmdMarkers;
     GrGpu*                            fDstGpu;
     GrVertexBufferAllocPool&          fVertexPool;
     GrIndexBufferAllocPool&           fIndexPool;
 
@@ skipping 12 matching lines @@
     typedef SkSTArray<kGeoPoolStatePreAllocCnt, GeometryPoolState> GeoPoolStateStack;
 
     GeoPoolStateStack                                   fGeoPoolStateStack;
     bool                                                fFlushing;
     uint32_t                                            fDrawID;
 
     typedef GrClipTarget INHERITED;
 };
 
 #endif