This may potentially fix a chromium issue and shouldn't hurt, so if anyone agrees that this fix sho…

This may potentially fix a chromium issue and shouldn't hurt, so if anyone agrees that this fix should be harmless, we can try merging it in to see if it's enough to fix "Invalid pointer write in GrRenderTarget::onRelease".

R=bsalomon@google.com, robertphillips@google.com

Committed: https://code.google.com/p/skia/source/detail?r=8982

[sugoi1, 2013-05-02 21:14:32]

If everyone agrees that this is harmless, I'll merge it in right away to see if
we're lucky enough for it to fix this :
https://code.google.com/p/chromium/issues/detail?id=161077

https://codereview.chromium.org/14607003/diff/1/src/gpu/gl/GrGpuGL.cpp
File src/gpu/gl/GrGpuGL.cpp (left):

https://codereview.chromium.org/14607003/diff/1/src/gpu/gl/GrGpuGL.cpp#oldcod...
src/gpu/gl/GrGpuGL.cpp:1007: GrGLuint rb,
Removed an unused parameter

https://codereview.chromium.org/14607003/diff/1/src/gpu/gl/GrGpuGL.cpp
File src/gpu/gl/GrGpuGL.cpp (right):

https://codereview.chromium.org/14607003/diff/1/src/gpu/gl/GrGpuGL.cpp#newcod...
src/gpu/gl/GrGpuGL.cpp:934: } else {
Since we were only checking texture sizes when creating a render target, I added
the non-render target size check too.

[robertphillips, 2013-05-02 21:24:57]

lgtm

[bsalomon, 2013-05-03 13:10:41]

On 2013/05/02 21:24:57, robertphillips wrote:
> lgtm

lgtm

[sugoi1, 2013-05-03 13:52:34]

Committed patchset #1 manually as r8982 (presubmit successful).