Closed
Bug 1313584
Opened 8 years ago
Closed 8 years ago
Hit MOZ_CRASH(GFX: Unhandled pname) at /Volumes/firefoxos/gecko-dev/dom/canvas/WebGLSampler.cpp:112
Categories
(Core :: Graphics: CanvasWebGL, defect)
Core
Graphics: CanvasWebGL
Tracking
()
RESOLVED
FIXED
mozilla52
People
(Reporter: vliu, Assigned: vliu)
References
Details
Attachments
(2 files)
7.18 KB,
text/x-log
|
Details | |
2.08 KB,
patch
|
mtseng
:
review+
gchang
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
Found on Nightly build. STR: 1. Load dom/canvas/test/webgl-conf/checkout/deqp/functional/gles3/samplerstatequery.html
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → vliu
Assignee | ||
Comment 1•8 years ago
|
||
To fulfill the whole test coverage in [1], the proposed patch intends to handle all of this. [1] was be called by [2] and [3] for different type. [1]: https://dxr.mozilla.org/mozilla-central/source/dom/canvas/test/webgl-conf/checkout/deqp/functional/gles3/es3fSamplerStateQueryTests.js#152 [2]: https://dxr.mozilla.org/mozilla-central/source/dom/canvas/test/webgl-conf/checkout/deqp/functional/gles3/es3fSamplerStateQueryTests.js#125 [3]: https://dxr.mozilla.org/mozilla-central/source/dom/canvas/test/webgl-conf/checkout/deqp/functional/gles3/es3fSamplerStateQueryTests.js#135
Attachment #8805465 -
Flags: review?(mtseng)
Updated•8 years ago
|
Attachment #8805465 -
Flags: review?(mtseng) → review+
Assignee | ||
Comment 2•8 years ago
|
||
try result: https://treeherder.mozilla.org/#/jobs?repo=try&tochange=3b8e8245bf62&fromchange=3c6c078b293f1ff829197689ac53047aca754dea&selectedJob=30418326
Pushed by vliu@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/bd64f2ee2d83 Add complete parameters handling for SamplerParameterX to cover test need. r=mtseng
Comment 4•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/bd64f2ee2d83
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox52:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
Assignee | ||
Comment 5•8 years ago
|
||
[Tracking Requested - why for this release]: This bug is for fixing webgl2 conformance test crashed. We need this fix uplift to FF-51.
tracking-firefox51:
--- → ?
Comment 6•8 years ago
|
||
Track 51+ as this is Web GL 2 conformance test. Hi :vliu, Do you think this is worth uplifting to 51 aurora?
Flags: needinfo?(vliu)
Updated•8 years ago
|
status-firefox51:
--- → affected
Comment 7•8 years ago
|
||
Comment on attachment 8805465 [details] [diff] [review] Add complete parameters handling for SamplerParameterX to cover test need. Approval Request Comment [Feature/regressing bug #]: webgl2 [User impact if declined]: [Describe test coverage new/current, TreeHerder]: [Risks and why]: [String/UUID change made/needed]:
Attachment #8805465 -
Flags: approval-mozilla-aurora?
Assignee | ||
Comment 8•8 years ago
|
||
(In reply to Gerry Chang [:gchang] from comment #6) > Track 51+ as this is Web GL 2 conformance test. > > Hi :vliu, > Do you think this is worth uplifting to 51 aurora? Yes and thanks to Jeff for requesting aurora.
Flags: needinfo?(vliu)
Comment 9•8 years ago
|
||
Comment on attachment 8805465 [details] [diff] [review] Add complete parameters handling for SamplerParameterX to cover test need. WebGL 2 is the feature we want to ship in 51. Aurora51+.
Attachment #8805465 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Updated•8 years ago
|
Blocks: webgl2-blockers
You need to log in
before you can comment on or make changes to this bug.
Description
•