Emit "#version 120" if the shader has array as out parameter.

Emit "#version 120" if the shader has array as out parameter.
Committed: https://code.google.com/p/angleproject/source/detail?r=1273

[gmanchromium, 2012-09-04 22:44:53]

On 2012/09/04 22:26:09, Alok Priyadarshi wrote:

Is there a test for the WebGL Conformance Tests that we can add that will fail
without this fix and pass with it?

[sabercrombie, 2012-09-04 23:13:03]

This works for me.  The game runs without corruption and I confirmed in the
debugger that Mesa was getting the #version 120 directive for shaders containing
array out function parameters.

[dgkoch, 2012-09-05 02:29:30]

LGTM

https://codereview.appspot.com/6494082/diff/1/src/compiler/VersionGLSL.cpp
File src/compiler/VersionGLSL.cpp (right):

https://codereview.appspot.com/6494082/diff/1/src/compiler/VersionGLSL.cpp#ne...
src/compiler/VersionGLSL.cpp:2: // Copyright (c) 2002-2010 The ANGLE Project
Authors. All rights reserved.
-2012

[nicolas, 2012-09-05 15:50:03]

Looks fine to me too.

[Alok Priyadarshi, 2012-09-06 16:00:35]

On 2012/09/04 22:44:53, gmanchromium wrote:
> On 2012/09/04 22:26:09, Alok Priyadarshi wrote:
> 
> Is there a test for the WebGL Conformance Tests that we can add that will fail
> without this fix and pass with it?

I will send a WebGL conformance test to you. Although as noted in the original
bug, without this fix the test may still pass on some drivers because they more
permissible than others.

[Alok Priyadarshi, 2012-09-06 16:02:40]

https://codereview.appspot.com/6494082/diff/1/src/compiler/VersionGLSL.cpp
File src/compiler/VersionGLSL.cpp (right):

https://codereview.appspot.com/6494082/diff/1/src/compiler/VersionGLSL.cpp#ne...
src/compiler/VersionGLSL.cpp:2: // Copyright (c) 2002-2010 The ANGLE Project
Authors. All rights reserved.
On 2012/09/05 02:29:30, dgkoch wrote:
> -2012

Done.