Store vertex size in DrawState when setVertexAttribs is called

Store vertex size in DrawState when setVertexAttribs is called

This will avoid recalculating vertex size throughout code, and will allow
for a set original vertex size once optimizations start removing VA's

BUG=skia:

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

[bsalomon, 2014-08-21 19:28:30]

https://codereview.chromium.org/498483002/diff/20001/src/gpu/GrDrawState.cpp
File src/gpu/GrDrawState.cpp (right):

https://codereview.chromium.org/498483002/diff/20001/src/gpu/GrDrawState.cpp#...
src/gpu/GrDrawState.cpp:31: a.fVertexSize != b.fVertexSize ||
Do we need this? Or is this making the current code future-proof against your
coming changes?

[egdaniel, 2014-08-21 21:30:30]

https://codereview.chromium.org/498483002/diff/20001/src/gpu/GrDrawState.cpp
File src/gpu/GrDrawState.cpp (right):

https://codereview.chromium.org/498483002/diff/20001/src/gpu/GrDrawState.cpp#...
src/gpu/GrDrawState.cpp:31: a.fVertexSize != b.fVertexSize ||
On 2014/08/21 19:28:30, bsalomon wrote:
> Do we need this? Or is this making the current code future-proof against your
> coming changes?

It is not necessary for the current drawstate. Given that fVACount and FVAPtr
and the same, then the size must be same (since we are not removing any VAs at
this point). I'll remove it from here for now and add it back for the incoming
drawState changes

[egdaniel, 2014-08-22 13:46:31]

ptal

[bsalomon, 2014-08-22 13:54:07]

lgtm, optionally could assert in op== that the sizes agree.

[egdaniel, 2014-08-22 14:13:06]

The CQ bit was checked by egdaniel@google.com

[commit-bot: I haz the power, 2014-08-22 14:13:13]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/egdaniel@google.com/498483002/60001

[commit-bot: I haz the power, 2014-08-22 14:19:42]

Committed patchset #4 (60001) as ab78e06830cb70410b0e9e4f96575e137867e89f