gpu: don't use Texture::ValidForTexture() for GL Image.

gpu: don't use Texture::ValidForTexture() for GL Image.

GLES2DecoderImpl::DoCopySubTextureCHROMIUM/DoCompressedCopySubTextureCHROMIUM
uses ValidForTexture to check dimensions of source texture. If the source
texture is backed by GL Image, the decoder can report false positive error.

TEST=video texture test on webgl_conformance
BUG=517548

[dshwang, 2015-10-28 18:49:56]

Description was changed from

==========
gpu: don't use Texture::ValidForTexture() for GL Image.

GLES2DecoderImpl::DoCopySubTextureCHROMIUM/DoCompressedCopySubTextureCHROMIUM
uses ValidForTexture to check dimensions of source texture. If the source
texture is backed by GL Image, the decoder can report false positive error.

BUG=517548
==========

to

==========
gpu: don't use Texture::ValidForTexture() for GL Image.

GLES2DecoderImpl::DoCopySubTextureCHROMIUM/DoCompressedCopySubTextureCHROMIUM
uses ValidForTexture to check dimensions of source texture. If the source
texture is backed by GL Image, the decoder can report false positive error.

BUG=517548
==========

[dshwang, 2015-10-28 18:49:56]

dongseong.hwang@intel.com changed reviewers:
+ kbr@chromium.org, piman@chromium.org

[dshwang, 2015-10-28 18:51:24]

Description was changed from

==========
gpu: don't use Texture::ValidForTexture() for GL Image.

GLES2DecoderImpl::DoCopySubTextureCHROMIUM/DoCompressedCopySubTextureCHROMIUM
uses ValidForTexture to check dimensions of source texture. If the source
texture is backed by GL Image, the decoder can report false positive error.

BUG=517548
==========

to

==========
gpu: don't use Texture::ValidForTexture() for GL Image.

GLES2DecoderImpl::DoCopySubTextureCHROMIUM/DoCompressedCopySubTextureCHROMIUM
uses ValidForTexture to check dimensions of source texture. If the source
texture is backed by GL Image, the decoder can report false positive error.

TEST=video texture test on webgl_conformance
BUG=517548
==========

[dshwang, 2015-10-28 18:52:55]

piman, could you review?

It's fixed as part of cube map CL in
https://codereview.chromium.org/1275773003/diff/80001/gpu/command_buffer/serv...,
but the CL was reverted.
This bug causes innocent https://codereview.chromium.org/1315323006/ to be
reverted, which optimizes webgl.texSubImage2D(video) using
CopySubTextureCHROMIUM

[Ken Russell (switch to Gerrit), 2015-10-28 22:29:00]

I think it would be better to fold this into patch set 2 of
https://codereview.chromium.org/1413833006/ in order to avoid dependent CLs,
which can be difficult to track down in case one requires a revert.

[Ken Russell (switch to Gerrit), 2015-10-28 22:30:55]

kbr@chromium.org changed reviewers:
+ zmo@chromium.org

[Ken Russell (switch to Gerrit), 2015-10-28 22:30:56]

+zmo as reviewer too, just in case.

[dshwang, 2015-10-29 09:12:47]

On 2015/10/28 22:30:56, Ken Russell wrote:
> +zmo as reviewer too, just in case.

thank you for guidance. I fold this cl into
https://codereview.chromium.org/1413833006/
close as duplicated.