Closed
Bug 1178893
Opened 9 years ago
Closed 9 years ago
WebGL conformance test conformance/state/gl-enum-tests fails (independent of hardware)
Categories
(Core :: Graphics: CanvasWebGL, defect)
Tracking
()
RESOLVED
FIXED
mozilla43
Tracking | Status | |
---|---|---|
firefox43 | --- | fixed |
People
(Reporter: jujjyl, Assigned: liammiddlebrook)
Details
(Keywords: foxfood, Whiteboard: [gfx-noted])
Attachments
(1 file)
1.42 KB,
patch
|
u480271
:
review+
|
Details | Diff | Splinter Review |
STR: Visit https://www.khronos.org/registry/webgl/sdk/tests/conformance/state/gl-enum-tests.html?webglVersion=1
Observed: The test reports an error:
FAIL getError expected: INVALID_ENUM. Was NO_ERROR : gl.getParameter(desktopGL['NUM_COMPRESSED_TEXTURE_FORMATS']) should return INVALID_ENUM.
Expected: The test should pass.
This was seen to fail on the following devices:
- SPARK, GIADA, MACBOOK_AIR_OSX, MACBOOK_AIR_WIN, MACBOOK_PRO_OSX, MACBOOK_PRO_WIN, MACMINI, MACPRO, NEXUS-4, NEXUS-5, NEXUS-10, SURFACE, WINDBOX, HASWELL, HPOMEN
See https://bugzilla.mozilla.org/show_bug.cgi?id=1178601 for hardware configuration details of these systems.
Updated•9 years ago
|
Whiteboard: [gfx-noted]
QA Whiteboard: [foxfood-triage]
Assignee | ||
Comment 1•9 years ago
|
||
Here's a patch that worked for me to fix this bug. It looks like someone put an extra case in the switch statement inside of getParamater that wasn't needed.
Updated•9 years ago
|
Attachment #8653021 -
Flags: review?(dglastonbury)
Comment on attachment 8653021 [details] [diff] [review]
bug-1178893-fix.patch
Review of attachment 8653021 [details] [diff] [review]:
-----------------------------------------------------------------
Yup, that looks correct.
Attachment #8653021 -
Flags: review?(dglastonbury) → review+
Updated•9 years ago
|
Assignee: nobody → liammiddlebrook
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Keywords: checkin-needed
Comment 4•9 years ago
|
||
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox43:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in
before you can comment on or make changes to this bug.
Description
•