Khronos Public Bugzilla
Bug 582 - Add test checking that getProgramInfoLog returns a string
Add test checking that getProgramInfoLog returns a string
Status: RESOLVED FIXED
Product: WebGL
Classification: Unclassified
Component: Conformance Tests
unspecified
All All
: P3 normal
: ---
Assigned To: 3dweb Working Group email alias
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-01-25 08:30 PST by Benoit Jacob
Modified: 2012-01-26 12:55 PST (History)
3 users (show)

See Also:


Attachments
Check that the infolog is a string (1.04 KB, patch)
2012-01-25 08:58 PST, Benoit Jacob
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.