[chromium-dev] Re: V8 Generated Constructors

2009-06-19 Thread kylep
Ah! That did the trick; thanks. On Jun 19, 10:24 am, Dimitri Glazkov wrote: > Since Audio object will be instantiated via JS, we need to add extra 2 > lines to that huge switch statement in V8Proxy::GetTemplate: > >     case V8ClassIndex::HTMLIMAGEELEMENT: >       desc->SetCallHandler(USE_CALLBA

[chromium-dev] Re: V8 Generated Constructors

2009-06-19 Thread Dimitri Glazkov
Since Audio object will be instantiated via JS, we need to add extra 2 lines to that huge switch statement in V8Proxy::GetTemplate: case V8ClassIndex::HTMLIMAGEELEMENT: desc->SetCallHandler(USE_CALLBACK(HTMLImageElementConstructor)); break; :DG< On Fri, Jun 19, 2009 at 10:05 AM

[chromium-dev] Re: V8 Generated Constructors

2009-06-19 Thread kylep
Ok, I see what you mean, but the macro in v8_proxy.h points to DOM_NODE_TYPES in v8_index.h, which includes VIDEO_HTMLELEMENT_TYPES (V), which conditionally defines V(HTMLAUDIOELEMENT, HTMLAudioElement) if video is enabled (which it is in webkit.gyp). So although that's not exactly the same as

[chromium-dev] Re: V8 Generated Constructors

2009-06-18 Thread Dimitri Glazkov
You're almost there. You also need to make sure to register it in v8_proxy.cpp (look for Image as a pattern to follow). BTW, with gyp, dependencies in bindings are pretty robust. No need to clobber anymore. :DG< On Thu, Jun 18, 2009 at 4:05 PM, kylep wrote: > > Ok, so. So far I've created a new

[chromium-dev] Re: V8 Generated Constructors

2009-06-18 Thread kylep
Ok, that didn't work. On Jun 18, 4:11 pm, Andrew Scherkus wrote: > My only quick suggestion would be: did you try a clobber build?  My > experience with bindings is they never seem to incrementally build or link > very well :( > > > > On Thu, Jun 18, 2009 at 4:05 PM, kylep wrote: > > > Ok, so.

[chromium-dev] Re: V8 Generated Constructors

2009-06-18 Thread Andrew Scherkus
My only quick suggestion would be: did you try a clobber build? My experience with bindings is they never seem to incrementally build or link very well :( On Thu, Jun 18, 2009 at 4:05 PM, kylep wrote: > > Ok, so. So far I've created a new file > V8HTMLAudioElementConstructor.cpp and modeled it o

[chromium-dev] Re: V8 Generated Constructors

2009-06-18 Thread kylep
Ok, so. So far I've created a new file V8HTMLAudioElementConstructor.cpp and modeled it off V8HTMLImageElementConstructor.cpp (changing all "image"s to "audio"s, plus modified V8CustomBinding.h and webkit.gyp. Change is here: http://codereview.chromium.org/132036 But I'm still getting "TypeError:

[chromium-dev] Re: V8 Generated Constructors

2009-06-16 Thread Dimitri Glazkov
A good place to start would be to look at existing *Constructor.cpp files in WebCore/bindings/v8 and see how they are hooked in (like Image constructor). Also, you have dimich and levin in close proximity you who have added a V8 constructor or two in the past (I think). ;DG< On Mon, Jun 15, 2009