Summary: | Auto-generate WebGLRenderingContext overloads in V8 | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Yaar Schnitman <yaar> | ||||||||||||
Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> | ||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||
Severity: | Normal | CC: | commit-queue, kbr | ||||||||||||
Priority: | P2 | ||||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||||
Hardware: | Other | ||||||||||||||
OS: | OS X 10.5 | ||||||||||||||
Attachments: |
|
Description
Yaar Schnitman
2010-04-19 13:14:45 PDT
Created attachment 53712 [details]
Patch
Comment on attachment 53712 [details]
Patch
Work in progress
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? Created attachment 53724 [details]
Why fails?
null-object-behaviour.html
Created attachment 53725 [details]
Why fail too?
uniform-location.html
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.
Created attachment 54111 [details]
Patch
Comment on attachment 54111 [details]
Patch
Feedback incorporated. All tests succeed on chromium and vanilla webkit. Please cq+ if good.
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. Created attachment 54121 [details]
Patch
Comment on attachment 54121 [details] Patch Added a fixme with link to new bug: https://bugs.webkit.org/show_bug.cgi?id=38024 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 on attachment 54121 [details] Patch Clearing flags on attachment: 54121 Committed r58161: <http://trac.webkit.org/changeset/58161> All reviewed patches have been landed. Closing bug. |