Bug 36200 – Temporarily disable WebGL layout tests failing due to multisampling

• Status: RESOLVED
• Resolution: FIXED
• Component: WebGL

[Kenneth Russell]

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.)

[Kenneth Russell]

Created attachment 50850 [details]
Patch

[Eric Seidel (no email)]

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

[Kenneth Russell]

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.

[Eric Seidel (no email)]

Comment on attachment 50852 [details]
Revised patch

I shoudl have been more clear.  no need to re-post.  But still looks good!

[Eric Seidel (no email)]

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

[Kenneth Russell]

Committed r56093: <http://trac.webkit.org/changeset/56093>