From 246dcdf544a41004faae669e6319843edfb5c95c Mon Sep 17 00:00:00 2001
From: "aaronleventhal%moonset.net"
 <aaronleventhal%moonset.net@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Sun, 4 Sep 2005 18:20:56 +0000
Subject: [PATCH] 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
---
 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 a4437af2d08..b4c068b4bba 100644
--- 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) {