Remove LegacyDefaultOptionalArguments flag from WebGL
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 15 Jul 2011 06:33:24 +0000 (06:33 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 15 Jul 2011 06:33:24 +0000 (06:33 +0000)
https://bugs.webkit.org/show_bug.cgi?id=64549

Patch by Mark Pilgrim <pilgrim@chromium.org> on 2011-07-14
Reviewed by Kenneth Russell.

Source/WebCore:

* html/canvas/WebGLRenderingContext.idl:

LayoutTests:

* fast/canvas/webgl/bad-arguments-test-expected.txt:
* fast/canvas/webgl/bad-arguments-test.html:
* fast/canvas/webgl/context-lost-expected.txt:
* fast/canvas/webgl/context-lost.html:
* fast/canvas/webgl/null-object-behaviour-expected.txt:
* fast/canvas/webgl/null-object-behaviour.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@91053 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/fast/canvas/webgl/bad-arguments-test-expected.txt
LayoutTests/fast/canvas/webgl/bad-arguments-test.html
LayoutTests/fast/canvas/webgl/context-lost-expected.txt
LayoutTests/fast/canvas/webgl/context-lost.html
LayoutTests/fast/canvas/webgl/null-object-behaviour-expected.txt
LayoutTests/fast/canvas/webgl/null-object-behaviour.html
Source/WebCore/ChangeLog
Source/WebCore/html/canvas/WebGLRenderingContext.idl

index 16bf98f..502cfbf 100644 (file)
@@ -1,3 +1,17 @@
+2011-07-14  Mark Pilgrim  <pilgrim@chromium.org>
+
+        Remove LegacyDefaultOptionalArguments flag from WebGL
+        https://bugs.webkit.org/show_bug.cgi?id=64549
+
+        Reviewed by Kenneth Russell.
+
+        * fast/canvas/webgl/bad-arguments-test-expected.txt:
+        * fast/canvas/webgl/bad-arguments-test.html:
+        * fast/canvas/webgl/context-lost-expected.txt:
+        * fast/canvas/webgl/context-lost.html:
+        * fast/canvas/webgl/null-object-behaviour-expected.txt:
+        * fast/canvas/webgl/null-object-behaviour.html:
+
 2011-07-14  Yuta Kitamura  <yutak@chromium.org>
 
         WebSocket: Introduce pywebsocket-0.6b2
index 208bbb6..740d3f8 100644 (file)
@@ -7,7 +7,6 @@ PASS Shader Compiled
 PASS getUniformLocation succeeded
 PASS context.compileShader(argument) threw exception TypeError: Type error.
 PASS context.linkProgram(argument) threw exception TypeError: Type error.
-PASS context.attachShader(argument) threw exception TypeError: Type error.
 PASS context.attachShader(program, argument) threw exception TypeError: Type error.
 PASS context.attachShader(argument, shader) threw exception TypeError: Type error.
 PASS context.detachShader(program, argument) threw exception TypeError: Type error.
@@ -34,7 +33,6 @@ PASS context.getShaderInfoLog(argument) threw exception TypeError: Type error.
 PASS context.getShaderSource(argument) threw exception TypeError: Type error.
 PASS context.compileShader(argument) threw exception TypeError: Type error.
 PASS context.linkProgram(argument) threw exception TypeError: Type error.
-PASS context.attachShader(argument) threw exception TypeError: Type error.
 PASS context.attachShader(program, argument) threw exception TypeError: Type error.
 PASS context.attachShader(argument, shader) threw exception TypeError: Type error.
 PASS context.detachShader(program, argument) threw exception TypeError: Type error.
@@ -61,7 +59,6 @@ PASS context.getShaderInfoLog(argument) threw exception TypeError: Type error.
 PASS context.getShaderSource(argument) threw exception TypeError: Type error.
 PASS context.compileShader(argument) is undefined.
 PASS context.linkProgram(argument) is undefined.
-PASS context.attachShader(argument) is undefined.
 PASS context.attachShader(program, argument) is undefined.
 PASS context.attachShader(argument, shader) is undefined.
 PASS context.detachShader(program, argument) is undefined.
@@ -88,7 +85,6 @@ PASS context.getShaderInfoLog(argument) is ''
 PASS context.getShaderSource(argument) is ''
 PASS context.compileShader(argument) is undefined.
 PASS context.linkProgram(argument) is undefined.
-PASS context.attachShader(argument) is undefined.
 PASS context.attachShader(program, argument) is undefined.
 PASS context.attachShader(argument, shader) is undefined.
 PASS context.detachShader(program, argument) is undefined.
index 4e86478..328fd46 100644 (file)
@@ -55,7 +55,6 @@ for (var i = 0; i < arguments.length; ++i) {
   argument = arguments[i].value;
   func("context.compileShader(argument)");
   func("context.linkProgram(argument)");
-  func("context.attachShader(argument)");
   func("context.attachShader(program, argument)");
   func("context.attachShader(argument, shader)");
   func("context.detachShader(program, argument)");
index 6513e68..1a94672 100644 (file)
@@ -34,8 +34,8 @@ PASS gl.bindTexture(gl.TEXTURE_2D, texture) was expected value: NO_ERROR.
 PASS gl.blendColor(1.0, 1.0, 1.0, 1.0) was expected value: NO_ERROR.
 PASS gl.blendEquation(gl.FUNC_ADD) was expected value: NO_ERROR.
 PASS gl.blendEquationSeparate(gl.FUNC_ADD, gl.FUNC_ADD) was expected value: NO_ERROR.
-PASS gl.blendFunc(gl.ONE) was expected value: NO_ERROR.
-PASS gl.blendFuncSeparate(gl.ONE, gl.ONE) was expected value: NO_ERROR.
+PASS gl.blendFunc(gl.ONE, gl.ONE) was expected value: NO_ERROR.
+PASS gl.blendFuncSeparate(gl.ONE, gl.ONE, gl.ONE, gl.ONE) was expected value: NO_ERROR.
 PASS gl.bufferData(gl.ARRAY_BUFFER, 0, gl.STATIC_DRAW) was expected value: NO_ERROR.
 PASS gl.bufferData(gl.ARRAY_BUFFER, arrayBufferView, gl.STATIC_DRAW) was expected value: NO_ERROR.
 PASS gl.bufferData(gl.ARRAY_BUFFER, arrayBuffer, gl.STATIC_DRAW) was expected value: NO_ERROR.
index 57d1533..5c0b847 100644 (file)
@@ -132,8 +132,8 @@ function testLostContext()
         "gl.blendColor(1.0, 1.0, 1.0, 1.0)",
         "gl.blendEquation(gl.FUNC_ADD)",
         "gl.blendEquationSeparate(gl.FUNC_ADD, gl.FUNC_ADD)",
-        "gl.blendFunc(gl.ONE)",
-        "gl.blendFuncSeparate(gl.ONE, gl.ONE)",
+        "gl.blendFunc(gl.ONE, gl.ONE)",
+        "gl.blendFuncSeparate(gl.ONE, gl.ONE, gl.ONE, gl.ONE)",
         "gl.bufferData(gl.ARRAY_BUFFER, 0, gl.STATIC_DRAW)",
         "gl.bufferData(gl.ARRAY_BUFFER, arrayBufferView, gl.STATIC_DRAW)",
         "gl.bufferData(gl.ARRAY_BUFFER, arrayBuffer, gl.STATIC_DRAW)",
index 66097aa..98b1c11 100644 (file)
@@ -4,14 +4,14 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE
 
 PASS Program Compiled
 PASS Shader Compiled
-PASS context.compileShader() was expected value: INVALID_VALUE.
-PASS context.linkProgram() was expected value: INVALID_VALUE.
-PASS context.attachShader() was expected value: INVALID_VALUE.
+PASS context.compileShader(undefined) was expected value: INVALID_VALUE.
+PASS context.linkProgram(undefined) was expected value: INVALID_VALUE.
+PASS context.attachShader(undefined, undefined) was expected value: INVALID_VALUE.
 PASS context.attachShader(program, undefined) was expected value: INVALID_VALUE.
 PASS context.attachShader(undefined, shader) was expected value: INVALID_VALUE.
 PASS context.detachShader(program, undefined) was expected value: INVALID_VALUE.
 PASS context.detachShader(undefined, shader) was expected value: INVALID_VALUE.
-PASS context.shaderSource() was expected value: INVALID_VALUE.
+PASS context.shaderSource(undefined, undefined) was expected value: INVALID_VALUE.
 PASS context.shaderSource(undefined, 'foo') was expected value: INVALID_VALUE.
 PASS context.bindAttribLocation(undefined, 0, 'foo') was expected value: INVALID_VALUE.
 PASS context.bindBuffer(context.ARRAY_BUFFER, 0) threw exception TypeError: Type error.
index 48785d5..70c4fb6 100644 (file)
@@ -20,14 +20,14 @@ var shouldGenerateGLError = wtu.shouldGenerateGLError;
 
 assertMsg(program != null, "Program Compiled");
 assertMsg(shader != null, "Shader Compiled");
-shouldGenerateGLError(context, context.INVALID_VALUE, "context.compileShader()");
-shouldGenerateGLError(context, context.INVALID_VALUE, "context.linkProgram()");
-shouldGenerateGLError(context, context.INVALID_VALUE, "context.attachShader()");
+shouldGenerateGLError(context, context.INVALID_VALUE, "context.compileShader(undefined)");
+shouldGenerateGLError(context, context.INVALID_VALUE, "context.linkProgram(undefined)");
+shouldGenerateGLError(context, context.INVALID_VALUE, "context.attachShader(undefined, undefined)");
 shouldGenerateGLError(context, context.INVALID_VALUE, "context.attachShader(program, undefined)");
 shouldGenerateGLError(context, context.INVALID_VALUE, "context.attachShader(undefined, shader)");
 shouldGenerateGLError(context, context.INVALID_VALUE, "context.detachShader(program, undefined)");
 shouldGenerateGLError(context, context.INVALID_VALUE, "context.detachShader(undefined, shader)");
-shouldGenerateGLError(context, context.INVALID_VALUE, "context.shaderSource()");
+shouldGenerateGLError(context, context.INVALID_VALUE, "context.shaderSource(undefined, undefined)");
 shouldGenerateGLError(context, context.INVALID_VALUE, "context.shaderSource(undefined, 'foo')");
 shouldGenerateGLError(context, context.INVALID_VALUE, "context.bindAttribLocation(undefined, 0, 'foo')");
 shouldThrow("context.bindBuffer(context.ARRAY_BUFFER, 0)");
index 3a69c3f..109b448 100644 (file)
@@ -1,3 +1,12 @@
+2011-07-14  Mark Pilgrim  <pilgrim@chromium.org>
+
+        Remove LegacyDefaultOptionalArguments flag from WebGL
+        https://bugs.webkit.org/show_bug.cgi?id=64549
+
+        Reviewed by Kenneth Russell.
+
+        * html/canvas/WebGLRenderingContext.idl:
+
 2011-07-14  Yuta Kitamura  <yutak@chromium.org>
 
         WebSocket: Introduce pywebsocket-0.6b2
index 553acd9..5103af0 100644 (file)
@@ -27,7 +27,6 @@ module html {
 
     interface [
         Conditional=WEBGL,
-        LegacyDefaultOptionalArguments,
         InterfaceUUID=98fb48ae-7216-489c-862b-8e1217fc4443,
         ImplementationUUID=ab4f0781-152f-450e-9546-5b3987491a54,
         CustomMarkFunction,