RESOLVED INVALID 136585
Remove statically used varyings from packing restrictions check 
https://bugs.webkit.org/show_bug.cgi?id=136585
Summary Remove statically used varyings from packing restrictions check
Roger Fong
Reported 2014-09-05 13:55:00 PDT
When checking packing restrictions on a shader's varyings, make sure to not include statically used varyings. Note that this was the reason why blink was passing this test and webkit was not. They were filtering out these statically used varyings before calling the packing restrictions check.
Attachments
patch (2.81 KB, patch)
2014-09-05 14:06 PDT, Roger Fong 		dino: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Roger Fong
Comment 1 2014-09-05 14:06:21 PDT
Created attachment 237709 [details] patch
WebKit Commit Bot
Comment 2 2014-09-05 14:08:31 PDT
Note that there are important steps to take when updating ANGLE. See http://trac.webkit.org/wiki/UpdatingANGLE
Roger Fong
Comment 3 2014-09-05 15:12:02 PDT
http://trac.webkit.org/changeset/173335
WebKit Commit Bot
Comment 4 2014-09-06 00:04:35 PDT
Re-opened since this is blocked by bug 136597
Alexey Proskuryakov
Comment 5 2014-09-06 00:06:08 PDT
This broke webgl/1.0.2/conformance/glsl/misc/shader-varying-packing-restrictions.html, rolling out. Diff is just a change from PASS to FAIL.
Alexey Proskuryakov
Comment 6 2014-09-11 11:02:56 PDT
A different fix is being pursued in bug 136734.
Note You need to log in before you can comment on or make changes to this bug.