Fix getAttribLocation to return -1 for built-in attributes per WebGL 2.0 spec

src/client/graphics/webgl_context.cpp

@@ -1,3 +1,4 @@
+#include <set>
 #include <string>
 #include <idgen.hpp>
 #include <common/image/image_processor.hpp>
@@ -831,6 +832,15 @@ namespace endor
     optional<WebGLAttribLocation> WebGLContext::getAttribLocation(shared_ptr<WebGLProgram> program, const string &name)
     {
       assert(program != nullptr && "Program is not null");
+
+      // WebGL 2.0 spec §5.18: Built-in attributes always return -1
+      static const std::set<string> kBuiltInAttribs = {
+        "gl_VertexID", "gl_InstanceID", "gl_Position", "gl_PointSize"};
+      if (kBuiltInAttribs.count(name))
+      {
+        return nullopt; // Returns nullopt which script bindings converts to -1
+      }
+
       return program->hasAttribLocation(name) ? optional<WebGLAttribLocation>(program->getAttribLocation(name)) : nullopt;
     }
 

src/client/script_bindings/webgl/active_info.cpp

@@ -60,5 +60,5 @@ namespace endor
       }
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor

src/client/script_bindings/webgl/active_info.hpp

@@ -38,5 +38,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor

src/client/script_bindings/webgl/framebuffer.cpp

@@ -31,5 +31,5 @@ namespace endor
       {
       }
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor

src/client/script_bindings/webgl/framebuffer.hpp

@@ -37,5 +37,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor

src/client/script_bindings/webgl/object.cpp

@@ -28,5 +28,5 @@ namespace endor
         // WebGLObject is a base class, typically not instantiated directly
       }
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor

src/client/script_bindings/webgl/object.hpp

@@ -45,5 +45,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor

src/client/script_bindings/webgl/program.cpp

@@ -20,5 +20,5 @@ namespace endor
       }
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor

src/client/script_bindings/webgl/program.hpp

@@ -32,5 +32,5 @@ namespace endor
         WebGLProgram(v8::Isolate *isolate, const v8::FunctionCallbackInfo<v8::Value> &args);
       };
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor

src/client/script_bindings/webgl/renderbuffer.cpp

@@ -19,5 +19,5 @@ namespace endor
         // WebGLRenderbuffer objects are created by WebGL context, not by user code
       }
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor

src/client/script_bindings/webgl/renderbuffer.hpp

@@ -33,5 +33,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor

src/client/script_bindings/webgl/texture.cpp

@@ -27,5 +27,5 @@ namespace endor
       {
       }
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor

src/client/script_bindings/webgl/texture.hpp

@@ -45,5 +45,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor

src/client/script_bindings/webgl/uniform_location.cpp

@@ -47,5 +47,5 @@ namespace endor
       }
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor

src/client/script_bindings/webgl/uniform_location.hpp

@@ -44,5 +44,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor

src/client/script_bindings/webgl/vertex_array.cpp

@@ -20,5 +20,5 @@ namespace endor
       {
       }
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor

src/client/script_bindings/webgl/vertex_array.hpp

@@ -33,5 +33,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor