Enable array index clamping in shader translator.

gpu/command_buffer/service/shader_translator.cc

Index: gpu/command_buffer/service/shader_translator.cc
diff --git a/gpu/command_buffer/service/shader_translator.cc b/gpu/command_buffer/service/shader_translator.cc
index c58d19820eba892366cac8b4ca4412988bdea0a9..c086df9a858e8ec63584b7d6dd2abf7662834f4f 100644
--- a/gpu/command_buffer/service/shader_translator.cc
+++ b/gpu/command_buffer/service/shader_translator.cc
@@ -127,7 +127,8 @@ bool ShaderTranslator::Translate(const char* shader) {
 
   bool success = false;
   int compile_options =
-      SH_OBJECT_CODE | SH_ATTRIBUTES_UNIFORMS | SH_MAP_LONG_VARIABLE_NAMES;
+      SH_OBJECT_CODE | SH_ATTRIBUTES_UNIFORMS |
+      SH_MAP_LONG_VARIABLE_NAMES | SH_CLAMP_INDIRECT_ARRAY_BOUNDS;

[greggman, 2013/01/12 01:31:37]

Do you want | SH_ENFORCE_PACKING_RESTRICTIONS
or no? Without that the clamping seems useless.

PS: I'm also kind curious why the ANGLE patch uses a function instead of
clamp(v, 0, array_size);

   if (needs_built_in_function_emulation_)
     compile_options |= SH_EMULATE_BUILT_IN_FUNCTIONS;
   if (ShCompile(compiler_, &shader, 1, compile_options)) {