Reenable WebGL index-validation.html test

RESOLVED FIXED in mozilla19



Canvas: WebGL
5 years ago
5 years ago


(Reporter: bjacob, Assigned: bjacob)


Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)



(1 attachment)

It was disabled by b3c814e2d015. It's not too bad, only the large-negative-value test is failing on 32bit machines. Regressed either in bug 732660 or with the porting to new bindings.

Comment 1

5 years ago
This was regressed by:

changeset:   109965:4c134de65570
user:        Benoit Jacob <>
date:        Thu Oct 11 14:23:34 2012 -0400
summary:     Bug 779611 - WebGL extensions should be [NoInterfaceObject] as their compliant IDL interface names would pollute the global object - r=bz

Comment 2

5 years ago
Erm, no, rather by

changeset:   109966:78c16f313547
user:        Benoit Jacob <>
date:        Thu Oct 11 15:23:03 2012 -0400
summary:     Bug 779611 - part 12: remove old-bindings remnants in the WebGL implementation - r=bz

Comment 3

5 years ago
Yeah OK I know what this almost certainly is --- the WebGLintptr typedef was changed to a typedef to GLintptr, and likely these were not actually the same.

Comment 4

5 years ago
OK, indeed: WebGLintptr was long long which is int64

GLintptr is ptrdiff_t.

Whence the bug on 32bit platforms

Comment 5

5 years ago
Created attachment 671006 [details] [diff] [review]
fix WebGL typedefs to reflect IDL types instead of being equal to OpenGL types

Again, the bug was that while most WebGL types are equal to the corresponding OpenGL type, some are different:

WebGLintptr is always long long (int64) in Web IDL while it is intptr_t in OpenGL

WebGLsizeiptr is the same

I also changed WebGLboolean to be bool, following Web IDL.

Also removed now-useless #include "WebGLContext.h" in nsDOMCI. And the #undef None there was almost certainly added there for it (X11 headers have a None macro) and affected WebGLContext.h (which has or used to have a None enum value somewhere). Removed too.

Reenabling the unit test.
Attachment #671006 - Flags: review?(bzbarsky)
Comment on attachment 671006 [details] [diff] [review]
fix WebGL typedefs to reflect IDL types instead of being equal to OpenGL types

Attachment #671006 - Flags: review?(bzbarsky) → review+

Comment 7

5 years ago
Target Milestone: --- → mozilla19

Comment 8

5 years ago
Last Resolved: 5 years ago
Resolution: --- → FIXED

Comment 9

5 years ago
Somehow, the part of that patch that actually reenables the test was dropped. Landed now:
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.