Closed
Bug 1291722
Opened 8 years ago
Closed 6 years ago
WebGL crash: Assertion failure: false (GFX: We should have caught all other errors.) [@mozilla::WebGLTexture::CopyTexImage2D]
Categories
(Core :: Graphics: CanvasWebGL, defect, P3)
Core
Graphics: CanvasWebGL
Tracking
()
RESOLVED
DUPLICATE
of bug 1290634
Tracking | Status | |
---|---|---|
firefox51 | --- | affected |
People
(Reporter: posidron, Assigned: jerry)
References
Details
(Keywords: crash, testcase, Whiteboard: [gfx-noted])
Attachments
(3 files)
(deleted),
text/html
|
Details | |
(deleted),
text/plain
|
Details | |
(deleted),
patch
|
Details | Diff | Splinter Review |
No description provided.
Reporter | ||
Comment 1•8 years ago
|
||
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → hshih
Status: NEW → ASSIGNED
Whiteboard: [gfx-noted]
Assignee | ||
Updated•8 years ago
|
Assignee | ||
Comment 2•8 years ago
|
||
GLContext return invalid_enum for:
CopyTexImage2D(this=0x000000012a621800, target=GL_TEXTURE_CUBE_MAP_POSITIVE_Y, level=0, internalFormat=GL_LUMINANCE, x=0, y=0, width=0, height=0, border=0)
The current framebuffer format is rgb8. I think the GL_LUMINANCE format is valid for rgb8.
I'm trying to simplify the test case.
Reporter | ||
Comment 3•8 years ago
|
||
Yes, thanks for reducing the testcase, I was hoping it would be a simple error and you could figure it out easily.
Assignee | ||
Comment 5•8 years ago
|
||
Cool, Jeff!
gles2 have GL_LUMINANCE, but core profile doesn't support that. That's why there is an invalid_enum error.
There is no assert with the patch in bug 1290634.
Assignee | ||
Comment 6•8 years ago
|
||
Attachment #8778219 -
Flags: review?(jgilbert)
Updated•7 years ago
|
Priority: -- → P3
Updated•6 years ago
|
Attachment #8778219 -
Flags: review?(jgilbert)
Comment 7•6 years ago
|
||
Conformance tests now cover this.
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•