Found on Nightly build. STR: 1. Load dom/canvas/test/webgl-conf/checkout/deqp/functional/gles3/samplerstatequery.html

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

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

[Tracking Requested - why for this release]: This bug is for fixing webgl2 conformance test crashed. We need this fix uplift to FF-51.

Track 51+ as this is Web GL 2 conformance test. Hi :vliu, Do you think this is worth uplifting to 51 aurora?

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]:

(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.

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+.