Bug 37018

Summary:

4 WebGL tests fail on my Mac Pro

Product:

WebKit

Reporter:

Eric Seidel (no email) <eric>

Component:

Tools / Tests

Assignee:

Zhenyao Mo <zmo>

Status:

RESOLVED WORKSFORME

Severity:

Normal

CC:

bfulgham, cmarrin, enne, eric, kbr, simon.fraser, zmo

Priority:

Version:

528+ (Nightly build)

Hardware:

OS:

OS X 10.5

Attachments:

Description	Flags
fast/canvas/webgl/context-attributes-alpha-depth-stencil-antialias-diffs.txt	none
fast/canvas/webgl/tex-image-and-sub-image-2d-with-image-data-diffs.txt	none
fast/canvas/webgl/viewport-unchanged-upon-resize-diffs.txt	none
abbreviated system profiler report	none

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	Details
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	Details
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	Details
abbreviated system profiler report (1.75 KB, text/plain) 2010-04-07 23:42 PDT, Eric Seidel (no email)	no flags	Details
View All Add attachment proposed patch, testcase, etc.

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.