2 files changed, 81 insertions, 10 deletions
diff --git a/components/script/dom/webglrenderingcontext.rs b/components/script/dom/webglrenderingcontext.rs
index d21691f14af..6655a56f921 100644
--- a/components/script/dom/webglrenderingcontext.rs
+++ b/components/script/dom/webglrenderingcontext.rs
@@ -378,16 +378,27 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext {
     #[allow(unsafe_code)]
     // https://www.khronos.org/registry/webgl/specs/latest/1.0/#5.14.5
     fn BufferData(&self, _cx: *mut JSContext, target: u32, data: Option<*mut JSObject>, usage: u32) {
+        match target {
+            constants::ARRAY_BUFFER |
+            constants::ELEMENT_ARRAY_BUFFER => (),
+            _ => return self.webgl_error(InvalidEnum),
+        }
+        match usage {
+            constants::STREAM_DRAW |
+            constants::STATIC_DRAW |
+            constants::DYNAMIC_DRAW => (),
+            _ => return self.webgl_error(InvalidEnum),
+        }
         let data = match data {
             Some(data) => data,
-            None => return,
+            None => return self.webgl_error(InvalidValue),
         };
         let data_vec = unsafe {
             let mut length = 0;
             let mut ptr = ptr::null_mut();
             let buffer_data = JS_GetObjectAsArrayBufferView(data, &mut length, &mut ptr);
             if buffer_data.is_null() {
-                panic!("Argument data to WebGLRenderingContext.bufferdata is not a Float32Array")
+                return self.webgl_error(InvalidValue) // https://github.com/servo/servo/issues/5014
             }
             let data_f32 = JS_GetFloat32ArrayData(buffer_data, ptr::null());
             let data_vec_length = length / mem::size_of::<f32>() as u32;
@@ -398,6 +409,56 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext {
             .unwrap()
     }
 
+    #[allow(unsafe_code)]
+    // https://www.khronos.org/registry/webgl/specs/latest/1.0/#5.14.5
+    fn BufferSubData(&self, _cx: *mut JSContext, target: u32, offset: i64, data: Option<*mut JSObject>) {
+        match target {
+            constants::ARRAY_BUFFER |
+            constants::ELEMENT_ARRAY_BUFFER => (),
+            _ => return self.webgl_error(InvalidEnum),
+        }
+        let data = match data {
+            Some(data) => data,
+            None => return,
+        };
+        if offset < 0 {
+            return self.webgl_error(InvalidValue);
+        }
+        let data_vec = unsafe {
+            let mut length = 0;
+            let mut ptr = ptr::null_mut();
+            let buffer_data = JS_GetObjectAsArrayBufferView(data, &mut length, &mut ptr);
+            if buffer_data.is_null() {
+                return self.webgl_error(InvalidValue) // https://github.com/servo/servo/issues/5014
+            }
+            let data_f32 = JS_GetFloat32ArrayData(buffer_data, ptr::null());
+            let data_vec_length = length / mem::size_of::<f32>() as u32;
+            slice::from_raw_parts(data_f32, data_vec_length as usize).to_vec()
+        };
+        // FIXME(simartin) Check that the defined region is inside the allocated one
+        // https://github.com/servo/servo/issues/8738
+        self.ipc_renderer
+            .send(CanvasMsg::WebGL(CanvasWebGLMsg::BufferSubData(target, offset as isize, data_vec)))
+            .unwrap()
+    }
+
+    // https://www.khronos.org/registry/webgl/specs/latest/1.0/#5.14.8
+    fn CompressedTexImage2D(&self, _cx: *mut JSContext, _target: u32, _level: i32, _internal_format: u32,
+                            _width: i32, _height: i32, _border: i32, _pixels: *mut JSObject) {
+        // FIXME: No compressed texture format is currently supported, so error out as per
+        // https://www.khronos.org/registry/webgl/specs/latest/1.0/#COMPRESSED_TEXTURE_SUPPORT
+        self.webgl_error(InvalidEnum)
+    }
+
+    // https://www.khronos.org/registry/webgl/specs/latest/1.0/#5.14.8
+    fn CompressedTexSubImage2D(&self, _cx: *mut JSContext, _target: u32, _level: i32,
+                               _xoffset: i32, _yoffset: i32, _width: i32, _height: i32,
+                               _format: u32, _pixels: *mut JSObject) {
+        // FIXME: No compressed texture format is currently supported, so error out as per
+        // https://www.khronos.org/registry/webgl/specs/latest/1.0/#COMPRESSED_TEXTURE_SUPPORT
+        self.webgl_error(InvalidEnum)
+    }
+
     // https://www.khronos.org/registry/webgl/specs/latest/1.0/#5.14.11
     fn Clear(&self, mask: u32) {
         self.ipc_renderer.send(CanvasMsg::WebGL(CanvasWebGLMsg::Clear(mask))).unwrap();
diff --git a/components/script/dom/webidls/WebGLRenderingContext.webidl b/components/script/dom/webidls/WebGLRenderingContext.webidl
index 27019ed7754..cd1937e4430 100644
--- a/components/script/dom/webidls/WebGLRenderingContext.webidl
+++ b/components/script/dom/webidls/WebGLRenderingContext.webidl
@@ -492,10 +492,11 @@ interface WebGLRenderingContextBase
     //void bufferData(GLenum target, GLsizeiptr size, GLenum usage);
     // FIXME(dmarcos) The function below is the original function in the webIdl:
     // void bufferData(GLenum target, BufferDataSource? data, GLenum usage);
-    // The Code genearator doesn't handle BufferDataSource so we're using 'optional object'
+    // The Code generator doesn't handle BufferDataSource so we're using 'optional object'
     // in the meantime
     void bufferData(GLenum target, optional object data, GLenum usage);
     //void bufferSubData(GLenum target, GLintptr offset, BufferDataSource? data);
+    void bufferSubData(GLenum target, GLintptr offset, optional object data);
 
     //[WebGLHandlesContextLoss] GLenum checkFramebufferStatus(GLenum target);
     void clear(GLbitfield mask);
@@ -505,13 +506,22 @@ interface WebGLRenderingContextBase
     void colorMask(GLboolean red, GLboolean green, GLboolean blue, GLboolean alpha);
     void compileShader(WebGLShader? shader);
 
-    //void compressedTexImage2D(GLenum target, GLint level, GLenum internalformat,
-    //                          GLsizei width, GLsizei height, GLint border,
-    //                          ArrayBufferView data);
-    //void compressedTexSubImage2D(GLenum target, GLint level,
-    //                             GLint xoffset, GLint yoffset,
-    //                             GLsizei width, GLsizei height, GLenum format,
-    //                             ArrayBufferView data);
+    // FIXME(simartin) The Code generator doesn't handle ArrayBufferView so we're
+    // using 'object' in the meantime
+    // void compressedTexImage2D(GLenum target, GLint level, GLenum internalformat,
+    //                           GLsizei width, GLsizei height, GLint border,
+    //                           ArrayBufferView data);
+    void compressedTexImage2D(GLenum target, GLint level, GLenum internalformat,
+                              GLsizei width, GLsizei height, GLint border,
+                              object data);
+    // void compressedTexSubImage2D(GLenum target, GLint level,
+    //                              GLint xoffset, GLint yoffset,
+    //                              GLsizei width, GLsizei height, GLenum format,
+    //                              ArrayBufferView data);
+    void compressedTexSubImage2D(GLenum target, GLint level,
+                                 GLint xoffset, GLint yoffset,
+                                 GLsizei width, GLsizei height, GLenum format,
+                                 object data);
 
     //void copyTexImage2D(GLenum target, GLint level, GLenum internalformat,
     //                    GLint x, GLint y, GLsizei width, GLsizei height,