Closed Bug 1048743 Opened 10 years ago Closed 8 years ago

WebGL2 - Implement Programs and Shaders

Categories

(Core :: Graphics: CanvasWebGL, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: u480271, Assigned: u480271)

References

Details

Attachments

(1 obsolete file)

Blocks: webgl2
Depends on: 899206
Attachment #8524366 - Flags: review?(jgilbert)
Comment on attachment 8524366 [details] [diff] [review]
[WebGL2] No support for binary shaders

Review of attachment 8524366 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/canvas/WebGLContextState.cpp
@@ +191,5 @@
>                  return WebGLObjectAsJSValue(cx, mBound3DTextures[mActiveTexture].get(), rv);
>              }
> +
> +            case LOCAL_GL_NUM_PROGRAM_BINARY_FORMATS:
> +                return JS::NumberValue(0);

I believe this should be done at the emscripten glue level. If WebGL2 doesn't support binary formats, we shouldn't even expose these entrypoints.
Attachment #8524366 - Flags: review?(jgilbert) → review-
Moved the check to emscripten.

https://github.com/kripken/emscripten/pull/3014
Comment on attachment 8524366 [details] [diff] [review]
[WebGL2] No support for binary shaders

># HG changeset patch
># User Dan Glastonbury <dglastonbury@mozilla.com>
># Date 1414366783 -36000
>#      Mon Oct 27 09:39:43 2014 +1000
># Node ID 144aac1bcb00b801efddeb705b3536628ee81d23
># Parent  c74f8b801a5d40bb7aa3c41f30cf5244d6e731aa
>Bug 1048743 - [WebGL2] No support for binary shaders. r=jgilbert
>
>diff --git a/dom/canvas/WebGLContextState.cpp b/dom/canvas/WebGLContextState.cpp
>--- a/dom/canvas/WebGLContextState.cpp
>+++ b/dom/canvas/WebGLContextState.cpp
>@@ -185,16 +185,19 @@ WebGLContext::GetParameter(JSContext* cx
>                 GLint val;
>                 gl->fGetIntegerv(pname, &val);
>                 return JS::NumberValue(uint32_t(val));
>             }
> 
>             case LOCAL_GL_TEXTURE_BINDING_3D: {
>                 return WebGLObjectAsJSValue(cx, mBound3DTextures[mActiveTexture].get(), rv);
>             }
>+
>+            case LOCAL_GL_NUM_PROGRAM_BINARY_FORMATS:
>+                return JS::NumberValue(0);
>         }
>     }
> 
>     switch (pname) {
>         //
>         // String params
>         //
>         case LOCAL_GL_VENDOR:
Attachment #8524366 - Attachment is obsolete: true
I believe we can call this fixed.
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: