|Summary:||Auto-generate WebGLRenderingContext overloads in V8|
|Product:||WebKit||Reporter:||Yaar Schnitman <yaar>|
|Component:||New Bugs||Assignee:||Nobody <webkit-unassigned>|
|Version:||528+ (Nightly build)|
|OS:||OS X 10.5|
Description Yaar Schnitman 2010-04-19 13:14:45 PDT
Auto-generate WebGLRenderingContext overloads in V8
Comment 2 Yaar Schnitman 2010-04-19 13:19:32 PDT
Comment on attachment 53712 [details] Patch Work in progress
Comment 3 Yaar Schnitman 2010-04-19 13:24:11 PDT
Ken, this is still work in progress, but I have some questions: 1. Two WebGL tests fail on seemingly unrelated reasons (see 2 attachments). Any clues why would that happen? 2. The ExceptionCode argument seems not to be used really in texImage2D and texSubImage2D. Am I right? If so, this patch could be made much smaller. 3. Any other overloads I'm missing?
Comment 4 Yaar Schnitman 2010-04-19 15:01:11 PDT
Created attachment 53724 [details] Why fails? null-object-behaviour.html
Comment 5 Yaar Schnitman 2010-04-19 15:02:04 PDT
Created attachment 53725 [details] Why fail too? uniform-location.html
Comment 6 Kenneth Russell 2010-04-20 12:03:48 PDT
Comment on attachment 53712 [details] Patch The update to the texImageTest looks okay but since the JSC custom bindings aren't changed in this patch I suspect they will start failing when run inside the WebKit build. Have you verified this? If this is the case, then the JSC custom bindings need to be temporarily modified so that the new test passes. The failures look to me like your LayoutTests directory is out of date with respect to WebCore.
Comment 8 Yaar Schnitman 2010-04-22 17:16:16 PDT
Comment on attachment 54111 [details] Patch Feedback incorporated. All tests succeed on chromium and vanilla webkit. Please cq+ if good.
Comment 9 Kenneth Russell 2010-04-22 17:30:51 PDT
There is a failure baked in to texImageTest-expected.txt. Either the JSC custom bindings should be modified so that this does not fail, or a FIXME should be added to the test (and another bug filed) to correct this failure in the future.
Comment 11 Yaar Schnitman 2010-04-22 19:44:09 PDT
Comment on attachment 54121 [details] Patch Added a fixme with link to new bug: https://bugs.webkit.org/show_bug.cgi?id=38024
Comment 12 Adam Barth 2010-04-23 00:37:50 PDT
Comment on attachment 54121 [details] Patch I don't 100% understand the test change, but I trust you to follow up in the bug you filed. I love the massive removal of custom code.
Comment 13 WebKit Commit Bot 2010-04-23 02:42:11 PDT
Comment on attachment 54121 [details] Patch Clearing flags on attachment: 54121 Committed r58161: <http://trac.webkit.org/changeset/58161>
Comment 14 WebKit Commit Bot 2010-04-23 02:42:16 PDT
All reviewed patches have been landed. Closing bug.