Commit 5ec33463 authored by ap's avatar ap

Reviewed by Dave Harrison.

        http://bugs.webkit.org/show_bug.cgi?id=11402
        REGRESSION: onChange does not work anymore for 1st item in popup

        * html/HTMLSelectElement.cpp:
        (WebCore::HTMLSelectElement::HTMLSelectElement):
        (WebCore::HTMLSelectElement::reset):
        Set m_lastOnChangeIndex to -1.



git-svn-id: http://svn.webkit.org/repository/webkit/trunk@17603 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 2dbfa899
2006-11-05 Alexey Proskuryakov <ap@nypop.com>
Reviewed by Dave Harrison.
http://bugs.webkit.org/show_bug.cgi?id=11402
REGRESSION: onChange does not work anymore for 1st item in popup
* html/HTMLSelectElement.cpp:
(WebCore::HTMLSelectElement::HTMLSelectElement):
(WebCore::HTMLSelectElement::reset):
Set m_lastOnChangeIndex to -1.
2006-11-04 Darin Adler <darin@apple.com>
Reviewed by Anders.
......@@ -64,7 +64,7 @@ HTMLSelectElement::HTMLSelectElement(Document* doc, HTMLFormElement* f)
, m_size(0)
, m_multiple(false)
, m_recalcListItems(false)
, m_lastOnChangeIndex(0)
, m_lastOnChangeIndex(-1)
, m_activeSelectionAnchorIndex(-1)
, m_activeSelectionEndIndex(-1)
, m_activeSelectionState(false)
......@@ -504,6 +504,7 @@ void HTMLSelectElement::reset()
if (!optionSelected && firstOption)
firstOption->setSelected(true);
setChanged(true);
m_lastOnChangeIndex = -1;
}
void HTMLSelectElement::notifyOptionSelected(HTMLOptionElement *selectedOption, bool selected)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment