Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[surfman] Unexpected WebGL error: 0x502 (1282) [BindVertexArray(None)]' #24593

Closed
Manishearth opened this issue Oct 30, 2019 · 2 comments
Closed

[surfman] Unexpected WebGL error: 0x502 (1282) [BindVertexArray(None)]' #24593

Manishearth opened this issue Oct 30, 2019 · 2 comments

Comments

@Manishearth
Copy link
Member

@Manishearth Manishearth commented Oct 30, 2019

When running BabylonJS demos (like this one on servo+surfman on UWP, I get the following error in 2D mode:

WebGL thread' panicked at 'assertion failed: `(left == right)`
  left: `1282`,
 right: `0`: Unexpected WebGL error: 0x502 (1282) [BindVertexArray(None)]', components\canvas\webgl_thread.rs:1749:9

cc @asajeffrey @jdm

@jdm
Copy link
Member

@jdm jdm commented Oct 30, 2019

I think there are a couple issues at play here, but fundamentally it seems like it's our mistake. We enable the OES_vertex_array_object webgl extension based on the GL_OES_vertex_array_object/GL_ARB_vertex_array_object extensions reported by the driver. These extensions provide GenVertexArraysOES/BindVertexArraysOES/IsVertexArraysOES/DeleteVertexArraysOES functions, but we're not using those in sparkle: https://github.com/servo/sparkle/blob/94325e87bc18c697045be5b17ceb37f04acd9431/src/lib.rs#L78-L86

We're generating bindings for ES3 which include these functions (https://github.com/servo/sparkle/blob/94325e87bc18c697045be5b17ceb37f04acd9431/build.rs#L38), but requesting ES2 contexts which don't support them.

@jdm jdm mentioned this issue Oct 31, 2019
4 of 4 tasks complete
@jdm
Copy link
Member

@jdm jdm commented Nov 1, 2019

Fixed as part of #24482.

@jdm jdm closed this Nov 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.