DestroyStreamTextureChromium should not disallow rendering

DestroyStreamTextureChromium should not disallow rendering

This function just destroy the surfaceTexture.
Disallow rendering should happen when we delete the gl texture.

BUG=
TEST=

[Min Qin, 2011-11-23 01:25:43]

This fixes a DCHECK error in SetTarget in texture_manager.cc.

[greggman, 2011-11-29 21:39:43]

http://codereview.chromium.org/8669008/diff/1/gpu/command_buffer/service/gles...
File gpu/command_buffer/service/gles2_cmd_decoder.cc (left):

http://codereview.chromium.org/8669008/diff/1/gpu/command_buffer/service/gles...
gpu/command_buffer/service/gles2_cmd_decoder.cc:7480:
texture_manager()->SetInfoTarget(feature_info_, info, 0);
Something needs to happen here. The texture is going from a stream texture to a
non-stream texture.

Setting it back to 0 means if you bind it again it will become renderable. Maybe
you want to set it to TEXTURE_2D instead of 0? That seems iffy.

Although, there is a DCHECK in TextureInfo::SetTarget to ensure this gets set
only once. You probably need to change that check so that it's okay to set IF it
used to be a GL_TEXTURE_EXTERNAL_OES

[Min Qin, 2011-11-29 22:09:09]

On Tue, Nov 29, 2011 at 1:39 PM, <gman@chromium.org> wrote:

>
> http://codereview.chromium.**org/8669008/diff/1/gpu/**
>
command_buffer/service/gles2_**cmd_decoder.cc<http://codereview.chromium.org/8669008/diff/1/gpu/command_buffer/service/gles2_cmd_decoder.cc>
> File gpu/command_buffer/service/**gles2_cmd_decoder.cc (left):
>
> http://codereview.chromium.**org/8669008/diff/1/gpu/**
>
command_buffer/service/gles2_**cmd_decoder.cc#oldcode7480<http://codereview.chromium.org/8669008/diff/1/gpu/command_buffer/service/gles2_cmd_decoder.cc#oldcode7480>
> gpu/command_buffer/service/**gles2_cmd_decoder.cc:7480:
> texture_manager()->**SetInfoTarget(feature_info_, info, 0);
> Something needs to happen here. The texture is going from a stream
> texture to a non-stream texture.
>
> Setting it back to 0 means if you bind it again it will become
> renderable. Maybe you want to set it to TEXTURE_2D instead of 0? That
> seems iffy.
>

We want it to be renderable even if becomes a non-stream texture.  So we
can display the last frame of the video. This call should do nothing other
than release the android surfaceTexture.


> Although, there is a DCHECK in TextureInfo::SetTarget to ensure this
> gets set only once. You probably need to change that check so that it's
> okay to set IF it used to be a GL_TEXTURE_EXTERNAL_OES
>
>
http://codereview.chromium.**org/8669008/<http://codereview.chromium.org/8669...
>

[Min Qin, 2011-11-29 22:44:35]

Added back the setTargetInfo call and fixed the DCHECK in setTarget.