Bug 30020

Summary: [chromium] Fix WebGL build after CustomGetter constructor changes
Product: WebKit Reporter: Kenneth Russell <kbr>
Component: WebGLAssignee: Kenneth Russell <kbr>
Severity: Normal CC: cmarrin, commit-queue, dglazkov, oliver
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Description Flags
Patch none

Description Kenneth Russell 2009-10-02 09:32:55 PDT
https://bugs.webkit.org/show_bug.cgi?id=29940 changed the form of V8 bindings for constructors marked CustomGetter. The constructors being proposed in the WebGL spec for new array-like objects don't need this special treatment so the CustomGetter should be changed to JSCCustomGetter. I'm not 100% sure even this attribute is needed but I've built both WebKit and Chromium with this change and verified that it fixes the Chromium WebGL build without affecting the WebKit build.
Comment 1 Kenneth Russell 2009-10-02 09:37:31 PDT
Created attachment 40530 [details]
Comment 2 Dimitri Glazkov (Google) 2009-10-02 11:55:16 PDT
Can we just make these non-custom? Doesn't look like they're doing much.
Comment 3 Kenneth Russell 2009-10-02 14:16:51 PDT
I tried removing the JSCCustomGetter attribute and doing so breaks the WebKit build. It seems this patch is correct for both Chrome and WebKit.
Comment 4 Dimitri Glazkov (Google) 2009-10-02 15:06:57 PDT
Comment on attachment 40530 [details]

Comment 5 WebKit Commit Bot 2009-10-02 15:33:48 PDT
Comment on attachment 40530 [details]

Clearing flags on attachment: 40530

Committed r49049: <http://trac.webkit.org/changeset/49049>
Comment 6 WebKit Commit Bot 2009-10-02 15:33:51 PDT
All reviewed patches have been landed.  Closing bug.