WebGL2 - Implement Programs and Shaders

RESOLVED FIXED

Status

()

Core
Canvas: WebGL
RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: kamidphish, Assigned: kamidphish)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 obsolete attachment)

(Assignee)

Description

3 years ago
https://www.khronos.org/registry/webgl/specs/latest/2.0/#3.7.6
(Assignee)

Updated

3 years ago
Blocks: 889977
Depends on: 899206
Keywords: dev-doc-needed
(Assignee)

Comment 1

3 years ago
Created attachment 8524366 [details] [diff] [review]
[WebGL2] No support for binary shaders
Attachment #8524366 - Flags: review?(jgilbert)
(Assignee)

Comment 2

3 years ago
https://tbpl.mozilla.org/?tree=Try&rev=b400100fb2e2
(Assignee)

Comment 3

3 years ago
https://tbpl.mozilla.org/?tree=Try&rev=22098a327b98
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-
(Assignee)

Comment 5

3 years ago
Moved the check to emscripten.

https://github.com/kripken/emscripten/pull/3014
(Assignee)

Comment 6

3 years ago
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
Last Resolved: 2 years ago
Resolution: --- → FIXED
Keywords: dev-doc-needed
You need to log in before you can comment on or make changes to this bug.