1255283 - Improve the correctness of EXT_disjoint_timer_query test

Status: RESOLVED FIXED

(Core :: Graphics: CanvasWebGL, defect)

Description

[Jeff Muizelaar [:jrmuizel]]

Attachment: Improve the correctness of EXT_disjoint_timer_query test

This is needed for us to pass on windows. There are two main changes:
1. Spin the event loop before asking for availability
2. Only test TIMESTAMP_EXT if it is supported. (It's not with D3D11)

Comment 1

[Kelsey Gilbert [:jgilbert]]

Comment on attachment 8728794 [details] [diff] [review]
Improve the correctness of EXT_disjoint_timer_query test

Review of attachment 8728794 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/canvas/test/webgl-mochitest/test_webgl_disjoint_timer_query.html
@@ +13,5 @@
>  <script>
>  
> +function defer(f)
> +{
> +    setTimeout(f, 0);

window.postMessage also works.

@@ +57,5 @@
> +      var timestampQuery = ext.createQueryEXT();
> +      ext.queryCounterEXT(timestampQuery, ext.TIMESTAMP_EXT);
> +      gl.flush();
> +      defer(function() {
> +      ok(ext.getQueryObjectEXT(timestampQuery, ext.QUERY_RESULT_AVAILABLE_EXT),

Indent the deferred bits please. It's still mildly important for it to be readable.

Comment 2

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/f12f057ff977

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/f12f057ff977