Closed Bug 1228711 Opened 10 years ago Closed 10 years ago

Cubemap images should be square ("Assertion failure: false (Unexpected GL error.)")

Categories

(Core :: Graphics: CanvasWebGL, defect)

defect
Not set
critical

Tracking

()

RESOLVED FIXED
mozilla45
Tracking Status
firefox45 --- fixed

People

(Reporter: jruderman, Assigned: jgilbert)

Details

(Keywords: assertion, reproducible, testcase, Whiteboard: gfx-noted)

Attachments

(5 files, 1 obsolete file)

Attached file testcase
Assertion failure: false (Unexpected GL error.), at dom/canvas/WebGLTextureUpload.cpp:1130
Attached file stack
I can easily reproduce this on mac.
Keywords: reproducible
Whiteboard: gfx-noted
JavaScript warning: https://bug1228711.bmoattachments.org/attachment.cgi?id=8693165, line 11: Error: WebGL: texImage2D: Unexpected error during upload: 0x0501 texImage2D: dui: 1908/1908/8033 Assertion failure: false (Unexpected GL error.), at /Users/bgirard/mozilla/mozilla-central/tree/dom/canvas/WebGLTextureUpload.cpp:1130
0x0501 = GL_INVALID_VALUE 0x1908 = GL_RGBA 0x8033 = GL_UNSIGNED_SHORT_4_4_4_4
What should webgl be doing with these values?
Flags: needinfo?(jgilbert)
I am stunned that this isn't tested. It's an INVALID_VALUE because it's a cubemap texImage specification that's non-square.
Assignee: nobody → jgilbert
Flags: needinfo?(jgilbert)
Attached file test_cubemap_must_be_square.solo.html (obsolete) —
Attachment #8695567 - Attachment is obsolete: true
Attachment #8695569 - Flags: review?(bgirard)
Attachment #8695570 - Flags: review?(dglastonbury)
Comment on attachment 8695570 [details] [diff] [review] 0002-Cubemap-images-must-be-square.patch Review of attachment 8695570 [details] [diff] [review]: ----------------------------------------------------------------- :benwa can get this.
Attachment #8695570 - Flags: review?(dglastonbury) → review?(bgirard)
Attachment #8695569 - Flags: review?(bgirard) → review+
Comment on attachment 8695570 [details] [diff] [review] 0002-Cubemap-images-must-be-square.patch Review of attachment 8695570 [details] [diff] [review]: ----------------------------------------------------------------- Thanks!
Attachment #8695570 - Flags: review?(bgirard) → review+
Summary: "Assertion failure: false (Unexpected GL error.)" → Cubemap images should be square ("Assertion failure: false (Unexpected GL error.)")
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: