Make Image::lock and unlock private and call them from Image::loadData.

Make Image::lock and unlock private and call them from Image::loadData.

This is because I want loadData to be able to decide whether to use LockRect/UnlockRect or UpdateSurface as the method to load the data, depending on whether the Images are backed by textures in the default pool or not.

I think it actually cleans up the code a bit.

Also, fixed a bug in subImageCompressed. The associated WebGL conformance test now passes.
Committed: https://code.google.com/p/angleproject/source/detail?r=1013

[apatrick1, 2012-03-28 22:57:12]

Hi Daniel,

Another one for you. Note that this on trunk and surfaces are oriented
accordingly :)

Thanks,

Al

https://codereview.appspot.com/5948043/diff/2001/src/libGLESv2/Texture.cpp
File src/libGLESv2/Texture.cpp (left):

https://codereview.appspot.com/5948043/diff/2001/src/libGLESv2/Texture.cpp#ol...
src/libGLESv2/Texture.cpp:1539: updateRegion.left = xoffset;
This is the bug.

[dgkoch, 2012-03-31 02:59:56]

LGTM