From 418305ae94bb68ca9da7bbda78d8b8fdcc8abf62 Mon Sep 17 00:00:00 2001
From: "aaronleventhal%moonset.net"
 <aaronleventhal%moonset.net@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Tue, 6 Sep 2005 14:07:25 +0000
Subject: [PATCH] Bug 306437. change event fires twice for HTML select
 (mousedown & mouseup). r=dbaron, sr=neil, a=schrep

git-svn-id: svn://10.0.0.236/branches/MOZILLA_1_8_BRANCH@179692 18797224-902f-48f8-a5cc-f745e15eee43
---
 mozilla/layout/forms/nsListControlFrame.cpp | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/mozilla/layout/forms/nsListControlFrame.cpp b/mozilla/layout/forms/nsListControlFrame.cpp
index a90bf17ff35..cf7f9d0a034 100644
--- a/mozilla/layout/forms/nsListControlFrame.cpp
+++ b/mozilla/layout/forms/nsListControlFrame.cpp
@@ -2785,9 +2785,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) {