Rebind to correct backbuffer after clearing READ buffer.

Rebind to correct backbuffer after clearing READ buffer.

BUG=none
TEST=unit tests

R=jbates@chromium.org


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=112394

[greggman, 2011-11-30 21:16:31]

Hey John, Do you mind reviewing this? Al is out.

[jbates, 2011-11-30 22:00:50]

LGTM, one question

http://codereview.chromium.org/8756001/diff/2001/gpu/command_buffer/service/g...
File gpu/command_buffer/service/gles2_cmd_decoder_unittest.cc (right):

http://codereview.chromium.org/8756001/diff/2001/gpu/command_buffer/service/g...
gpu/command_buffer/service/gles2_cmd_decoder_unittest.cc:5510: Result* result =
GetSharedMemoryAs<Result*>();
It doesn't look like result is ever accessed (there's just a sizeof below). Is
that intentional?

[greggman, 2011-11-30 22:13:15]

http://codereview.chromium.org/8756001/diff/2001/gpu/command_buffer/service/g...
File gpu/command_buffer/service/gles2_cmd_decoder_unittest.cc (right):

http://codereview.chromium.org/8756001/diff/2001/gpu/command_buffer/service/g...
gpu/command_buffer/service/gles2_cmd_decoder_unittest.cc:5510: Result* result =
GetSharedMemoryAs<Result*>();
On 2011/11/30 22:00:51, jbates wrote:
> It doesn't look like result is ever accessed (there's just a sizeof below). Is
> that intentional?

Yea, actual results from readPixels are checked in a different test. I removed
result from this test.
yes, the result of readpixels is tested in the readpixels tests else where.

[commit-bot: I haz the power, 2011-12-01 01:17:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gman@chromium.org/8756001/5002

[commit-bot: I haz the power, 2011-12-01 04:25:49]

Change committed as 112394