DOMHTMLOptionsCollection is missing some implementation

• Component: WebKit Misc.
• Status: RESOLVED FIXED

[Jon Honeycutt]

All of the DOMHTMLOptionsCollection functions are stubs.

[Jon Honeycutt]

Created attachment 43199 [details]
patch

Comment on attachment 43199 [details]
patch

> +    if (index >= m_collection->length())
> +        return E_FAIL;
> +
> +    *result = DOMNode::createInstance(m_collection->item(index));
> +    return S_OK;

Can you just let HTMLOptionsCollection::item() return 0 and then null-check the result instead of bounds-checking index? It would be more like DOMHTMLCollection::item().

r=me either way

[Jon Honeycutt]

I made Dan's change.

Landed in r51091.