Khronos Public Bugzilla

Bug 582

Summary:	Add test checking that getProgramInfoLog returns a string
Product:	WebGL	Reporter:	Benoit Jacob <bjacob>
Component:	Conformance Tests	Assignee:	3dweb Working Group email alias <3dweb>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3	CC:	gman, kbr, zmo
Version:	unspecified
Target Milestone:	---
Hardware:	All
OS:	All
Attachments:	Check that the infolog is a string

Description Benoit Jacob 2012-01-25 08:30:13 PST

We seem to be missing a test checking that the return value of getProgramInfoLog is a string.

The WebGL spec doesn't say anything special about getProgramInfoLog, but it returns a DOMString so it's unexpected for it to return null.

Chromium sometimes returns null instead of a (possibly empty) string:

http://code.google.com/codesearch#OAMlx_jo-ck/src/content/renderer/gpu/webgraphicscontext3d_command_buffer_impl.cc&exact_package=chromium&type=cs&l=796

We have come across a practical application that didn't run because of that.

Comment 1 Benoit Jacob 2012-01-25 08:34:11 PST

See http://code.google.com/p/chromium/issues/detail?id=111337

Comment 2 Benoit Jacob 2012-01-25 08:58:38 PST

Created attachment 98 [details]
Check that the infolog is a string

We should also check if similar tests are needed for other string-returning functions such as getShaderInfoLog.

Comment 3 Benoit Jacob 2012-01-25 09:03:37 PST

Hm, Chromium does pass this test here (does return empty strings). I wonder how to hit the case in 

http://code.google.com/codesearch#OAMlx_jo-ck/src/content/renderer/gpu/webgraphicscontext3d_command_buffer_impl.cc&exact_package=chromium&type=cs&l=796

Comment 4 Benoit Jacob 2012-01-25 09:07:57 PST

(In reply to comment #3)
> Hm, Chromium does pass this test here (does return empty strings). I wonder how
> to hit the case in 
> 
> http://code.google.com/codesearch#OAMlx_jo-ck/src/content/renderer/gpu/webgraphicscontext3d_command_buffer_impl.cc&exact_package=chromium&type=cs&l=796

I'm told that this is because I'm on Linux and this bug only occurs on Windows and Mac.

Comment 5 Benoit Jacob 2012-01-26 12:55:01 PST

Fixed in r16739, r16741, r16742.