|Summary:||Temporarily disable WebGL layout tests failing due to multisampling|
|Product:||WebKit||Reporter:||Kenneth Russell <kbr>|
|Component:||WebGL||Assignee:||Kenneth Russell <kbr>|
|Severity:||Critical||CC:||cmarrin, eric, simon.fraser, zmo|
|Version:||528+ (Nightly build)|
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 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