Bug 36200

Summary: Temporarily disable WebGL layout tests failing due to multisampling
Product: WebKit Reporter: Kenneth Russell <kbr>
Component: WebGLAssignee: Kenneth Russell <kbr>
Status: RESOLVED FIXED    
Severity: Critical CC: cmarrin, eric, simon.fraser, zmo
Priority: P1    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
eric: review+
Revised patch eric: review+

Description Kenneth Russell 2010-03-16 16:17:49 PDT
After the introduction of multisampling support for WebGL in https://bugs.webkit.org/show_bug.cgi?id=33416 , a few layout tests are failing on some of the commit bots. Detection of buggy OpenGL implementations and workarounds for them are being investigated under https://bugs.webkit.org/show_bug.cgi?id=36194 , but in the interim to unblock the commit queue we need to disable these tests. (The overall code changes in 33416 were blocking other work and it is highly undesirable to roll them back.)
Comment 1 Kenneth Russell 2010-03-16 16:26:47 PDT
Created attachment 50850 [details]
Patch
Comment 2 Eric Seidel (no email) 2010-03-16 16:28:28 PDT
Comment on attachment 50850 [details]
Patch

I think Kenneth meant "build bots".  I don't think the commit-bot is failing these tests, but one of the build bots certainly is:
http://build.webkit.org/results/Leopard%20Intel%20Debug%20(Tests)/r56090%20(11512)/results.html

Either way, I think this is OK.

I would like to see the "fix it" bug instead of this bug listed in the Skipped list.  The ChangeLog could list both if you like.

r=me
Comment 3 Kenneth Russell 2010-03-16 16:36:11 PDT
Created attachment 50852 [details]
Revised patch

Revised patch mentioning the "real" bug for this fix (36194) rather than this one in the Skipped file. Leaving the ChangeLog to avoid confusing webkit-patch.
Comment 4 Eric Seidel (no email) 2010-03-16 16:37:20 PDT
Comment on attachment 50852 [details]
Revised patch

I shoudl have been more clear.  no need to re-post.  But still looks good!
Comment 5 Eric Seidel (no email) 2010-03-16 16:38:53 PDT
webkit-patch finds the "bug" for a diff by grepping the whole diff for bug numbers.  The first one it finds is the winner.

We might yet change that to only grep ChangeLogs, but for now that's how it works.
http://trac.webkit.org/browser/trunk/WebKitTools/Scripts/webkitpy/bugzilla.py#L51
Comment 6 Kenneth Russell 2010-03-16 16:39:26 PDT
Committed r56093: <http://trac.webkit.org/changeset/56093>