lgtm + nits https://codereview.appspot.com/7365047/diff/1/include/gpu/gl/GrGLConfig.h File include/gpu/gl/GrGLConfig.h (right): https://codereview.appspot.com/7365047/diff/1/include/gpu/gl/GrGLConfig.h#new... include/gpu/gl/GrGLConfig.h:101: * from VBOs. Chromium's command buffer doesn't allow glVertexAttrib with ARRAY https://codereview.appspot.com/7365047/diff/1/src/gpu/gl/GrGLUtil.cpp File src/gpu/gl/GrGLUtil.cpp (right): https://codereview.appspot.com/7365047/diff/1/src/gpu/gl/GrGLUtil.cpp#newcode173 src/gpu/gl/GrGLUtil.cpp:173: } a comment at a location of your choosing?
https://codereview.appspot.com/7365047/diff/1/include/gpu/gl/GrGLConfig.h File include/gpu/gl/GrGLConfig.h (right): https://codereview.appspot.com/7365047/diff/1/include/gpu/gl/GrGLConfig.h#new... include/gpu/gl/GrGLConfig.h:101: * from VBOs. Chromium's command buffer doesn't allow glVertexAttrib with On 2013/02/21 15:57:34, robertphillips wrote: > ARRAY Done. https://codereview.appspot.com/7365047/diff/1/src/gpu/gl/GrGLUtil.cpp File src/gpu/gl/GrGLUtil.cpp (right): https://codereview.appspot.com/7365047/diff/1/src/gpu/gl/GrGLUtil.cpp#newcode173 src/gpu/gl/GrGLUtil.cpp:173: } On 2013/02/21 15:57:34, robertphillips wrote: > a comment at a location of your choosing? Done.