[Webkit-unassigned] [Bug 45192] Provide a way to trigger a <select multiple> onchange event on changes

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=45192





--- Comment #2 from Darin Adler <darin at apple.com>  2010-09-03 13:01:00 PST ---
(From update of attachment 66532)
> +    // On a <select multiple> this needs to be handled slightly differently since onchange events
> +    // would have been fired via the mouse events on list box selects.
> +    if (WebCore::HTMLSelectElement* select = core(self)) {
> +        if (!select->multiple())
> +            select->setSelectedIndexByUser(index, true, true);
> +        else
> +            select->setSelectedIndexOnMultipleByUser(index, true);
> +    }

It seems strange that we need to call two different functions. Can we instead just change the setSelectedIndexByUser in WebCore to do the right thing when multiple() is true and usesMenuList() is false? Do we know of any specific problems caused if we do that?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.