From ac04d5df23ece9a5a7ff48db92910bb3a3cb14b2 Mon Sep 17 00:00:00 2001 From: "bugzilla%arlen.demon.co.uk" Date: Fri, 17 Sep 2004 10:48:59 +0000 Subject: [PATCH] Bug 258193 nsIDOMMouseEvent::GetButton() broken with contextmenu event p=trev@gtchat.de r=jst sr=bzbarsky git-svn-id: svn://10.0.0.236/trunk@162492 18797224-902f-48f8-a5cc-f745e15eee43 --- mozilla/content/events/src/nsDOMMouseEvent.cpp | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/mozilla/content/events/src/nsDOMMouseEvent.cpp b/mozilla/content/events/src/nsDOMMouseEvent.cpp index c31da0f5504..dd94bb25bae 100644 --- a/mozilla/content/events/src/nsDOMMouseEvent.cpp +++ b/mozilla/content/events/src/nsDOMMouseEvent.cpp @@ -152,7 +152,7 @@ nsDOMMouseEvent::GetButton(PRUint16* aButton) } // If button has been set then use that instead. - if (mButton > 0) { + if (mButton >= 0) { *aButton = (PRUint16)mButton; } else { @@ -173,9 +173,12 @@ nsDOMMouseEvent::GetButton(PRUint16* aButton) case NS_MOUSE_RIGHT_BUTTON_DOWN: case NS_MOUSE_RIGHT_CLICK: case NS_MOUSE_RIGHT_DOUBLECLICK: + case NS_CONTEXTMENU: *aButton = 2; break; default: + // This event doesn't have a mouse button associated with it + *aButton = (PRUint16)-1; break; } }