Fix assert failure caused by ImageExtractor and optimize test

Fix assert failure caused by ImageExtractor and optimize test

unlockPixels is always called in the ImageExtractor destructor if
m_skiaImage is non-null. Clear m_skiaImage if extractImage fails and
lockPixels is never called so that lockPixels and unlockPixels calls
will be in balance.

Also optimize webgl-large-texture.html test, since some of the test
timeouts could be caused by the test running too long. Optimizations
include avoiding conversion between RGBA and RGB, checking the pixels in
32 bit chunks instead of 8 bits at a time, and loading the image from a
file instead of generating a JPEG data URL dynamically.

TEST=fast/canvas/webgl/webgl-large-texture.html
BUG=358313
R=kbr@chromium.org, skyostil@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=170642

[Sami, 2014-04-01 15:25:04]

lgtm, thanks!

[Ken Russell (switch to Gerrit), 2014-04-01 18:43:38]

LGTM too; thanks for the fix.

[oetuaho-nv, 2014-04-02 07:35:41]

The CQ bit was checked by oetuaho@nvidia.com

[commit-bot: I haz the power, 2014-04-02 07:35:48]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/oetuaho@nvidia.com/221083002/1

[commit-bot: I haz the power, 2014-04-02 08:40:24]

Change committed as 170642