Reviewed by Adam.
authoradele <adele@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 19 Mar 2007 18:21:52 +0000 (18:21 +0000)
committeradele <adele@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 19 Mar 2007 18:21:52 +0000 (18:21 +0000)
commitd59b8632a9832e8576961d38c87a12391d3391b7
tree6ac7af0dafee745bf2db7eb9b034d79dbd3970df
parent33ee240eba533787175d6081df580cd11336220f
    Reviewed by Adam.

        Adding comments and a little cleanup from my last checkin.

        * html/HTMLSelectElement.cpp:
        (WebCore::HTMLSelectElement::setSelectedIndex): Removed commented out assert.  Added comment about how we use onChange.
        (WebCore::HTMLSelectElement::selectAll): Added comment about how we use saveLastSelection and onChange.
        (WebCore::HTMLSelectElement::dispatchFocusEvent): ditto.
        (WebCore::HTMLSelectElement::dispatchBlurEvent): ditto.
        (WebCore::HTMLSelectElement::menuListDefaultEventHandler): ditto.
        (WebCore::HTMLSelectElement::listBoxDefaultEventHandler): ditto.
        (WebCore::HTMLSelectElement::menuListOnChange): Added assert that usesMenuList() is true.
        (WebCore::HTMLSelectElement::listBoxOnChange): Added assert that usesMenuList() is false.
        (WebCore::HTMLSelectElement::saveLastSelection): Added early return for menu lists.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20322 268f45cc-cd09-0410-ab3c-d52691b4dbfc
WebCore/ChangeLog
WebCore/html/HTMLSelectElement.cpp