Bug 306437. Onchange events fired twice for option selection via mouseclick. r=dbaron, sr=neil

git-svn-id: svn://10.0.0.236/trunk@179630 18797224-902f-48f8-a5cc-f745e15eee43
2005-09-04 18:20:56 +00:00
parent 5a13630914
commit 246dcdf544
1 changed files with 3 additions and 1 deletions
--- a/mozilla/layout/forms/nsListControlFrame.cpp
+++ b/mozilla/layout/forms/nsListControlFrame.cpp
@@ -2779,9 +2779,11 @@ nsListControlFrame::MouseDown(nsIDOMEvent* aMouseEvent)
    // Handle Like List
    CaptureMouseEvents(GetPresContext(), PR_TRUE);
    mChangesSinceDragStart = HandleListSelection(aMouseEvent, selectedIndex);
+#ifdef ACCESSIBILITY
    if (mChangesSinceDragStart) {
-      UpdateSelection(); // dispatch event, update combobox, etc.
+      FireMenuItemActiveEvent();
    }
+#endif
  } else {
    // NOTE: the combo box is responsible for dropping it down
    if (mComboboxFrame) {