RESOLVED WORKSFORME 37018
4 WebGL tests fail on my Mac Pro
https://bugs.webkit.org/show_bug.cgi?id=37018
Summary 4 WebGL tests fail on my Mac Pro
Eric Seidel (no email)
Reported 2010-04-02 01:12:39 PDT
fast/canvas/webgl/context-attributes-alpha-depth-stencil-antialias.html fast/canvas/webgl/tex-image-and-sub-image-2d-with-image-data.html fast/canvas/webgl/viewport-unchanged-upon-resize.html These seem to pass fine on the commit-queue bot and the Leopard build bot, however on my desktop these fail. I'm happy to provide you with any hardware profiles you may need. I'll attach diffs in a sec.
Attachments
fast/canvas/webgl/context-attributes-alpha-depth-stencil-antialias-diffs.txt (1.97 KB, text/plain)
2010-04-02 01:13 PDT, Eric Seidel (no email)
no flags
fast/canvas/webgl/tex-image-and-sub-image-2d-with-image-data-diffs.txt (2.50 KB, text/plain)
2010-04-02 01:14 PDT, Eric Seidel (no email)
no flags
fast/canvas/webgl/viewport-unchanged-upon-resize-diffs.txt (648 bytes, text/plain)
2010-04-02 01:15 PDT, Eric Seidel (no email)
no flags
abbreviated system profiler report (1.75 KB, text/plain)
2010-04-07 23:42 PDT, Eric Seidel (no email)
no flags
Eric Seidel (no email)
Comment 1 2010-04-02 01:13:55 PDT
Created attachment 52401 [details] fast/canvas/webgl/context-attributes-alpha-depth-stencil-antialias-diffs.txt
Eric Seidel (no email)
Comment 2 2010-04-02 01:14:33 PDT
Created attachment 52402 [details] fast/canvas/webgl/tex-image-and-sub-image-2d-with-image-data-diffs.txt
Eric Seidel (no email)
Comment 3 2010-04-02 01:15:04 PDT
Created attachment 52403 [details] fast/canvas/webgl/viewport-unchanged-upon-resize-diffs.txt
Zhenyao Mo
Comment 4 2010-04-02 07:59:21 PDT
Eric, what graphics card do you have on your mac?
Zhenyao Mo
Comment 5 2010-04-06 10:21:00 PDT
Eric, could you provide your OS and Graphics Card information? Also, do those failure happen consistently or randomly?
Eric Seidel (no email)
Comment 6 2010-04-07 23:38:43 PDT
These failures happen every time.
Eric Seidel (no email)
Comment 7 2010-04-07 23:42:25 PDT
Created attachment 52838 [details] abbreviated system profiler report
Zhenyao Mo
Comment 8 2010-04-08 11:10:50 PDT
The cause of the problem might be delay issue for glReadPixels. Let's see if the readPixels patch might fix this problem (we flush before reading). https://bugs.webkit.org/show_bug.cgi?id=34718
Eric Seidel (no email)
Comment 9 2010-04-14 10:54:21 PDT
It's actually four tests as of a few days ago. I ran the tests again just now and all 4 still fail: fast/canvas/webgl/context-attributes-alpha-depth-stencil-antialias.html = TEXT fast/canvas/webgl/copy-tex-image-and-sub-image-2d.html = TEXT fast/canvas/webgl/tex-image-and-sub-image-2d-with-image-data.html = TEXT fast/canvas/webgl/viewport-unchanged-upon-resize.html = TEXT
Zhenyao Mo
Comment 10 2010-04-14 11:04:40 PDT
So I guess it's not due to glReadPixels().
Eric Seidel (no email)
Comment 11 2010-04-14 11:27:13 PDT
Running the same tests in Chrome 5.0375.6, fails only fast/canvas/webgl/context-attributes-alpha-depth-stencil-antialias.html and only one subtest: Testing depth = true PASS webGL = getWebGL('depthOn', { stencil: false, antialias: false }, [ 0, 0, 0, 1 ], 1, 0) is non-null. PASS contextAttribs = webGL.getContextAttributes() is non-null. PASS contextAttribs.depth is true PASS contextAttribs.alpha is true PASS contextAttribs.stencil is false PASS contextAttribs.antialias is false PASS contextAttribs.premultipliedAlpha is true FAIL pixel should be 0,0,0,255. Was 0,0,0,0.
Eric Seidel (no email)
Comment 12 2010-05-12 10:20:03 PDT
The number is up to 5 now: fast/canvas/webgl/context-attributes-alpha-depth-stencil-antialias.htm fast/canvas/webgl/copy-tex-image-and-sub-image-2d.html fast/canvas/webgl/point-size.html expected fast/canvas/webgl/tex-image-and-sub-image-2d-with-image-data.html fast/canvas/webgl/viewport-unchanged-upon-resize.htm
Kenneth Russell
Comment 13 2010-05-12 11:17:33 PDT
This issue is holding up the WebKit commit queue. See https://bugs.webkit.org/show_bug.cgi?id=38912 .
Brent Fulgham
Comment 14 2014-01-09 20:36:44 PST
These tests now pass.
Note You need to log in before you can comment on or make changes to this bug.