Bug 109521 – [v8] V8HTMLCollection.cpp's WebCore::wrap(HTMLCollection* ...) missing cases in switch() for HTMLPropertiesCollection

• Status: RESOLVED
• Resolution: WONTFIX
• Component: WebKit Misc.

[Thomas Sepez]

Discovered as part of the V8 bindings integrity project.

The code in question reads:

v8::Handle<v8::Object> wrap(HTMLCollection* impl, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate)
{
    ASSERT(impl);
    switch (impl->type()) {
    case FormControls:
        return wrap(static_cast<HTMLFormControlsCollection*>(impl), creationContext, isolate);
    case SelectOptions:
        return wrap(static_cast<HTMLOptionsCollection*>(impl), creationContext, isolate);
    case DocAll:
        return wrap(static_cast<HTMLAllCollection*>(impl), creationContext, isolate);
    default:
        break;
    }

    return V8HTMLCollection::createWrapper(impl, creationContext, isolate);
}

But there is a Source/WebCore/html/HTMLPropertiesCollection.idl.

[Thomas Sepez]

Now tracked at https://code.google.com/p/chromium/issues/detail?id=235578.
Nothing need happen in WebKit.