Closed Bug 1324972 Opened 8 years ago Closed 8 years ago

Failures in 2.0.0/deqp/functional/gles3/primitiverestart

Categories

(Core :: Graphics: CanvasWebGL, defect, P1)

Product:
Core
Component:
Graphics: CanvasWebGL
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla53
Tracking Flags:
Tracking Status
firefox51 --- fixed
firefox52 --- fixed
firefox53 --- fixed

People

(Reporter: jgilbert, Assigned: jgilbert)

References

Details

(Whiteboard: gfx-noted)

Attachments

(1 file, 2 obsolete files)

Bug 1324972 (flattened) - Disable unnecessary manual index validation for WebGL 2. -
(deleted), text/x-review-board-request
daoshengmu
: review+
Details
It's in the DrawElements calls, because our index validation kicks in and panics when it sees 0xff, 0xffff, or 0xffffffff. We can just disable index validation for WebGL 2 drivers. D3D10+ means we don't need this validation, and likewise we shouldn't need it for modern GL drivers.
Comment on attachment 8820515 [details] Bug 1324972 (flattened) - Disable unnecessary manual index validation for WebGL 2. - https://reviewboard.mozilla.org/r/100006/#review100498 ::: dom/canvas/WebGLBuffer.cpp:222 (Diff revision 1) > + return false; > + > return mCache->BeenUsedWithMultipleTypes(); > } > > +//// r+ for fixing a nit. What does this line work for? If you want to add comment line, probably you should follow Ln 170.
Attachment #8820515 - Flags: review?(dmu) → review+
Comment on attachment 8820597 [details] Bug 1324972 - Catch ANGLE's zealous index-out-of-bounds INVALID_OPs. - https://reviewboard.mozilla.org/r/100096/#review100566
Attachment #8820597 - Flags: review?(dmu) → review+
Blocks: webgl2-blockers
Comment on attachment 8821800 [details] Bug 1324972 - ANGLE DrawElements validation is wrong. - https://reviewboard.mozilla.org/r/100922/#review101568 r = me if sending a PR to https://github.com/mozilla/angle
Attachment #8821800 - Flags: review?(dmu) → review+
Attachment #8820597 - Attachment is obsolete: true
Attachment #8821800 - Attachment is obsolete: true
Pushed by jgilbert@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/eb4b2c6559b6 (flattened) - Disable unnecessary manual index validation for WebGL 2. - r=daoshengmu
https://hg.mozilla.org/mozilla-central/rev/eb4b2c6559b6
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox53: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Blocks: 1316310
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: