Bug 109521 - [v8] V8HTMLCollection.cpp's WebCore::wrap(HTMLCollection* ...) missing cases in switch() for HTMLPropertiesCollection
Summary: [v8] V8HTMLCollection.cpp's WebCore::wrap(HTMLCollection* ...) missing cases ...
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-02-11 16:54 PST by Thomas Sepez
Modified: 2013-04-25 13:50 PDT (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Sepez 2013-02-11 16:54:56 PST
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.
Comment 1 Thomas Sepez 2013-04-25 13:50:14 PDT
Now tracked at https://code.google.com/p/chromium/issues/detail?id=235578.
Nothing need happen in WebKit.