BindTransformFeedback(0) is legal.

BindTransformFeedback(0) is legal.

0 is the default transform feedback.

Currently we generate an INVALID_OP in command buffer service, which
is incorrect.

BUG=429053
TEST=gpu_unittests
R=bajones@chromium.org

Committed: https://crrev.com/5347489d1d33c82a670350d027a3c00b002c704a
Cr-Commit-Position: refs/heads/master@{#331733}

[Zhenyao Mo, 2015-05-27 23:40:55]

bajones: PTAL

[Ken Russell (switch to Gerrit), 2015-05-27 23:48:14]

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

[Ken Russell (switch to Gerrit), 2015-05-27 23:48:14]

The code change looks good according to the spec, but is there a test for this?

[bajones, 2015-05-27 23:49:02]

On 2015/05/27 23:40:55, Zhenyao Mo wrote:
> bajones: PTAL

LGTM! Thanks!

[Zhenyao Mo, 2015-05-27 23:51:53]

On 2015/05/27 23:48:14, Ken Russell wrote:
> The code change looks good according to the spec, but is there a test for
this?

Yes, there is a conformance test submitted by bajones.  That's how this bug is
caught.

[bajones, 2015-05-27 23:52:17]

On 2015/05/27 23:51:53, Zhenyao Mo wrote:
> On 2015/05/27 23:48:14, Ken Russell wrote:
> > The code change looks good according to the spec, but is there a test for
> this?
> 
> Yes, there is a conformance test submitted by bajones.  That's how this bug is
> caught.

https://github.com/KhronosGroup/WebGL/pull/1009

[Zhenyao Mo, 2015-05-28 02:06:28]

The CQ bit was checked by zmo@chromium.org

[commit-bot: I haz the power, 2015-05-28 02:07:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1145103006/20001

[commit-bot: I haz the power, 2015-05-28 02:59:08]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-28 03:00:13]

Patchset 2 (id:??) landed as
https://crrev.com/5347489d1d33c82a670350d027a3c00b002c704a
Cr-Commit-Position: refs/heads/master@{#331733}