1136416 - WebGL 1.0.3 conformance error: conformance/context/context-lost.html

Status: RESOLVED FIXED

(Core :: Graphics: CanvasWebGL, defect)

Description

[Luke]

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:39.0) Gecko/20100101 Firefox/39.0
Build ID: 20150224030228

Steps to reproduce:

https://www.khronos.org/registry/webgl/sdk/tests/conformance/context/context-lost.html?webglVersion=1


Actual results:

FAIL getError expected: NO_ERROR. Was INVALID_OPERATION : after evaluating: OES_vertex_array_object.bindVertexArrayOES(vertexArrayObject)
FAIL getError expected: NO_ERROR. Was INVALID_OPERATION : after evaluating: OES_vertex_array_object.isVertexArrayOES(vertexArrayObject)
FAIL getError expected: NO_ERROR. Was INVALID_OPERATION : after evaluating: OES_vertex_array_object.deleteVertexArrayOES(vertexArrayObject)


Expected results:

Chrome passes all the tests. Firefox fails the test with both the ANGLE and OpenGL backends.

Comment 1

[Jukka Jylänki]

Tested several device configurations and reproduced this test to fail on all of them: 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) , so this looks like a machine independent issue.

Comment 2

[Ethan Lin[:ethlin]]

Attachment: Remove operation error.

May I just simply remove the operation errors?

Comment 3

[Kelsey Gilbert [:jgilbert] (returns Nov01)]

Comment on attachment 8738898 [details] [diff] [review]
Remove operation error.

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

The spec is ambiguous. I will get clarification.

Comment 4

[Kelsey Gilbert [:jgilbert] (returns Nov01)]

This is:
https://github.com/KhronosGroup/WebGL/issues/1596

Comment 5

[Kelsey Gilbert [:jgilbert] (returns Nov01)]

The clarification is that we should follow the steps in "5.14.14 Detecting and enabling extensions".

Comment 6

[Kelsey Gilbert [:jgilbert] (returns Nov01)]

(In reply to Jeff Gilbert [:jgilbert] from comment #5)
> The clarification is that we should follow the steps in "5.14.14 Detecting
> and enabling extensions".

I got this backwards. Prefer the steps in 5.14 to those in 5.14.14.

Comment 7

[Kelsey Gilbert [:jgilbert] (returns Nov01)]

Comment on attachment 8738898 [details] [diff] [review]
Remove operation error.

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

Let's fix this properly.

We need to remove WebGLExtensionBase::mIsLost, and update all the extensions to support the process listed in 5.14 from the WebGL 1 spec.

Comment 8

[Jukka Jylänki]

Retested this today on my HASWELL PC with

Windows 10 Home 64-bit
3.0 GHz Intel 8-Core i7-5960X
32GB of RAM
3840x2160 pixels display

SLI 2x ASUS NVIDIA GeForce GTX 980 Ti, 4+4GB of VRAM, driver version 365.19
	- nvd3dumx.dll, nvwgf2umx.dll:
		- version 10.18.13.6191
		- date 2/8/2016
		- D3D Version 12.1, WDDM 2.0, WHQL approved

with Firefox Nightly 50.0a1 (2016-06-19), and I see the failure still occurs. Looks like the above patch has not yet made its way in?

Comment 9

[Kelsey Gilbert [:jgilbert] (returns Nov01)]

Attachment: Bug 1136416 - Hack to pass the context-lost test for now. -

Review commit: https://reviewboard.mozilla.org/r/65092/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/65092/

Comment 10

[Kelsey Gilbert [:jgilbert] (returns Nov01)]

I have patches to fix this properly, but it's under-specced, so I'm going to nail down the spec first before doing reviews.

Comment 11

[Ethan Lin[:ethlin]]

Comment on attachment 8772236 [details]
Bug 1136416 - Hack to pass the context-lost test for now. -

https://reviewboard.mozilla.org/r/65092/#review62160

It looks fine to pass the test for now.

Comment 12

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/fef2ed0327d5
Hack to pass the context-lost test for now. - r=ethlin
https://hg.mozilla.org/integration/mozilla-inbound/rev/bdf984d32e47
Mark test.

Comment 13

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/fef2ed0327d5
https://hg.mozilla.org/mozilla-central/rev/bdf984d32e47